Skip to content

Fix HelmRelease label#58

Closed
gmolau wants to merge 1 commit intofluxcd:mainfrom
gmolau:patch-1
Closed

Fix HelmRelease label#58
gmolau wants to merge 1 commit intofluxcd:mainfrom
gmolau:patch-1

Conversation

@gmolau
Copy link
Copy Markdown

@gmolau gmolau commented Apr 4, 2025

"Revision" is the sequential number that Helm assigns to each release revision (i.e. to each update or rollback of a release). It is an incrementing counter set by Helm itself. chartVersion is the version of the chart semver or otherwise.

Previously the chartVersion was tracked in the "revision" label. This is now changed so that each label contains the right value.

Signed-off-by: Georg Molau <georg.molau@gmail.com>
Copy link
Copy Markdown
Member

@matheuscscp matheuscscp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @gmolau! Can you please test the dashboards with this change and add some screenshots here? 🙏 Thanks!!!

@stefanprodan
Copy link
Copy Markdown
Member

stefanprodan commented Apr 7, 2025

The revision label should be consistent across of Flux resources, in Flux the revision matches the source. I'm not for making this change as it would break everyone's alerts.

You can make this change in your own fork.

@gmolau gmolau closed this Jun 16, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants