Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore(backend): remove usage of deprecated commonLabels in Kustomize manifests #11289

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Original file line number Diff line number Diff line change
Expand Up @@ -4,10 +4,12 @@ metadata:
name: cache-deployer-deployment
labels:
app: cache-deployer
application-crd-id: kubeflow-pipelines
spec:
replicas: 1
selector:
matchLabels:
application-crd-id: kubeflow-pipelines
app: cache-deployer
strategy:
type: Recreate
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,7 @@ kind: Role
metadata:
labels:
app: kubeflow-pipelines-cache-deployer-role
application-crd-id: kubeflow-pipelines
name: kubeflow-pipelines-cache-deployer-role
rules:
- apiGroups:
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,8 @@ apiVersion: rbac.authorization.k8s.io/v1
kind: RoleBinding
metadata:
name: kubeflow-pipelines-cache-deployer-rolebinding
labels:
application-crd-id: kubeflow-pipelines
roleRef:
apiGroup: rbac.authorization.k8s.io
kind: Role
Expand Down
6 changes: 4 additions & 2 deletions manifests/kustomize/base/cache-deployer/kustomization.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -4,8 +4,10 @@ resources:
- cache-deployer-role.yaml
- cache-deployer-rolebinding.yaml
- cache-deployer-deployment.yaml
commonLabels:
app: cache-deployer
images:
- name: gcr.io/ml-pipeline/cache-deployer
newTag: 2.3.0
labels:
- includeSelectors: true
pairs:
app: cache-deployer
6 changes: 4 additions & 2 deletions manifests/kustomize/base/cache/kustomization.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -6,8 +6,10 @@ resources:
- cache-rolebinding.yaml
- cache-sa.yaml
- cache-service.yaml
commonLabels:
app: cache-server
images:
- name: gcr.io/ml-pipeline/cache-server
newTag: 2.3.0
labels:
- includeSelectors: true
pairs:
app: cache-server
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,8 @@ apiVersion: v1
kind: ConfigMap
metadata:
name: pipeline-install-config
labels:
application-crd-id: kubeflow-pipelines
data:
warning: |
1. Do not use kubectl to edit this configmap, because some values are used
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,8 @@ kind: Secret
apiVersion: v1
metadata:
name: postgres-secret-extended
labels:
application-crd-id: kubeflow-pipelines
stringData:
username: user
password: "password"
Original file line number Diff line number Diff line change
@@ -1,7 +1,9 @@
apiVersion: kustomize.config.k8s.io/v1beta1
kind: Kustomization
commonLabels:
app: cache-server
resources:
- cluster-role.yaml
- cluster-role-binding.yaml
labels:
- includeSelectors: true
pairs:
app: cache-server
Original file line number Diff line number Diff line change
@@ -1,9 +1,6 @@
apiVersion: kustomize.config.k8s.io/v1beta1
kind: Kustomization
namespace: kubeflow
commonLabels:
app.kubernetes.io/name: kubeflow-pipelines
app.kubernetes.io/component: ml-pipeline
resources:
- ../../pipeline/cluster-scoped
- ../../cache-deployer/cluster-scoped
Expand Down Expand Up @@ -31,3 +28,8 @@ patches:

