Separate out secured and insecured port Helm values (#1488)

Signed-off-by: Ivan Sim <ivan.sim@kasten.io>
kanisterio · Jun 16, 2022 · c33f40f · c33f40f
1 parent 4297c8d
commit c33f40f
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 1 deletion.
diff --git a/helm/kanister-operator/templates/_helpers.tpl b/helm/kanister-operator/templates/_helpers.tpl
@@ -70,6 +70,6 @@ on the value of bpValidatingWebhook.enabled
 {{- if .Values.bpValidatingWebhook.enabled -}}
     {{ .Values.controller.service.port }}
 {{- else -}}
-    {{ 8000 }}
+    {{ .Values.controller.service.insecuredPort }}
 {{- end -}}
 {{- end -}}
diff --git a/helm/kanister-operator/values.yaml b/helm/kanister-operator/values.yaml
@@ -13,7 +13,10 @@ serviceAccount:
   annotations:
 controller:
   service:
+    # port is used as the secured service port if the validating
+    # webhook is enabled. Otherwise, insecuredPort is used.
     port: 443
+    insecuredPort: 8000
   # updateCRDs specifies if kanister controller should create/update the CRDs
   # false : CRDs would be created by helm
   # true : CRDs would be created by kanister controller