Apply suggestions from code review

Co-authored-by: Brandon Mayfield <563214+bmayfi3ld@users.noreply.github.com>
Co-authored-by: Oliver Bähler <oliverbaehler@hotmail.com>
Signed-off-by: chgl <chgl@users.noreply.github.com>

charts/argo-workflows/templates/_helpers.tpl

@@ -94,3 +94,21 @@ Return the appropriate apiVersion for ingress
 {{- print "networking.k8s.io/v1" -}}
 {{- end -}}
 {{- end -}}
+
+{{/*
+Return the appropriate service layout for ingress
+*/}}
+{{- define "argo-workflows.ingress.service" -}}
+{{- if semverCompare "<1.19-0" .kubeVersion -}}
+- backend:
+    serviceName: {{ .serviceName }}
+    servicePort: {{ .servicePort }}
+{{- else -}}
+- backend:
+    service:
+      name: {{ .serviceName }}
+      port: 
+        number: {{ .servicePort }}
+  pathType: ImplementationSpecific
+{{- end -}}
+{{- end -}}

charts/argo-workflows/templates/controller/workflow-controller-deployment.yaml

@@ -24,9 +24,9 @@ spec:
       {{- end }}
     spec:
       serviceAccountName: {{ template "argo-workflows.controllerServiceAccountName" . }}
-      {{- if .Values.controller.podSecurityContext }}
+      {{- with .Values.controller.podSecurityContext }}
       securityContext:
-        {{- toYaml .Values.controller.podSecurityContext | nindent 8 }}
+        {{- toYaml . | nindent 8 }}
       {{- end }}
       containers:
         - name: controller

charts/argo-workflows/templates/controller/workflow-controller-sa.yaml

@@ -2,5 +2,7 @@ apiVersion: v1
 kind: ServiceAccount
 metadata:
   name: {{ template "argo-workflows.controllerServiceAccountName" . }}
+  {{ with .Values.controller.serviceAccountAnnotations }}
   annotations:
-{{ toYaml .Values.controller.serviceAccountAnnotations | indent 4 }}
+    {{- toYaml .| nindent 4 }}
+  {{- end }}

charts/argo-workflows/templates/controller/workflow-controller-service.yaml

@@ -6,8 +6,8 @@ metadata:
   labels:
     {{- include "argo-workflows.labels" (dict "context" . "component" .Values.controller.name "name" .Values.controller.name) | nindent 4 }}
     app.kubernetes.io/version: {{ default .Chart.AppVersion .Values.controller.image.tag | quote }}
-    {{- if .Values.controller.serviceLabels }}
+    {{- with .Values.controller.serviceLabels }}
-    {{ toYaml .Values.controller.serviceLabels | nindent 4}}
+      {{ toYaml . | nindent 4 }}
     {{- end }}
   {{- with .Values.controller.serviceAnnotations }}
   annotations:
@@ -33,5 +33,6 @@ spec:
   type: {{ .Values.controller.serviceType }}
   {{- if and (eq .Values.controller.serviceType "LoadBalancer") .Values.controller.loadBalancerSourceRanges }}
   loadBalancerSourceRanges:
-{{ toYaml .Values.controller.loadBalancerSourceRanges | indent 4 }}{{- end }}
+    {{- toYaml .Values.controller.loadBalancerSourceRanges | nindent 4 }}
+  {{- end }}
 {{- end -}}

charts/argo-workflows/templates/controller/workflow-controller-servicemonitor.yaml

@@ -5,19 +5,19 @@ metadata:
   name: {{ template "argo-workflows.controller.fullname" . }}
   labels:
     {{- include "argo-workflows.labels" (dict "context" . "component" .Values.controller.name "name" .Values.controller.name) | nindent 4 }}
-    {{- if .Values.controller.serviceMonitor.additionalLabels }}
+    {{- with .Values.controller.serviceMonitor.additionalLabels }}
-{{ toYaml .Values.controller.serviceMonitor.additionalLabels | indent 4 }}
+      {{- toYaml . | nindent 4 }}
     {{- end }}
 spec:
   endpoints:
-  {{- if .Values.controller.metricsConfig.enabled }}
+  {{- with .Values.controller.metricsConfig.enabled }}
     - port: metrics
-      path: {{ .Values.controller.metricsConfig.path }}
+      path: {{ . }}
       interval: 30s
   {{- end }}
-  {{- if .Values.controller.telemetryConfig.enabled }}
+  {{- with .Values.controller.telemetryConfig.enabled }}
     - port: telemetry
-      path: {{ .Values.controller.telemetryConfig.path }}
+      path: {{ . }}
       interval: 30s
   {{- end }}
   namespaceSelector:

charts/argo-workflows/templates/controller/workflow-rb.yaml

@@ -3,9 +3,9 @@ apiVersion: rbac.authorization.k8s.io/v1
 kind: RoleBinding
 metadata:
   name: {{ template "argo-workflows.fullname" . }}-workflow
