Updated default value to be that of default in k8s, ClusterFirst - also added checks so if policy is None one of the configs are set

Author: andrewgkew

helm/robusta/templates/runner.yaml

@@ -43,7 +43,14 @@ spec:
       {{- toYaml . | nindent 8 }}
       {{- end }}
       {{- if .Values.runner.dnsConfig.enabled }}
-      dnsPolicy: {{ .Values.runner.dnsConfig.policy | quote }}
+      {{- $policy := .Values.runner.dnsConfig.policy }}
+      {{- $hasConfig := or .Values.runner.dnsConfig.nameservers .Values.runner.dnsConfig.searches .Values.runner.dnsConfig.options }}
+
+      {{- if and (eq $policy "None") (not $hasConfig) }}
+        {{- fail "dnsConfig: when dnsPolicy is 'None', you must set at least one of nameservers, searches, or options" }}
+      {{- end }}
+      dnsPolicy: {{ $policy | quote }}
+      {{- if $hasConfig }}
       dnsConfig:
         {{- with .Values.runner.dnsConfig.nameservers }}
         nameservers:
@@ -57,6 +64,7 @@ spec:
         options:
           {{- toYaml . | nindent 10 }}
         {{- end }}
+      {{- end }}
       {{ else }}
       dnsPolicy: ClusterFirst
       {{- end }}

helm/robusta/values.yaml

@@ -759,7 +759,7 @@ runner:
   #Enabled custom DNS configuration for runner
   dnsConfig:
     enabled: false
-    policy: None
+    policy: ClusterFirst
     nameservers: []
     searches: []
     options: []