fix: use with for cleaner syntax and un-nest controller worker values

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

@@ -43,13 +43,13 @@ spec:
           {{- if .Values.singleNamespace }}
           - "--namespaced"
           {{- end }}
-          {{- if .Values.controller.scaling.workflowWorkers }}
+          {{- with .Values.controller.workflowWorkers }}
           - "--workflow-workers"
-          - "{{ .Values.controller.scaling.workflowWorkers }}"
+          - {{ . | quote }}
           {{- end }}
-          {{- if .Values.controller.scaling.podWorkers }}
+          {{- if .Values.controller.podWorkers }}
           - "--pod-workers"
-          - "{{ .Values.controller.scaling.podWorkers }}"
+          - {{ . | quote }}
           {{- end }}
           env:
           - name: ARGO_NAMESPACE

charts/argo/values.yaml

@@ -57,9 +57,8 @@ controller:
   #  spec:
   #    ttlStrategy:
   #      secondsAfterCompletion: 84600
-  scaling: {}
-  #   workflowWorkers: 32
-  #   podWorkers: 32
+  # workflowWorkers: 32
+  # podWorkers: 32
   telemetryConfig:
     enabled: false
     path: /telemetry