-{{- if .Values.workflow.namespace }}
+  {{- with .Values.workflow.namespace }}
-  namespace: {{ .Values.workflow.namespace }}
+  namespace: {{ . }} 
-{{- end }}
+  {{- end }}
 roleRef:
   apiGroup: rbac.authorization.k8s.io
   kind: Role
@@ -13,7 +13,7 @@ roleRef:
 subjects:
 - kind: ServiceAccount
   name: {{ .Values.workflow.serviceAccount.name }}
-  {{- if .Values.workflow.namespace }}
+  {{- with .Values.workflow.namespace }}
-  namespace: {{ .Values.workflow.namespace }}
+  namespace: {{ . }}
   {{- end }}
 {{- end }}

charts/argo-workflows/templates/controller/workflow-role.yaml

@@ -3,8 +3,8 @@ apiVersion: rbac.authorization.k8s.io/v1
 kind: Role
 metadata:
   name: {{ template "argo-workflows.fullname" . }}-workflow
-  {{- if .Values.workflow.namespace }}
+  {{- with .Values.workflow.namespace }}
-  namespace: {{ .Values.workflow.namespace }}
+  namespace: {{ . }}
   {{- end }}
 rules:
 - apiGroups:

charts/argo-workflows/templates/controller/workflow-sa.yaml

@@ -3,8 +3,8 @@ apiVersion: v1
 kind: ServiceAccount
 metadata:
   name: {{ .Values.workflow.serviceAccount.name }}
-  {{- if .Values.workflow.namespace }}
+  {{- with .Values.workflow.namespace }}
-  namespace: {{ .Values.workflow.namespace }}
+  namespace: {{ . }}
   {{- end }}
   {{- with .Values.workflow.serviceAccount.annotations }}
   annotations:

charts/argo-workflows/templates/server/server-cluster-roles.yaml

@@ -1,10 +1,10 @@
 {{- if .Values.server.enabled }}
 apiVersion: rbac.authorization.k8s.io/v1
-{{- if .Values.singleNamespace }}
+  {{- if .Values.singleNamespace }}
 kind: Role
-{{- else }}
+  {{- else }}
 kind: ClusterRole
-{{- end }}
+  {{- end }}
 metadata:
   name: {{ template "argo-workflows.server.fullname" . }}
   labels:
@@ -30,7 +30,7 @@ rules:
   - list
   - watch
   - delete
-{{- if .Values.server.sso }}
+  {{- if .Values.server.sso }}
 - apiGroups:
   - ""
   resources:
@@ -46,9 +46,7 @@ rules:
   - secrets
   verbs:
   - create
-{{- end}}
+    {{- if .Values.server.sso.rbac }}
-{{- if .Values.server.sso }}
-  {{- if .Values.server.sso.rbac }}
 - apiGroups:
   - ""
   resources:
@@ -56,8 +54,8 @@ rules:
   verbs:
   - get
   - list
+    {{- end }}
   {{- end }}
-{{- end }}
 - apiGroups:
   - ""
   resources:

charts/argo-workflows/templates/server/server-deployment-pdb.yaml

@@ -1,5 +1,4 @@
-{{- if .Values.server.enabled -}}
+{{- if and .Values.server.enabled .Values.server.pdb.enabled -}}
-{{- if .Values.server.pdb.enabled -}}
 apiVersion: policy/v1beta1
 kind: PodDisruptionBudget
 metadata:
@@ -18,4 +17,3 @@ spec:
     matchLabels:
       {{- include "argo-workflows.selectorLabels" (dict "context" . "name" .Values.server.name) | nindent 6 }}
 {{- end -}}
-{{- end -}}

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

@@ -16,17 +16,18 @@ spec:
       labels:
         {{- include "argo-workflows.labels" (dict "context" . "component" .Values.server.name "name" .Values.server.name) | nindent 8 }}
         app.kubernetes.io/version: {{ default .Chart.AppVersion .Values.server.image.tag | quote }}
-        {{- if .Values.server.podLabels }}
+        {{- with .Values.server.podLabels }}
-        {{- toYaml .Values.server.podLabels | nindent 8 }}
+          {{- toYaml . | nindent 8 }}
         {{- end }}
-      {{- if .Values.server.podAnnotations }}
+      {{- with .Values.server.podAnnotations }}
       annotations:
-{{ toYaml .Values.server.podAnnotations | indent 8}}{{- end }}
+        {{- toYaml .Values.server.podAnnotations | nindent 8 }}
+      {{- end }}
     spec:
       serviceAccountName: {{ .Values.server.serviceAccount | quote }}
-      {{- if .Values.server.podSecurityContext }}
+      {{- with .Values.server.podSecurityContext }}
       securityContext:
-        {{- toYaml .Values.server.podSecurityContext | nindent 8 }}
+        {{- toYaml . | nindent 8 }}
       {{- end }}
       containers:
         - name: argo-server
@@ -37,8 +38,8 @@ spec:
           args:
           - server
           - --configmap={{ .Release.Name }}-{{ .Values.controller.name }}-configmap
