[stable/coredns] allow annotations on configmap (#20430)

* [stable/coredns] allow annotations on configmap

Signed-off-by: Alex Williams <alex.williams@skyscanner.net>

* [stable/coredns] remove autoscaler servicemonitor

Signed-off-by: Alex Williams <alex.williams@skyscanner.net>

stable/coredns/Chart.yaml

@@ -1,6 +1,6 @@
 apiVersion: v1
 name: coredns
-version: 1.9.1
+version: 1.9.2
 appVersion: 1.6.7
 description: CoreDNS is a DNS server that chains plugins and provides Kubernetes DNS Services
 keywords:

stable/coredns/README.md

@@ -89,6 +89,7 @@ The command removes all the Kubernetes components associated with the chart and
 | `autoscaler.resources.limits.memory`    | Container maximum memory for cluster-proportional-autoscaler                          | `10Mi`                                                      |
 | `autoscaler.resources.requests.cpu`     | Container requested CPU for cluster-proportional-autoscaler                           | `20m`                                                       |
 | `autoscaler.resources.requests.memory`  | Container requested memory for cluster-proportional-autoscaler                        | `10Mi`                                                      |
+| `autoscaler.configmap.annotations`      | Annotations to add to autoscaler config map. For example to stop CI renaming them     | {}                                                          |
 
 See `values.yaml` for configuration notes. Specify each parameter using the `--set key=value[,key=value]` argument to `helm install`. For example,
 

stable/coredns/templates/configmap-autoscaler.yaml

@@ -15,9 +15,13 @@ metadata:
     kubernetes.io/name: "CoreDNS"
     {{- end }}
     app.kubernetes.io/name: {{ template "coredns.name" . }}-autoscaler
-{{- if .Values.customLabels }}
-{{ toYaml .Values.customLabels | indent 4 }}
-{{- end }}
+    {{- if .Values.customLabels }}
+    {{- toYaml .Values.customLabels | nindent 4 }}
+    {{- end }}
+  {{- if .Values.autoscaler.configmap.annotations }}
+  annotations:
+    {{- toYaml .Values.autoscaler.configmap.annotations | nindent 4 }}
+  {{- end }}
 data:
   # When cluster is using large nodes(with more cores), "coresPerReplica" should dominate.
   # If using small nodes, "nodesPerReplica" should dominate.

stable/coredns/templates/deployment-autoscaler.yaml

@@ -38,6 +38,7 @@ spec:
         {{ toYaml .Values.customLabels | nindent 8 }}
         {{- end }}
       annotations:
+        checksum/configmap: {{ include (print $.Template.BasePath "/configmap-autoscaler.yaml") . | sha256sum }}
         {{- if .Values.isClusterService }}
         scheduler.alpha.kubernetes.io/critical-pod: ''
         scheduler.alpha.kubernetes.io/tolerations: '[{"key":"CriticalAddonsOnly", "operator":"Exists"}]'

stable/coredns/templates/servicemonitor-autoscaler.yaml

@@ -1,21 +0,0 @@
-{{- if .Values.autoscaler.enabled }}
----
-apiVersion: v1
-kind: ServiceAccount
-metadata:
-  name: {{ template "coredns.fullname" . }}-autoscaler
-  namespace: {{ .Release.Namespace }}
-  labels:
-    app.kubernetes.io/managed-by: {{ .Release.Service | quote }}
-    app.kubernetes.io/instance: {{ .Release.Name | quote }}
-    helm.sh/chart: "{{ .Chart.Name }}-{{ .Chart.Version | replace "+" "_" }}"
-    {{- if .Values.isClusterService }}
-    k8s-app: {{ .Chart.Name }}-autoscaler
-    kubernetes.io/cluster-service: "true"
-    kubernetes.io/name: "CoreDNS"
-    {{- end }}
-    app.kubernetes.io/name: {{ template "coredns.name" . }}-autoscaler
-{{- if .Values.customLabels }}
-{{ toYaml .Values.customLabels | indent 4 }}
-{{- end }}
-{{- end }}

stable/coredns/values.yaml

@@ -187,3 +187,9 @@ autoscaler:
     limits:
       cpu: "20m"
       memory: "10Mi"
+
+  # Options for autoscaler configmap
+  configmap:
+    ## Annotations for the coredns-autoscaler configmap
+    # i.e. strategy.spinnaker.io/versioned: "false" to ensure configmap isn't renamed
+    annotations: {}