feat(argo-workflows): Allow NodePort to be specified for server service (#938)

* Allow NodePort to be specified for server service.

Signed-off-by: Peter Oruba <peter@oruba.de>

* Different version number and different condition in NodePort setting evaluation, both as per PR review suggested.

Signed-off-by: Peter Oruba <peter@oruba.de>

charts/argo-workflows/Chart.yaml

@@ -2,7 +2,7 @@ apiVersion: v2
 name: argo-workflows
 description: A Helm chart for Argo Workflows
 type: application
-version: 0.6.0
+version: 0.7.0
 appVersion: "v3.1.8"
 icon: https://raw.githubusercontent.com/argoproj/argo-workflows/master/docs/assets/argo.png
 home: https://github.com/argoproj/argo-helm
@@ -15,4 +15,4 @@ maintainers:
   - name: benjaminws
 annotations:
   artifacthub.io/changes: |
-    - "[Added]: Add controller metricsConfig.portName parameter"
+    - "[Added]: Add server.serviceNodePort parameter"

charts/argo-workflows/templates/server/server-service.yaml

@@ -17,6 +17,9 @@ spec:
     name: {{ . }}
     {{- end }}
     targetPort: 2746
+    {{- if and (eq .Values.server.serviceType "NodePort") .Values.server.serviceNodePort }}
+    nodePort: {{ .Values.server.serviceNodePort }}
+    {{- end }}
   selector:
     {{- include "argo-workflows.selectorLabels" (dict "context" . "name" .Values.server.name) | nindent 4 }}
   sessionAffinity: None

charts/argo-workflows/values.yaml

@@ -223,6 +223,7 @@ server:
   name: server
   serviceType: ClusterIP
   servicePort: 2746
+  # serviceNodePort: 32746
   # servicePortName: http
   serviceAccount:
     create: true