-          {{- if .Values.server.extraArgs }}
+          {{- with .Values.server.extraArgs }}
-          {{- toYaml .Values.server.extraArgs | nindent 10 }}
+            {{- toYaml . | nindent 10 }}
           {{- end }}
           - "--secure={{ .Values.server.secure }}"
           {{- if .Values.singleNamespace }}
@@ -98,7 +99,7 @@ spec:
       affinity:
         {{- toYaml . | nindent 8 }}
       {{- end }}
-      {{- if .Values.server.priorityClassName }}
+      {{- with .Values.server.priorityClassName }}
-      priorityClassName: {{ .Values.server.priorityClassName }}
+      priorityClassName: {{ . }}
       {{- end }}
 {{- end -}}

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

@@ -3,10 +3,11 @@
 {{- $serviceName := include "argo-workflows.server.fullname" . -}}
 {{- $servicePort := .Values.server.servicePort -}}
 apiVersion: {{ include "argo-workflows.ingress.apiVersion" . }}
+kind: Ingress
 metadata:
   name: {{ template "argo-workflows.server.fullname" . }}
   labels:
-    {{- include "argo-workflows.labels" $ | nindent 4 }}
+   {{- include "argo-workflows.labels" (dict "context" . "component" .Values.server.name "name" .Values.server.name) | nindent 4 }}
     {{- range $key, $value := .Values.server.ingress.labels }}
     {{ $key }}: {{ $value | quote }}
     {{- end }}
@@ -16,24 +17,21 @@ metadata:
     {{- end }}
 spec:
   rules:
+    {{- $kubeVersion := .Capabilities.KubeVersion.GitVersion}}
     {{- range .Values.server.ingress.hosts }}
     - host: {{ . }}
       http:
         paths:
           {{- if $.Values.server.ingress.paths }}
           {{- range $.Values.server.ingress.paths }}
-          - backend:
+          {{- include "argo-workflows.ingress.service" (dict "kubeVersion" $kubeVersion "serviceName" .serviceName "servicePort" .servicePort) | nindent 10 }}
-              serviceName: {{ .serviceName }}
-              servicePort: {{ .servicePort }}
           {{- end }}
           {{- end }}
-          - backend:
+          {{- include "argo-workflows.ingress.service" (dict "kubeVersion" $kubeVersion "serviceName" $serviceName "servicePort" $servicePort) | nindent 10 }}
-              serviceName: {{ $serviceName }}
-              servicePort: {{ $servicePort }}
     {{- end -}}
-  {{- if .Values.server.ingress.tls }}
+    {{- with .Values.server.ingress.tls }}
   tls:
-{{ toYaml .Values.server.ingress.tls | indent 4 }}
+      {{- toYaml . | nindent 4 }}
+    {{- end -}}
   {{- end -}}
 {{- end -}}
-{{- end -}}

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

@@ -3,6 +3,8 @@ apiVersion: v1
 kind: ServiceAccount
 metadata:
   name: {{ template "argo-workflows.serverServiceAccountName" . }}
+  {{- with .Values.server.serviceAccountAnnotations  }}
   annotations:
-{{ toYaml .Values.server.serviceAccountAnnotations | indent 4 }}
+    {{- toYaml . | indent 4 }}
+  {{- end }}
 {{- end -}}

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

@@ -6,14 +6,15 @@ metadata:
   labels:
     {{- include "argo-workflows.labels" (dict "context" . "component" .Values.server.name "name" .Values.server.name) | nindent 4 }}
     app.kubernetes.io/version: {{ default .Chart.AppVersion .Values.server.image.tag | quote }}
-  {{- if .Values.server.serviceAnnotations }}
+  {{- with .Values.server.serviceAnnotations }}
   annotations:
-{{ toYaml .Values.server.serviceAnnotations | indent 4}}{{- end }}
+    {{- toYaml . | nindent 4 }}
+  {{- end }}
 spec:
   ports:
   - port: {{ .Values.server.servicePort }}
-    {{- if .Values.server.servicePortName }}
+    {{- with .Values.server.servicePortName }}
-    name: {{ .Values.server.servicePortName }}
+    name: {{ . }}
     {{- end }}
     targetPort: 2746
   selector:
@@ -25,5 +26,6 @@ spec:
   {{- end }}
   {{- if and (eq .Values.server.serviceType "LoadBalancer") .Values.server.loadBalancerSourceRanges }}
   loadBalancerSourceRanges:
-{{ toYaml .Values.server.loadBalancerSourceRanges | indent 4 }}{{- end }}
+    {{- toYaml .Values.server.loadBalancerSourceRanges | nindent 4 }}
+  {{- end }}
 {{- end -}}

charts/argo/Chart.yaml

@@ -2,7 +2,7 @@ apiVersion: v2
 appVersion: v2.12.5
 description: A Helm chart for Argo Workflows
 name: argo
-version: 0.16.11
+version: 1.0.0
 icon: https://raw.githubusercontent.com/argoproj/argo/master/docs/assets/argo.png
 home: https://github.com/argoproj/argo-helm
 dependencies: