From 11fc3b4485e52f899974dec9210c30cad4cc5626 Mon Sep 17 00:00:00 2001
From: Yu Inoue <34591767+eternity1984@users.noreply.github.com>
Date: Sat, 18 Nov 2023 06:11:35 +0900
Subject: [PATCH] fix: Separate service port and container port in Helm (#549)

---
 charts/planka/templates/deployment.yaml | 2 +-
 charts/planka/values.yaml               | 4 ++++
 2 files changed, 5 insertions(+), 1 deletion(-)

diff --git a/charts/planka/templates/deployment.yaml b/charts/planka/templates/deployment.yaml
index 60e8201e..dd79eb2f 100644
--- a/charts/planka/templates/deployment.yaml
+++ b/charts/planka/templates/deployment.yaml
@@ -35,7 +35,7 @@ spec:
           imagePullPolicy: {{ .Values.image.pullPolicy }}
           ports:
             - name: http
-              containerPort: {{ .Values.service.port }}
+              containerPort: {{ .Values.service.containerPort | default 1337 }}
               protocol: TCP
           livenessProbe:
             httpGet:
diff --git a/charts/planka/values.yaml b/charts/planka/values.yaml
index 7d6e319a..a5697a7b 100644
--- a/charts/planka/values.yaml
+++ b/charts/planka/values.yaml
@@ -46,6 +46,10 @@ securityContext: {}
 service:
   type: ClusterIP
   port: 1337
+  ## @param service.containerPort Planka HTTP container port
+  ## If empty will default to 1337
+  ##
+  containerPort: 1337
 
 ingress:
   enabled: false