Incorrect flag --metricsport used in Helm chart instead of --metricsPort

[vijaysakhare709]

Describe the bug

The Argo Rollouts Helm chart incorrectly uses --metricsport (all lowercase), which is not recognized by the Argo Rollouts binary. The correct flag is --metricsPort (camelCase), as shown in the CLI help. This causes the container to crash with the error:

sql
Copy
Edit
unknown flag: --metricsport

Related helm chart

argo-rollouts

Helm chart version

latest

To Reproduce

Install Argo Rollouts using the official Helm chart:

bash
Copy
Edit
helm install argo-rollouts argo/argo-rollouts --namespace argo-rollout --create-namespace
Wait for the pod to start:

bash
Copy
Edit
kubectl get pods -n argo-rollout
Check the logs of the controller pod:

bash
Copy
Edit
kubectl logs -n argo-rollout
You will see the error:

sql
Copy
Edit
unknown flag: --metricsport

Expected behavior

The container should start successfully using the correct flag --metricsPort.

Screenshots

Error: unknown flag: --metricsport
Usage:
argo-rollouts [flags]
...
--metricsPort int Set the port the metrics endpoint should be exposed over (default 8090)

Additional context

This issue seems to come from a typo in the Helm chart values or template. The flag name should be updated to match the official CLI documentation.

I’m happy to raise a PR if needed.

[yu-croco]

Hi @vijaysakhare709 , I made a PR ( #3424 ) for this issue and it turns out that the latest version of Argo Rollouts (v1.8.3) has metricsport, so CI in my PR failed.
Ref: https://github.com/argoproj/argo-rollouts/blob/v1.8.3/cmd/rollouts-controller/main.go#L292

CI: https://github.com/argoproj/argo-helm/actions/runs/16872844132/job/47790736445

But master branch has metricsPort.
https://github.com/argoproj/argo-rollouts/blob/master/cmd/rollouts-controller/main.go#L305