configurations:
- params.yaml
labels:
- includeSelectors: true
pairs:
app.kubernetes.io/component: ml-pipeline
app.kubernetes.io/name: kubeflow-pipelines
Original file line number Diff line number Diff line change
@@ -1,8 +1,6 @@
apiVersion: kustomize.config.k8s.io/v1beta1
kind: Kustomization
namespace: kubeflow
commonLabels:
app: kubeflow-pipelines-profile-controller
resources:
- service.yaml
- deployment.yaml
Expand All @@ -13,4 +11,8 @@ configMapGenerator:
- sync.py
- name: kubeflow-pipelines-profile-controller-env
envs:
- params.env
- params.env
labels:
- includeSelectors: true
pairs:
app: kubeflow-pipelines-profile-controller
Original file line number Diff line number Diff line change
@@ -1,8 +1,10 @@
apiVersion: kustomize.config.k8s.io/v1beta1
kind: Kustomization
namespace: kubeflow
commonLabels:
app: ml-pipeline-ui
resources:
- cluster-role.yaml
- cluster-role-binding.yaml
labels:
- includeSelectors: true
pairs:
app: ml-pipeline-ui
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,12 @@ apiVersion: apps/v1
kind: Deployment
metadata:
name: cache-server
label:
application-crd-id: kubeflow-pipelines
spec:
selector:
matchLabels:
application-crd-id: kubeflow-pipelines
template:
spec:
containers:
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -2,8 +2,16 @@ apiVersion: apps/v1
kind: Deployment
metadata:
name: ml-pipeline
label:
application-crd-id: kubeflow-pipelines
spec:
selector:
matchLabels:
application-crd-id: kubeflow-pipelines
template:
metadata:
label:
application-crd-id: kubeflow-pipelines
spec:
containers:
- name: ml-pipeline-api-server
Expand Down
6 changes: 4 additions & 2 deletions manifests/kustomize/env/aws/kustomization.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -30,5 +30,7 @@ patches:
# Identifier for application manager to apply ownerReference.
# The ownerReference ensures the resources get garbage collected
# when application is deleted.
commonLabels:
application-crd-id: kubeflow-pipelines
labels:
- includeSelectors: true
pairs:
application-crd-id: kubeflow-pipelines
6 changes: 4 additions & 2 deletions manifests/kustomize/env/azure/kustomization.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -21,5 +21,7 @@ secretGenerator:
# Identifier for application manager to apply ownerReference.
# The ownerReference ensures the resources get garbage collected
# when application is deleted.
commonLabels:
application-crd-id: kubeflow-pipelines
labels:
- includeSelectors: true
pairs:
application-crd-id: kubeflow-pipelines
8 changes: 5 additions & 3 deletions manifests/kustomize/env/cert-manager/base/kustomization.yaml
Original file line number Diff line number Diff line change
@@ -1,13 +1,15 @@
apiVersion: kustomize.config.k8s.io/v1beta1
kind: Kustomization
namespace: kubeflow
namespace: kubeflow

resources:
- cache-cert-issuer.yaml
- cache-cert.yaml
- cache-webhook-config.yaml
commonLabels:
app: cache-server-cert-manager

configurations:
- params.yaml
labels:
- includeSelectors: true
pairs:
app: cache-server-cert-manager
12 changes: 5 additions & 7 deletions manifests/kustomize/env/dev/kustomization.yaml
Original file line number Diff line number Diff line change
@@ -1,19 +1,17 @@
apiVersion: kustomize.config.k8s.io/v1beta1
kind: Kustomization

resources:
# Application controller is used to provide Google Cloud Console integration.
resources:
- ../../third-party/application
- ../../base/application
- ../platform-agnostic
- ../gcp/inverse-proxy

# Identifier for application manager to apply ownerReference.
# The ownerReference ensures the resources get garbage collected
# when application is deleted.
commonLabels:
application-crd-id: kubeflow-pipelines

# !!! If you want to customize the namespace,
# please refer sample/cluster-scoped-resources to update the namespace for cluster-scoped-resources
namespace: kubeflow
labels:
- includeSelectors: true
pairs:
application-crd-id: kubeflow-pipelines
12 changes: 5 additions & 7 deletions manifests/kustomize/env/dev/postgresql/kustomization.yaml
Original file line number Diff line number Diff line change
@@ -1,19 +1,17 @@
apiVersion: kustomize.config.k8s.io/v1beta1
kind: Kustomization

resources:
# Application controller is used to provide Google Cloud Console integration.
resources:
- ../../../third-party/application
- ../../../base/application
- ../../platform-agnostic-postgresql
- ../../gcp/inverse-proxy

# Identifier for application manager to apply ownerReference.
# The ownerReference ensures the resources get garbage collected
# when application is deleted.
commonLabels:
application-crd-id: kubeflow-pipelines

# !!! If you want to customize the namespace,
# please refer sample/cluster-scoped-resources to update the namespace for cluster-scoped-resources
namespace: kubeflow
labels:
- includeSelectors: true
pairs:
application-crd-id: kubeflow-pipelines
12 changes: 5 additions & 7 deletions manifests/kustomize/env/gcp/kustomization.yaml
Original file line number Diff line number Diff line change
@@ -1,8 +1,8 @@
apiVersion: kustomize.config.k8s.io/v1beta1
kind: Kustomization

