Add chart configurations and deployment health check (#33)

Added replica configuration to the helm chart Added port configuration to the helm chart Added dockerhub link to README
ExpediaGroup · Apr 1, 2022 · adec952 · adec952
1 parent 40884a8
commit adec952
Show file tree

Hide file tree

Showing 5 changed files with 40 additions and 5 deletions.
diff --git a/README.md b/README.md
@@ -4,6 +4,8 @@
 Kubernetes Mutating Webhook
 ===========
 
+https://hub.docker.com/r/expediagroup/kubernetes-sidecar-injector
+
 This [mutating webhook](https://kubernetes.io/docs/reference/access-authn-authz/admission-controllers/#mutatingadmissionwebhook) was developed to inject sidecars to a Kubernetes pod. 
 
 ## Developing

diff --git a/charts/kubernetes-sidecar-injector/Chart.yaml b/charts/kubernetes-sidecar-injector/Chart.yaml
@@ -1,6 +1,6 @@
 name: kubernetes-sidecar-injector
 home: https://github.com/expediagroup/kubernetes-sidecar-injector
-version: 1.0.2
+version: 1.0.3
 appVersion: "0.1"
 keywords:
   - kubernetes

diff --git a/charts/kubernetes-sidecar-injector/templates/mutatingwebhook.yaml b/charts/kubernetes-sidecar-injector/templates/mutatingwebhook.yaml
@@ -79,7 +79,7 @@ metadata:
   labels:
     {{- include "common.labels" . | indent 4 }}
 spec:
-  replicas: 1
+  replicas: {{ .Values.replicaCount }}
   selector:
     matchLabels:
       {{- include "common.labels" . | indent 6 }}
@@ -102,7 +102,7 @@ spec:
           image: "{{ .Values.image.repository }}:{{ .Values.image.tag }}"
           imagePullPolicy: IfNotPresent
           args:
-            - --port=8443
+            - --port={{ .Values.container.port }}
             - --certFile=/opt/kubernetes-sidecar-injector/certs/cert.pem
             - --keyFile=/opt/kubernetes-sidecar-injector/certs/key.pem
             - --injectPrefix={{ trimSuffix "/" .Values.selectors.injectPrefix }}
@@ -112,6 +112,30 @@ spec:
             - name: {{ .Release.Name }}-certs
               mountPath: /opt/kubernetes-sidecar-injector/certs
               readOnly: true
+          ports:
+            - name: https
+              containerPort: {{ .Values.container.port }}
+              protocol: TCP
+          livenessProbe:
+            httpGet:
+              path: /healthz
+              port: https
+              scheme: HTTPS
+            initialDelaySeconds: 5
+            periodSeconds: 10
+            successThreshold: 1
+            failureThreshold: 5
+            timeoutSeconds: 4
+          readinessProbe:
+            httpGet:
+              path: /healthz
+              port: https
+              scheme: HTTPS
+            initialDelaySeconds: 30
+            periodSeconds: 10
+            successThreshold: 1
+            failureThreshold: 5
+            timeoutSeconds: 4
       volumes:
         - name: {{ .Release.Name }}-certs
           secret:

diff --git a/charts/kubernetes-sidecar-injector/templates/service.yaml b/charts/kubernetes-sidecar-injector/templates/service.yaml
@@ -7,7 +7,8 @@ metadata:
     {{- include "common.labels" . | indent 4 }}
 spec:
   ports:
-  - port: 443
-    targetPort: 8443
+  - name: https
+    port: {{ .Values.service.port }}
+    targetPort: https
   selector:
     {{- include "common.labels" . | indent 4 }}
diff --git a/charts/kubernetes-sidecar-injector/values.yaml b/charts/kubernetes-sidecar-injector/values.yaml
@@ -2,6 +2,14 @@ image:
   repository: expediagroup/kubernetes-sidecar-injector
   tag: latest
 
+replicaCount: 1
+
+service:
+  port: 443
+
+container:
+  port: 8443
+
 podAnnotations: {}
 podLabels: {}