Merge pull request #27 from devopshobbies/helm

fix(Helm): add configurable NodePort support for web service
devopshobbies · Nov 4, 2024 · acf28db · acf28db
2 parents 6eceab8 + 2314cf7
commit acf28db
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 0 deletions.
diff --git a/helm/templates/web/service.yaml b/helm/templates/web/service.yaml
@@ -10,5 +10,8 @@ spec:
     - port: {{ .Values.web.service.port }}
       targetPort: {{ .Values.web.service.targetPort }}
       protocol: {{ .Values.web.service.protocol }}
+      {{- if eq .Values.web.service.type "NodePort" }}
+      nodePort: {{ .Values.web.service.nodePort }}
+      {{- end }}
   selector:
     {{- include "web.labels" . | nindent 4 }}
diff --git a/helm/values.yaml b/helm/values.yaml
@@ -14,6 +14,7 @@ web:
     port: 80
     targetPort: 8080
     protocol: TCP
+    nodePort: 30080
   environment:
     OPENAI_API_KEY: ""
   nodeSelector: {}