Ingress-nginx: Any reason to change the release tag scheme?

[jchauncey]

It went from nginx-<semver> to controller-<semver>. This broke some automation we have around rebuilding the project for internal releases so just want to make sure if this was intentional or an accident?

[aledbf]

@jchauncey apologies for the change. This is required due to the migration of the helm chart to this repository and the need to release both artifacts from this repository.
The name controller-<semver> is the new one to be used.

[jchauncey]

Sounds good!

[mathieu-benoit]

Just curious @aledbf, so moving forward, the associated helm chart won't be anymore here: https://github.com/helm/charts/tree/master/stable/nginx-ingress but will be here now: https://github.com/kubernetes/ingress-nginx/tree/master/charts/ingress-nginx, correct? So it won't appear anymore in https://hub.helm.sh/charts/stable/nginx-ingress moving forward?

[aledbf]

the associated helm chart won't be anymore here: https://github.com/helm/charts/tree/master/stable/nginx-ingress but will be here now: https://github.com/kubernetes/ingress-nginx/tree/master/charts/ingress-nginx, correct?

Correct.

So it won't appear anymore in https://hub.helm.sh/charts/stable/nginx-ingress moving forward?

The stable repository is being deprecated. Please check https://github.com/helm/charts/issues/21103

[mathieu-benoit]

Go it, thanks!

The stable repository is being deprecated. Please check helm/charts#21103

Didn't know/notice that (but makes a lot of sense to track individually the helm chart release for example), thanks for the info!

[scottrigby]

PS @jchauncey @mathieu-benoit there's now a documented migration path from stable/nginx-ingress to ingress-nginx/ingress-nginx: https://github.com/kubernetes/ingress-nginx/pull/6038 Hope it helps!