Added missing attributes and improvements to deployment yaml file and…

keda/.helmignore

@@ -19,4 +19,4 @@
 .project
 .idea/
 *.tmproj
-.vscode/
+.vscode/

keda/Chart.yaml

@@ -1,12 +1,29 @@
 apiVersion: v1
 name: keda
 description: Event based autoscaler for Azure Functions deployments on Kubernetes
+
+# A chart can be either an 'application' or a 'library' chart.
+#
+# Application charts are a collection of templates that can be packaged into versioned archives
+# to be deployed.
+#
+# Library charts provide useful utilities or functions for the chart developer. They're included as
+# a dependency of application charts to inject those utilities and functions into the rendering
+# pipeline. Library charts do not define any templates and therefore cannot be deployed.
+type: application
+
+# This is the chart version. This version number should be incremented each time you make changes
+# to the chart and its templates, including the app version.
 version: 1.1.0
-appVersion: "1.1.0"
+
+# This is the version number of the application being deployed. This version number should be
+# incremented each time you make changes to the application.
+appVersion: 1.1.0
+
 home: https://github.com/kedacore/keda
 icon: https://raw.githubusercontent.com/kedacore/keda/master/images/keda-logo-500x500-white.png
 sources:
   - https://github.com/kedacore/keda
 maintainers:
   - name: Ahmed ElSayed
-    email: ahmels@microsoft.com
+    email: ahmels@microsoft.com

keda/crds/keda.k8s.io_triggerauthentications_crd.yaml

@@ -80,4 +80,4 @@ spec:
   versions:
   - name: v1alpha1
     served: true
-    storage: true
+    storage: true

keda/templates/api_service.yaml → keda/templates/apiservice.yaml

@@ -1,13 +1,13 @@
-apiVersion: apiregistration.k8s.io/v1beta1
-kind: APIService
-metadata:
-  name: v1beta1.external.metrics.k8s.io
-spec:
-  service:
-    name: {{ .Values.operatorName }}
-    namespace: {{ .Release.Namespace }}
-  group: external.metrics.k8s.io
-  version: v1beta1
-  insecureSkipTLSVerify: true
-  groupPriorityMinimum: 100
-  versionPriority: 100
+apiVersion: apiregistration.k8s.io/v1beta1
+kind: APIService
+metadata:
+  name: v1beta1.external.metrics.k8s.io
+spec:
+  service:
+    name: {{ .Values.operatorName }}
+    namespace: {{ .Release.Namespace }}
+  group: external.metrics.k8s.io
+  version: v1beta1
+  insecureSkipTLSVerify: true
+  groupPriorityMinimum: 100
+  versionPriority: 100

keda/templates/operator.yaml → keda/templates/deployment.yaml

@@ -1,10 +1,10 @@
 apiVersion: apps/v1
 kind: Deployment
 metadata:
-  labels:
-    app: {{ .Values.operatorName }}
   name: {{ .Values.operatorName }}
   namespace: {{ .Release.Namespace }}
+  labels:
+    app: {{ .Values.operatorName }}
 spec:
   replicas: {{ .Values.replicaCount }}
   selector:
@@ -18,14 +18,22 @@ spec:
         aadpodidbinding: {{ .Values.aadPodIdentity }}
         {{- end }}
     spec:
-      serviceAccountName: {{ .Values.operatorName }}
+    {{- with .Values.imagePullSecrets }}
+      imagePullSecrets:
+        {{- toYaml . | nindent 8 }}
+    {{- end }}
+      serviceAccountName: {{ .Values.serviceAccount.name }}
+      securityContext:
+        {{- toYaml .Values.podSecurityContext | nindent 8 }}
       containers:
         - name: {{ .Values.operatorName }}
-          image: "{{ .Values.image.keda }}"
+          securityContext:
+            {{- toYaml .Values.securityContext | nindent 12 }}
+          image: "{{ .Values.image.keda }}:{{ .Chart.AppVersion }}"
           command:
           - keda
           args:
-          - '--zap-level={{ .Values.logLevel }}'
+          - "--zap-level={{ .Values.logLevel }}"
           imagePullPolicy: {{ .Values.image.pullPolicy }}
           env:
             - name: WATCH_NAMESPACE
@@ -41,30 +49,48 @@ spec:
           - name: grpc-certs
             mountPath: /grpccerts
           {{- end }}
+          resources:
+            {{- toYaml .Values.resources | nindent 12 }}
         - name: {{ .Values.operatorName }}-metrics-apiserver
-          image: {{ .Values.image.metricsAdapter }}
+          image: "{{ .Values.image.metricsAdapter }}:{{ .Chart.AppVersion }}"
           imagePullPolicy: {{ .Values.image.pullPolicy }}
           env:
             - name: WATCH_NAMESPACE
               value: ""
           args:
-          - /usr/local/bin/keda-adapter
-          - --secure-port=6443
-          - --logtostderr=true
-          - --v=0
+            - /usr/local/bin/keda-adapter
+            - --secure-port=6443
+            - --logtostderr=true
+            - --v=0
           ports:
-          - containerPort: 6443
-            name: https
-          - containerPort: 8080
-            name: http
+            - containerPort: {{ .Values.service.portHttpsTarget }}
+              name: https
+              protocol: TCP
+            - containerPort: {{ .Values.service.portHttpTarget }}
+              name: http
+              protocol: TCP
           {{- if .Values.grpcTLSCertsSecret }}
           volumeMounts:
           - name: grpc-certs
             mountPath: /grpccerts
           {{- end }}
-        {{- if .Values.grpcTLSCertsSecret }}
-        volumes:
+          resources:
+            {{- toYaml .Values.resources | nindent 12 }}
+      {{- if .Values.grpcTLSCertsSecret }}
+      volumes:
         - name: grpc-certs
           secret:
             secretName: {{ .Values.grpcTLSCertsSecret }}
-        {{- end }}
+      {{- end }}
+      {{- with .Values.nodeSelector }}
+      nodeSelector:
+        {{- toYaml . | nindent 8 }}
+      {{- end }}
+    {{- with .Values.affinity }}
+      affinity:
+        {{- toYaml . | nindent 8 }}
+    {{- end }}
+    {{- with .Values.tolerations }}
+      tolerations:
+        {{- toYaml . | nindent 8 }}
+    {{- end }}