From f058f765ba5aa09d1f6eb683230932c208a1042a Mon Sep 17 00:00:00 2001 From: Kamesh Sampath Date: Mon, 18 Jul 2022 22:38:39 +0530 Subject: [PATCH] fix: configure nodePort when defined (#81) Allows the chart to configure the nodePort value if defined. --- charts/drone/templates/service.yaml | 3 +++ charts/drone/values.yaml | 3 ++- 2 files changed, 5 insertions(+), 1 deletion(-) diff --git a/charts/drone/templates/service.yaml b/charts/drone/templates/service.yaml index ece70cd..6a5a2f8 100644 --- a/charts/drone/templates/service.yaml +++ b/charts/drone/templates/service.yaml @@ -15,6 +15,9 @@ spec: - port: {{ .Values.service.port }} targetPort: http protocol: TCP + {{- if .Values.service.nodePort }} + nodePort: {{ .Values.service.nodePort }} + {{- end }} name: http selector: {{- include "drone.selectorLabels" . | nindent 4 }} diff --git a/charts/drone/values.yaml b/charts/drone/values.yaml index 372848d..68b6d9d 100644 --- a/charts/drone/values.yaml +++ b/charts/drone/values.yaml @@ -47,8 +47,9 @@ updateStrategy: {} service: type: ClusterIP - port: 80 + port: 8080 annotations: {} + nodePort: ## If you'd like to create an ingress in front of the Drone server, you can enable it ## here. Please refer to your service provider's documenatation for any configuration