Skip to content

Commit d81cd38

Browse files
authored
fix(base-cluster): only toYaml if field exists (teutonet#1226)
otherwise `{}` will be inserted, breaking everything
1 parent b9b2154 commit d81cd38

File tree

6 files changed

+6
-10
lines changed

6 files changed

+6
-10
lines changed

charts/base-cluster/templates/_helmRelease.yaml

+1-3
Original file line numberDiff line numberDiff line change
@@ -6,9 +6,7 @@ metadata:
66
name: {{ .name }}
77
namespace: {{ .namespace | default .Release.Namespace }}
88
labels: {{- include "common.labels.standard" . | nindent 4 }}
9-
{{- with .additionalLabels }}
10-
{{- toYaml . | nindent 4 }}
11-
{{- end }}
9+
{{- with .additionalLabels }}{{- toYaml . | nindent 4 }}{{- end }}
1210
spec:
1311
chart:
1412
spec: {{- include "base-cluster.helm.chartSpec" (dict "repo" "cetic" "chart" "static" "context" .context) | nindent 6 }}

charts/base-cluster/templates/cert-manager/rules/certificate-expiration.yaml

+1-1
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@ metadata:
99
name: certificate-expiration
1010
namespace: cert-manager
1111
labels: {{- include "common.labels.standard" . | nindent 4 }}
12-
{{- toYaml .Values.monitoring.labels | nindent 4 }}
12+
{{- with .Values.monitoring.labels }}{{ toYaml . | nindent 4 }}{{- end }}
1313
app.kubernetes.io/component: prometheus
1414
app.kubernetes.io/part-of: cert-manager
1515
spec:

charts/base-cluster/templates/flux/podMonitor.yaml

+1-1
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@ metadata:
99
name: flux
1010
namespace: {{ .Release.Namespace }}
1111
labels: {{- include "common.labels.standard" . | nindent 4 }}
12-
{{- toYaml .Values.monitoring.labels | nindent 4 }}
12+
{{- with .Values.monitoring.labels }}{{- toYaml . | nindent 4 }}{{- end }}
1313
app.kubernetes.io/component: prometheus
1414
app.kubernetes.io/part-of: flux
1515
spec:

charts/base-cluster/templates/flux/rules/flux-status.yaml

+1-1
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@ metadata:
99
name: flux-status
1010
namespace: {{ .Release.Namespace }}
1111
labels: {{- include "common.labels.standard" . | nindent 4 }}
12-
{{- toYaml .Values.monitoring.labels | nindent 4 }}
12+
{{- with .Values.monitoring.labels }}{{- toYaml . | nindent 4 }}{{- end }}
1313
app.kubernetes.io/component: prometheus
1414
app.kubernetes.io/part-of: flux
1515
spec:

charts/base-cluster/templates/global/namespaces.yaml

+1-3
Original file line numberDiff line numberDiff line change
@@ -4,9 +4,7 @@ kind: Namespace
44
metadata:
55
name: {{ $name }}
66
labels: {{- include "common.labels.standard" $ | nindent 4 -}}
7-
{{- with $namespace.additionalLabels -}}
8-
{{- toYaml . | nindent 4 -}}
9-
{{- end }}
7+
{{- with $namespace.additionalLabels -}}{{- toYaml . | nindent 4 -}}{{- end }}
108
---
119
apiVersion: v1
1210
kind: LimitRange

charts/base-cluster/templates/nfs-server-provisioner/rules/storage-size.yaml

+1-1
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@ metadata:
99
name: storage-size
1010
namespace: nfs-server-provisioner
1111
labels: {{- include "common.labels.standard" $ | nindent 4 }}
12-
{{- toYaml .Values.monitoring.labels | nindent 4 }}
12+
{{- with .Values.monitoring.labels }}{{- toYaml . | nindent 4 }}{{- end }}
1313
app.kubernetes.io/component: prometheus
1414
app.kubernetes.io/part-of: nfs-server-provisioner
1515
spec:

0 commit comments

Comments
 (0)