fix(argo-cd): Correct ApplicationSet controller port (argoproj#1200)

* Add default value for the applicationSet controller container port

Signed-off-by: amaury ravanel <amaury.ravanel@gmail.com>

* Apply suggestions from code review

Signed-off-by: Marco Kilchhofer <mkilchhofer@users.noreply.github.com>

Co-authored-by: Marco Kilchhofer <mkilchhofer@users.noreply.github.com>

charts/argo-cd/Chart.yaml

@@ -2,7 +2,7 @@ apiVersion: v2
 appVersion: v2.3.2
 description: A Helm chart for Argo CD, a declarative, GitOps continuous delivery tool for Kubernetes.
 name: argo-cd
-version: 4.2.3
+version: 4.2.4
 home: https://github.com/argoproj/argo-helm
 icon: https://argo-cd.readthedocs.io/en/stable/assets/logo.png
 keywords:
@@ -21,4 +21,4 @@ dependencies:
     condition: redis-ha.enabled
 annotations:
   artifacthub.io/changes: |
-    - "[Fixed]: Correct ArgoCD notification subscriptions type"
+    - "[Fixed]: Correct ArgoCD applicationset controller port"

charts/argo-cd/templates/argocd-applicationset/service.yaml

@@ -15,7 +15,7 @@ spec:
   ports:
   - name: {{ .Values.applicationSet.service.portName }}
     port: {{ .Values.applicationSet.service.port }}
-    targetPort: {{ .Values.applicationSet.containerPort }}
+    targetPort: webhook
   selector:
     {{- include "argo-cd.selectorLabels" (dict "context" . "name" .Values.applicationSet.name) | nindent 4 }}
 {{- end }}

charts/argo-cd/values.yaml

@@ -1896,7 +1896,7 @@ applicationSet:
   # -- Additional containers to be added to the applicationset controller pod
   extraContainers: []
 
-    ## Metrics service configuration
+  ## Metrics service configuration
   metrics:
     # -- Deploy metrics service
     enabled: false