From c58c6c98c58db40b3b78882faa09bfa5a03358b2 Mon Sep 17 00:00:00 2001 From: Yu Inoue Date: Sat, 18 Nov 2023 00:19:54 +0900 Subject: [PATCH 1/2] Fix containerPort to not be affected by service.port --- 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 60e8201ea..1d8ff19cd 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.targetPort | default 1337 }} protocol: TCP livenessProbe: httpGet: diff --git a/charts/planka/values.yaml b/charts/planka/values.yaml index 7d6e319aa..a59fefca7 100644 --- a/charts/planka/values.yaml +++ b/charts/planka/values.yaml @@ -46,6 +46,10 @@ securityContext: {} service: type: ClusterIP port: 1337 + ## @param service.targetPort Planka HTTP container port + ## If empty will default to 1337 + ## + targetPort: 1337 ingress: enabled: false From 55726a54c0ed1607b58eb7725e4b593b0c4992aa Mon Sep 17 00:00:00 2001 From: Yu Inoue Date: Sat, 18 Nov 2023 00:58:44 +0900 Subject: [PATCH 2/2] Rename targetPort to containerPort --- charts/planka/templates/deployment.yaml | 2 +- charts/planka/values.yaml | 4 ++-- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/charts/planka/templates/deployment.yaml b/charts/planka/templates/deployment.yaml index 1d8ff19cd..dd79eb2f8 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.targetPort | default 1337 }} + containerPort: {{ .Values.service.containerPort | default 1337 }} protocol: TCP livenessProbe: httpGet: diff --git a/charts/planka/values.yaml b/charts/planka/values.yaml index a59fefca7..a5697a7bf 100644 --- a/charts/planka/values.yaml +++ b/charts/planka/values.yaml @@ -46,10 +46,10 @@ securityContext: {} service: type: ClusterIP port: 1337 - ## @param service.targetPort Planka HTTP container port + ## @param service.containerPort Planka HTTP container port ## If empty will default to 1337 ## - targetPort: 1337 + containerPort: 1337 ingress: enabled: false