Merge pull request #138 from indiealexhcloud/feature/non-priv-port-de…

…fault Use non-privileged port for webhook by default
vultr · Oct 17, 2024 · 9e84d9b · 9e84d9b
2 parents 94df500 + 49e86b9
commit 9e84d9b
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 1 deletion.
diff --git a/deploy/cert-manager-webhook-vultr/templates/deployment.yaml b/deploy/cert-manager-webhook-vultr/templates/deployment.yaml
@@ -27,12 +27,13 @@ spec:
           args:
             - --tls-cert-file=/tls/tls.crt
             - --tls-private-key-file=/tls/tls.key
+            - --secure-port={{ .Values.webhook.port }}
           env:
             - name: GROUP_NAME
               value: {{ .Values.groupName | quote }}
           ports:
             - name: https
-              containerPort: 443
+              containerPort: {{ .Values.webhook.port }}
               protocol: TCP
           livenessProbe:
             httpGet:

diff --git a/deploy/cert-manager-webhook-vultr/values.yaml b/deploy/cert-manager-webhook-vultr/values.yaml
@@ -22,6 +22,9 @@ image:
 nameOverride: ""
 fullnameOverride: ""
 
+webhook:
+  port: 4443
+
 service:
   type: ClusterIP
   port: 443