feat(argo-rollouts): Add possibility to configure healthz and metrics ports to be compatible with Istio

Signed-off-by: Yevhen Tienkaiev <hronom@gmail.com>

charts/argo-rollouts/Chart.yaml

@@ -2,7 +2,7 @@ apiVersion: v2
 appVersion: v1.5.0
 description: A Helm chart for Argo Rollouts
 name: argo-rollouts
-version: 2.28.0
+version: 2.29.0
 home: https://github.com/argoproj/argo-helm
 icon: https://argoproj.github.io/argo-rollouts/assets/logo.png
 keywords:
@@ -18,9 +18,5 @@ annotations:
     fingerprint: 2B8F22F57260EFA67BE1C5824B11F800CD9D2252
     url: https://argoproj.github.io/argo-helm/pgp_keys.asc
   artifacthub.io/changes: |
-    - kind: changed
-      description: Upgrade Argo Rollouts to v1.5.0
     - kind: added
-      description: Introduce chart signing
-    - kind: added
-      description: Ability to provide additional volumes and volumeMounts
+      description: Ability to configure healthz and metrics ports to be compatible with Istio

charts/argo-rollouts/README.md

@@ -75,12 +75,12 @@ For full list of changes please check ArtifactHub [changelog].
 | containerSecurityContext | object | `{}` | Security Context to set on container level |
 | controller.affinity | object | `{}` | Assign custom [affinity] rules to the deployment |
 | controller.component | string | `"rollouts-controller"` | Value of label `app.kubernetes.io/component` |
+| controller.containerPorts.healthz | int | `8080` | Healthz container port |
+| controller.containerPorts.metrics | int | `8090` | Metrics container port |
 | controller.createClusterRole | bool | `true` | flag to enable creation of cluster controller role (requires cluster RBAC) |
 | controller.extraArgs | list | `[]` | Additional command line arguments to pass to rollouts-controller.  A list of flags. |
 | controller.extraContainers | list | `[]` | Literal yaml for extra containers to be added to controller deployment. |
 | controller.extraEnv | list | `[]` | Additional environment variables for rollouts-controller. A list of name/value maps. |
-| controller.healthz.port | int | `8080` |  |
-| controller.healthz.portName | string | `"healthz"` |  |
 | controller.image.pullPolicy | string | `"IfNotPresent"` | Image pull policy |
 | controller.image.registry | string | `"quay.io"` | Registry to use |
 | controller.image.repository | string | `"argoproj/argo-rollouts"` | Repository to use |
@@ -89,9 +89,8 @@ For full list of changes please check ArtifactHub [changelog].
 | controller.livenessProbe | object | See [values.yaml] | Configure liveness [probe] for the controller |
 | controller.metricProviderPlugins | object | `{}` | Configures 3rd party metric providers for controller |
 | controller.metrics.enabled | bool | `false` | Deploy metrics service |
-| controller.metrics.service.port | int | `8090` |  |
-| controller.metrics.service.portName | string | `"metrics"` |  |
-| controller.metrics.service.targetPort | int | `8090` |  |
+| controller.metrics.service.port | int | `8090` | Metrics service port |
+| controller.metrics.service.portName | string | `"metrics"` | Metrics service port name |
 | controller.metrics.serviceMonitor.additionalAnnotations | object | `{}` | Annotations to be added to the ServiceMonitor |
 | controller.metrics.serviceMonitor.additionalLabels | object | `{}` | Labels to be added to the ServiceMonitor |
 | controller.metrics.serviceMonitor.enabled | bool | `false` | Enable a prometheus ServiceMonitor |

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

@@ -37,8 +37,8 @@ spec:
       containers:
       - image: "{{ .Values.controller.image.registry }}/{{ .Values.controller.image.repository }}:{{ default .Chart.AppVersion .Values.controller.image.tag }}"
         args:
-        - --healthzPort={{ .Values.controller.healthz.port }}
-        - --metricsport={{ .Values.controller.metrics.service.port }}
+        - --healthzPort={{ .Values.controller.containerPorts.healthz }}
+        - --metricsport={{ .Values.controller.containerPorts.metrics }}
         {{- if not .Values.clusterInstall }}
         - --namespaced
         {{- end }}
@@ -55,10 +55,10 @@ spec:
         imagePullPolicy: {{ .Values.controller.image.pullPolicy }}
         name: argo-rollouts
         ports:
-        - containerPort: {{ .Values.controller.metrics.service.port }}
-          name: {{ .Values.controller.metrics.service.portName }}
-        - containerPort: {{ .Values.controller.healthz.port }}
-          name: {{ .Values.controller.healthz.portName }}
+        - containerPort: {{ .Values.controller.containerPorts.metrics }}
+          name: metrics
+        - containerPort: {{ .Values.controller.containerPorts.healthz }}
+          name: healthz
         livenessProbe:
           {{- toYaml .Values.controller.livenessProbe | nindent 10 }}
         readinessProbe:

charts/argo-rollouts/templates/controller/metrics-service.yaml

@@ -16,7 +16,7 @@ spec:
   - name: {{ .Values.controller.metrics.service.portName }}
     protocol: TCP
     port: {{ .Values.controller.metrics.service.port }}
-    targetPort: {{ .Values.controller.metrics.service.targetPort }}
+    targetPort: metrics
   selector:
     app.kubernetes.io/component: {{ .Values.controller.component }}
     {{- include "argo-rollouts.selectorLabels" . | nindent 4 }}

charts/argo-rollouts/values.yaml

@@ -106,17 +106,21 @@ controller:
   # -- flag to enable creation of cluster controller role (requires cluster RBAC)
   createClusterRole: true
 
-  healthz:
-    portName: healthz
-    port: 8080
+  # Controller container ports
+  containerPorts:
+    # -- Metrics container port
+    metrics: 8090
+    # -- Healthz container port
+    healthz: 8080
 
   metrics:
     # -- Deploy metrics service
     enabled: false
     service:
+      # -- Metrics service port name
       portName: metrics
+      # -- Metrics service port
       port: 8090
-      targetPort: 8090
     serviceMonitor:
       # -- Enable a prometheus ServiceMonitor
       enabled: false