Skip to content

Commit

Permalink
fix(argo-cd): Default applicationSet metrics port to 8085 (#2125)
Browse files Browse the repository at this point in the history
Change applicationSet controller metrics port to 8085

Signed-off-by: Tero Ahonen <[email protected]>
Co-authored-by: Tero Ahonen <[email protected]>
  • Loading branch information
teroahonen-vungle and teroahonen-vungle committed Jun 21, 2023
1 parent c44f10e commit b8c29f7
Show file tree
Hide file tree
Showing 3 changed files with 5 additions and 3 deletions.
4 changes: 3 additions & 1 deletion charts/argo-cd/Chart.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@ appVersion: v2.7.6
kubeVersion: ">=1.23.0-0"
description: A Helm chart for Argo CD, a declarative, GitOps continuous delivery tool for Kubernetes.
name: argo-cd
version: 5.36.5
version: 5.36.6
home: https://github.com/argoproj/argo-helm
icon: https://argo-cd.readthedocs.io/en/stable/assets/logo.png
sources:
Expand All @@ -28,3 +28,5 @@ annotations:
artifacthub.io/changes: |
- kind: changed
description: Upgrade Argo CD to v2.7.6
- kind: changed
description: applicationSet.containerPorts.metrics to 8085
2 changes: 1 addition & 1 deletion charts/argo-cd/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -1044,7 +1044,7 @@ If you want to use an existing Redis (eg. a managed service from a cloud provide
| applicationSet.certificate.privateKey.size | int | `2048` | Key bit size of the private key. If algorithm is set to `Ed25519`, size is ignored. |
| applicationSet.certificate.renewBefore | string | `""` (defaults to 360h = 15d if not specified) | How long before the expiry a certificate should be renewed. |
| applicationSet.certificate.secretName | string | `"argocd-application-controller-tls"` | The name of the Secret that will be automatically created and managed by this Certificate resource |
| applicationSet.containerPorts.metrics | int | `8080` | Metrics container port |
| applicationSet.containerPorts.metrics | int | `8085` | Metrics container port |
| applicationSet.containerPorts.probe | int | `8081` | Probe container port |
| applicationSet.containerPorts.webhook | int | `7000` | Webhook container port |
| applicationSet.containerSecurityContext | object | See [values.yaml] | ApplicationSet controller container-level security context |
Expand Down
2 changes: 1 addition & 1 deletion charts/argo-cd/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -2506,7 +2506,7 @@ applicationSet:
# ApplicationSet controller container ports
containerPorts:
# -- Metrics container port
metrics: 8080
metrics: 8085
# -- Probe container port
probe: 8081
# -- Webhook container port
Expand Down

0 comments on commit b8c29f7

Please sign in to comment.