Skip to content

Commit

Permalink
DT-326: add optional commonLabels to opik pods (#847)
Browse files Browse the repository at this point in the history
* DT-326: add optional commonLabels to opik pods

* fix
  • Loading branch information
liyaka authored Dec 10, 2024
1 parent 20ebaf0 commit 6e38c84
Show file tree
Hide file tree
Showing 3 changed files with 15 additions and 0 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -9,6 +9,12 @@ spec:
jobTemplate:
spec:
template:
metadata:
labels:
app: clickhouse-backup
{{- with .Values.commonLabels }}
{{- toYaml . | nindent 12 }}
{{- end }}
spec:
containers:
- name: clickhouse-backup-job
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -6,6 +6,9 @@ metadata:
labels:
component: clickhouse
{{- include "opik.labels" $ | nindent 4 }}
{{- with .Values.commonLabels }}
{{- toYaml . | nindent 4 }}
{{- end }}
spec:
defaults:
templates:
Expand Down
6 changes: 6 additions & 0 deletions deployment/helm_chart/opik/templates/deployment.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,9 @@ metadata:
labels:
{{- include "opik.labels" $ | nindent 4 }}
component: {{ include "opik.name" $ }}-{{ $key | lower }}
{{- with $.Values.commonLabels }}
{{- toYaml . | nindent 4 }}
{{- end }}
{{- with $value.annotations }}
annotations: {{ . | toYaml | nindent 4 }}
{{- end }}
Expand All @@ -27,6 +30,9 @@ spec:
labels:
{{- include "opik.labels" $ | nindent 8 }}
component: {{ include "opik.name" $ }}-{{ $key | lower }}
{{- with $.Values.commonLabels }}
{{- toYaml . | nindent 8 }}
{{- end }}
spec:
serviceAccountName: {{ include "opik.name" $ }}-{{ $key | lower }}
{{- with $value.imagePullSecrets }}
Expand Down

0 comments on commit 6e38c84

Please sign in to comment.