resources:
# Application controller is used to provide Google Cloud Console integration.
resources:
- ../../third-party/application
- ../../base/application
- ../../base/installs/generic
Expand All @@ -12,12 +12,6 @@ resources:
- minio-gcs-gateway
- cloudsql-proxy

# Identifier for application manager to apply ownerReference.
# The ownerReference ensures the resources get garbage collected
# when application is deleted.
commonLabels:
application-crd-id: kubeflow-pipelines

# !!! If you want to customize the namespace,
# please also update base/cache-deployer/cluster-scoped/cache-deployer-clusterrolebinding.yaml
namespace: kubeflow
Expand All @@ -30,3 +24,7 @@ configMapGenerator:
- name: pipeline-install-config
env: params.env
behavior: merge
labels:
- includeSelectors: true
pairs:
application-crd-id: kubeflow-pipelines
10 changes: 4 additions & 6 deletions manifests/kustomize/env/plain-multi-user/kustomization.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -11,12 +11,10 @@ resources:
- ../../third-party/minio/options/istio
- ../../third-party/metacontroller/base

# Identifier for application manager to apply ownerReference.
# The ownerReference ensures the resources get garbage collected
# when application is deleted.
commonLabels:
application-crd-id: kubeflow-pipelines

# !!! If you want to customize the namespace,
# please also update base/cache-deployer/cluster-scoped/cache-deployer-clusterrolebinding.yaml
namespace: kubeflow
labels:
- includeSelectors: true
pairs:
application-crd-id: kubeflow-pipelines
Original file line number Diff line number Diff line change
Expand Up @@ -12,12 +12,10 @@ resources:
- ../../third-party/minio/base
- ../../third-party/minio/options/istio

# Identifier for application manager to apply ownerReference.
# The ownerReference ensures the resources get garbage collected
# when application is deleted.
commonLabels:
application-crd-id: kubeflow-pipelines

# !!! If you want to customize the namespace,
# please also update base/cache-deployer/cluster-scoped/cache-deployer-clusterrolebinding.yaml
namespace: kubeflow
labels:
- includeSelectors: true
pairs:
application-crd-id: kubeflow-pipelines
Original file line number Diff line number Diff line change
Expand Up @@ -12,12 +12,10 @@ resources:
- ../../third-party/minio/base
- ../../third-party/minio/options/istio

# Identifier for application manager to apply ownerReference.
# The ownerReference ensures the resources get garbage collected
# when application is deleted.
commonLabels:
application-crd-id: kubeflow-pipelines

# !!! If you want to customize the namespace,
# please also update base/cache-deployer/cluster-scoped/cache-deployer-clusterrolebinding.yaml
namespace: kubeflow
labels:
- includeSelectors: true
pairs:
application-crd-id: kubeflow-pipelines
Original file line number Diff line number Diff line change
Expand Up @@ -8,12 +8,6 @@ resources:
- ../../third-party/minio/base
- ../../third-party/postgresql/base

# Identifier for application manager to apply ownerReference.
# The ownerReference ensures the resources get garbage collected
# when application is deleted.
commonLabels:
application-crd-id: kubeflow-pipelines

# !!! If you want to customize the namespace,
# please also update base/cache-deployer/cluster-scoped/cache-deployer-clusterrolebinding.yaml
namespace: kubeflow
Original file line number Diff line number Diff line change
Expand Up @@ -6,12 +6,6 @@ resources:
- ../../third-party/tekton-custom-task
- ../plain-multi-user

# Identifier for application manager to apply ownerReference.
# The ownerReference ensures the resources get garbage collected
# when application is deleted.
commonLabels:
application-crd-id: kubeflow-pipelines

patches:
- path: tekton-config.yaml
- patch: |-
Expand Down Expand Up @@ -65,4 +59,8 @@ patches:
- name: ml-pipeline-ui
env:
- name: POD_LOG_CONTAINER_NAME
value: step-user-main
value: step-user-main
labels:
- includeSelectors: true
pairs:
application-crd-id: kubeflow-pipelines
Original file line number Diff line number Diff line change
Expand Up @@ -6,10 +6,6 @@ resources:
- ../../third-party/tekton-custom-task
- ../plain

# Identifier for application manager to apply ownerReference.
# The ownerReference ensures the resources get garbage collected
# when application is deleted.

labels:
- includeSelectors: true
pairs:
Expand Down
Loading