Merge pull request #12 from kool-dev/service_ports

Adding support to multiple ports on services + bump generic 1.4.0

charts/generic/Chart.yaml

@@ -1,4 +1,4 @@
 apiVersion: v2
 name: generic
 description: Generic Chart
-version: 1.3.1
+version: 1.4.0

charts/generic/README.md

@@ -0,0 +1,22 @@
+## generic
+
+Kool generic chart, you can deploy pretty much anything using this little guy.
+
+
+### Upgrading to 1.4
+
+Service signature has changed in order to support multiple ports in the same service:
+
+```diff
+-services:
+-  default:
+-    ...
+-    port: 80
+-    # targetPort: 80
++services:
++  default:
++    ...
++    ports:
++      - port: 80
++        # targetPort: 80
+```

charts/generic/templates/services.yaml

@@ -21,8 +21,10 @@ spec:
   externalName: {{ $service.externalName }}
   {{- end }}
   ports:
-    - port: {{ $service.port }}
-      targetPort: {{ $service.targetPort | default $service.port }}
+    {{- range $port := $service.ports }}
+    - port: {{ $port.port }}
+      targetPort: {{ $port.targetPort | default $port.port }}
+    {{- end }}
   selector: {{ $selectorLabels | nindent 4 }}
 ---
 {{- end }}

charts/generic/values.yaml

@@ -187,8 +187,9 @@ serviceAccount:
 #   default:
 #     # fullName:
 #     type: ClusterIP
-#     port: 80
-#     # targetPort: 80
+#     ports:
+#       - port: 80
+#         # targetPort: 80
 
 # ingresses:
 #   - name: app