bjw-s / helm-charts

A collection of Helm charts
https://bjw-s.github.io/helm-charts/
Apache License 2.0
580 stars 106 forks source link

inconsistent deployment naming in multi-controller vs single-controller scenarios #299

Open jazzlyn opened 6 months ago

jazzlyn commented 6 months ago

Details

:wave: during the upgrade of app-template to v3.0.2 I noticed the following:

What steps did you take and what happened:

when using 2 controllers, my deployments are named client-devenv and client-integration.

apiVersion: helm.toolkit.fluxcd.io/v2beta2
kind: HelmRelease
metadata:
  name: client
[...]
spec:
  values:
    controllers:
      devenv:
        [...]
      integration:
        [...]

whereas when using just 1 controller, the deployment name is client.

apiVersion: helm.toolkit.fluxcd.io/v2beta2
kind: HelmRelease
metadata:
  name: client
[...]
spec:
  values:
    controllers:
      devenv:
        [...]

What did you expect to happen:

given the example with 1 controller I would assume the deployment name is client-devenv.

Anything else you would like to add:

it is possible to work around with a controller nameOverride: devenv.

bjw-s commented 6 months ago

This is not a chart bug, because the behavior is intentional. I have forgotten to add it to the docs initially and didn't have time to add it in yet :/

Will try to do so this weekend to prevent further confusion.