Skip to content

stable/redis-ha Consistently use labels.standard - #414

Open
Kidswiss wants to merge 1 commit into
DandyDeveloper:masterfrom
Kidswiss:fix/standard_labels
Open

stable/redis-ha Consistently use labels.standard#414
Kidswiss wants to merge 1 commit into
DandyDeveloper:masterfrom
Kidswiss:fix/standard_labels

Conversation

@Kidswiss

@Kidswiss Kidswiss commented Aug 12, 2026

Copy link
Copy Markdown

What this PR does / why we need it:

Not escaping the .Chart.version cleanly in labels can lead to issues while using the chart via flux helm controller.

Because the controller tracks the digest of any given chart and appends it to the chart version with a + which will result in invalid labels.

Special notes for your reviewer:

Please check if I really caught all instances.

Checklist

  • DCO signed
  • Title of the PR starts with chart name (e.g. [stable/mychartname])

Not escaping the `.Chart.version` cleanly in labels can lead to issues
while using the chart via flux helm controller.

Because the controller tracks the digest of any given chart and appends
it to the chart version with a `+` which will result in invalid labels.

Signed-off-by: Simon Beck <simon.beck@vshn.ch>
@Kidswiss
Kidswiss force-pushed the fix/standard_labels branch from 98daeb2 to 1054f2f Compare August 12, 2026 11:44
@Kidswiss Kidswiss changed the title Consistently use labels.standard stable/redis-ha Consistently use labels.standard Aug 12, 2026
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.

1 participant