service annotations: add default

karlivory · Aug 30, 2023 · 1f6a923 · 1f6a923
1 parent 4f71816
commit 1f6a923
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 5 deletions.
diff --git a/charts/bind9/Chart.yaml b/charts/bind9/Chart.yaml
@@ -2,5 +2,5 @@ apiVersion: v2
 name: bind9
 description: bind9 primary-secondary (master-slave) setup
 type: application
-version: 0.1.3
+version: 0.1.4
 appVersion: "9.18"
diff --git a/charts/bind9/templates/services.yaml b/charts/bind9/templates/services.yaml
@@ -4,9 +4,9 @@ kind: Service
 metadata:
   name: bind-primary
   namespace: "{{ .Release.Namespace }}"
-  {{- with .Values.primary.service.annotations }}
+  {{- if .Values.primary.service.annotations }}
   annotations:
-    {{- toYaml . | nindent 4 }}
+    {{- toYaml .Values.primary.service.annotations | nindent 4 }}
   {{- end }}
 spec:
   type: ClusterIP
@@ -29,9 +29,9 @@ kind: Service
 metadata:
   name: bind-secondary
   namespace: "{{ .Release.Namespace }}"
-  {{- with .Values.secondary.service.annotations }}
+  {{- if .Values.secondary.service.annotations }}
   annotations:
-    {{- toYaml . | nindent 4 }}
+    {{- toYaml .Values.secondary.service.annotations | nindent 4 }}
   {{- end }}
 spec:
   type: LoadBalancer

diff --git a/charts/bind9/values.yaml b/charts/bind9/values.yaml
@@ -10,6 +10,7 @@ zone:
   dnsRecords: []
 
 primary:
+  annotations: {}
   service:
     # The bind9 primary (master) server needs to have a static ip, since 
     # /etc/bind/named.conf does not accept dns names in its "masters { <master-ip> };"
@@ -19,6 +20,7 @@ primary:
     replicas: 1
 
 secondary:
+  annotations: {}
   service:
     loadBalancerIP: ""
   deployment: