Skip to content

Commit 6423b48

Browse files
authored
[Helm Chart] only add annotations stanza if they are annotations to add (#2580)
only add annotations stanza in scalejob if they are annotations we want to add Signed-off-by: AvivGuiser <[email protected]>
1 parent e32ae9c commit 6423b48

File tree

4 files changed

+8
-0
lines changed

4 files changed

+8
-0
lines changed

Diff for: charts/selenium-grid/templates/chrome-node-scaledjobs.yaml

+2
Original file line numberDiff line numberDiff line change
@@ -6,10 +6,12 @@ kind: ScaledJob
66
metadata:
77
name: {{ template "seleniumGrid.chromeNode.fullname" (list $nodeConfig $) }}
88
namespace: {{ $.Release.Namespace }}
9+
{{- if $.Values.autoscaling.annotations }}
910
annotations:
1011
{{- with $.Values.autoscaling.annotations }}
1112
{{- toYaml . | nindent 4 }}
1213
{{- end }}
14+
{{- end }}
1315
labels:
1416
app: {{ template "seleniumGrid.chromeNode.fullname" (list $nodeConfig $) }}
1517
app.kubernetes.io/name: {{ template "seleniumGrid.chromeNode.fullname" (list $nodeConfig $) }}

Diff for: charts/selenium-grid/templates/edge-node-scaledjob.yaml

+2
Original file line numberDiff line numberDiff line change
@@ -6,10 +6,12 @@ kind: ScaledJob
66
metadata:
77
name: {{ template "seleniumGrid.edgeNode.fullname" (list $nodeConfig $) }}
88
namespace: {{ $.Release.Namespace }}
9+
{{- if $.Values.autoscaling.annotations }}
910
annotations:
1011
{{- with $.Values.autoscaling.annotations }}
1112
{{- toYaml . | nindent 4 }}
1213
{{- end }}
14+
{{- end }}
1315
labels:
1416
app: {{ template "seleniumGrid.edgeNode.fullname" (list $nodeConfig $) }}
1517
app.kubernetes.io/name: {{ template "seleniumGrid.edgeNode.fullname" (list $nodeConfig $) }}

Diff for: charts/selenium-grid/templates/firefox-node-scaledjob.yaml

+2
Original file line numberDiff line numberDiff line change
@@ -6,10 +6,12 @@ kind: ScaledJob
66
metadata:
77
name: {{ template "seleniumGrid.firefoxNode.fullname" (list $nodeConfig $) }}
88
namespace: {{ $.Release.Namespace }}
9+
{{- if $.Values.autoscaling.annotations }}
910
annotations:
1011
{{- with $.Values.autoscaling.annotations }}
1112
{{- toYaml . | nindent 4 }}
1213
{{- end }}
14+
{{- end }}
1315
labels:
1416
app: {{ template "seleniumGrid.firefoxNode.fullname" (list $nodeConfig $) }}
1517
app.kubernetes.io/name: {{ template "seleniumGrid.firefoxNode.fullname" (list $nodeConfig $) }}

Diff for: charts/selenium-grid/templates/relay-node-scaledjobs.yaml

+2
Original file line numberDiff line numberDiff line change
@@ -6,10 +6,12 @@ kind: ScaledJob
66
metadata:
77
name: {{ template "seleniumGrid.relayNode.fullname" (list $nodeConfig $) }}
88
namespace: {{ $.Release.Namespace }}
9+
{{- if $.Values.autoscaling.annotations }}
910
annotations:
1011
{{- with $.Values.autoscaling.annotations }}
1112
{{- toYaml . | nindent 4 }}
1213
{{- end }}
14+
{{- end }}
1315
labels:
1416
app: {{ template "seleniumGrid.relayNode.fullname" (list $nodeConfig $) }}
1517
app.kubernetes.io/name: {{ template "seleniumGrid.relayNode.fullname" (list $nodeConfig $) }}

0 commit comments

Comments
 (0)