Apply changes from code review

Changes in detail:
- Add missing examples/documentation in values.yaml
- Beautify indentation
- Use 'with' instead of 'if' to don't repeat variable names
- Fix changelog annotation
- Bump minor chart version as this adds a feature

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

charts/argo-cd/Chart.yaml

@@ -2,7 +2,7 @@ apiVersion: v2
 appVersion: 2.1.2
 description: A Helm chart for ArgoCD, a declarative, GitOps continuous delivery tool for Kubernetes.
 name: argo-cd
-version: 3.21.2
+version: 3.22.0
 home: https://github.com/argoproj/argo-helm
 icon: https://argoproj.github.io/argo-cd/assets/logo.png
 keywords:
@@ -21,5 +21,4 @@ dependencies:
     condition: redis-ha.enabled
 annotations:
   artifacthub.io/changes: |
-    - "[Fixed]: Dex deployment now also supports hostAliases"
+    - "[Added]: Support for server.additionalApplications[].info"
-    - "[Fixed]: server.additionalApplications.[].info"

charts/argo-cd/templates/argocd-server/applications.yaml

@@ -2,45 +2,45 @@
 apiVersion: v1
 kind: List
 items:
-{{- range .Values.server.additionalApplications }}
+  {{- range .Values.server.additionalApplications }}
   - apiVersion: argoproj.io/v1alpha1
     kind: Application
     metadata:
-      {{- if .additionalAnnotations }}
+      {{- with .additionalAnnotations }}
       annotations:
-      {{- range $key, $value := .additionalAnnotations }}
+        {{- range $key, $value := . }}
         {{ $key }}: {{ $value | quote }}
+        {{- end }}
       {{- end }}
-      {{- end }}
+      {{- with .additionalLabels }}
-      {{- if .additionalLabels }}
       labels:
-{{- toYaml .additionalLabels | nindent 8 }}
+        {{- toYaml . | nindent 8 }}
       {{- end }}
       name: {{ .name }}
-      {{- if .namespace }}
+      {{- with .namespace }}
-      namespace: {{ .namespace }}
+      namespace: {{ . }}
       {{- end }}
-      {{- if .finalizers }}
+      {{- with .finalizers }}
       finalizers:
-{{- toYaml .finalizers | nindent 8 }}
+        {{- toYaml . | nindent 8 }}
       {{- end }}
     spec:
       project: {{ tpl .project $ }}
       source:
-{{- toYaml .source | nindent 8 }}
+        {{- toYaml .source | nindent 8 }}
       destination:
-{{- toYaml .destination | nindent 8 }}
+        {{- toYaml .destination | nindent 8 }}
-      {{- if .syncPolicy }}
+      {{- with .syncPolicy }}
       syncPolicy:
-{{- toYaml .syncPolicy | nindent 8 }}
+        {{- toYaml . | nindent 8 }}
       {{- end }}
-      {{- if .ignoreDifferences }}
+      {{- with .ignoreDifferences }}
       ignoreDifferences:
-{{- toYaml .ignoreDifferences | nindent 8 }}
+        {{- toYaml . | nindent 8 }}
       {{- end }}
-      {{- if .info }}
+      {{- with .info }}
       info:
-{{- toYaml .info | nindent 8 }}
+        {{- toYaml . | nindent 8 }}
       {{- end }}
-{{- end }}
+  {{- end }}
-{{- end }}
+{{- end }}

charts/argo-cd/values.yaml

@@ -847,13 +847,21 @@ server:
   #     path: guestbook
   #     directory:
   #       recurse: true
-  #  destination:
+  #   destination:
   #     server: https://kubernetes.default.svc
   #     namespace: guestbook
-  #  syncPolicy:
+  #   syncPolicy:
-  #    automated:
+  #     automated:
-  #      prune: false
+  #       prune: false
-  #      selfHeal: false
+  #       selfHeal: false
+  #   ignoreDifferences:
+  #   - group: apps
+  #     kind: Deployment
+  #     jsonPointers:
+  #     - /spec/replicas
+  #   info:
+  #   - name: url
+  #     value: https://argoproj.github.io/
 
   ## Projects
   ## reference: https://github.com/argoproj/argo-cd/blob/master/docs/operator-manual/