Update chart to support multiples ports.

CST-Group · Dec 14, 2021 · 219d72c · 219d72c
1 parent 4e99e2a
commit 219d72c
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 7 deletions.
diff --git a/charts/codelet-application-helm/Chart.yaml b/charts/codelet-application-helm/Chart.yaml
@@ -15,7 +15,7 @@ type: application
 # This is the chart version. This version number should be incremented each time you make changes
 # to the chart and its templates, including the app version.
 # Versions are expected to follow Semantic Versioning (https://semver.org/)
-version: 0.0.5 
+version: 0.0.6 
 
 # This is the version number of the application being deployed. This version number should be
 # incremented each time you make changes to the application. Versions are not expected to

diff --git a/charts/codelet-application-helm/templates/codelet-application-deployment.yaml b/charts/codelet-application-helm/templates/codelet-application-deployment.yaml
@@ -17,8 +17,10 @@ spec:
           image: {{ .Values.container.image.name }}
           imagePullPolicy: {{ .Values.container.image.pullPolicy }}
           ports:
-          - name: deployment-port
-            containerPort: {{ .Values.container.containerPort }}
+            {{- range $key, $val := .Values.container.ports }}
+          - name: {{ $key }}
+            containerPort: {{ $val | quote }}
+         {{- end }}
           env:          
             {{- range $key, $val := .Values.container.env }}
           - name: {{ $key }}

diff --git a/charts/codelet-application-helm/templates/codelet-application-service.yml b/charts/codelet-application-helm/templates/codelet-application-service.yml
@@ -7,7 +7,8 @@ spec:
   selector:
     app: {{ .Release.Name }}-service
   ports:
-    - name: http
-      port: {{ .Values.container.containerPort }}
-      targetPort: deployment-port
-      protocol: TCP
+    {{- range $key, $val := .Values.service.targetPorts }}
+  - name: {{ $key }}
+    containerPort: {{ $val | quote }}
+    protocol: TCP
+ {{- end }}