-
Notifications
You must be signed in to change notification settings - Fork 488
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
helm chart: align k8s version label with Values #5773
Conversation
@@ -2,7 +2,7 @@ apiVersion: v2 | |||
name: grafana-agent | |||
description: 'Grafana Agent' | |||
type: application | |||
version: 0.27.2 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Bumping this will automatically publish a new chart version. IMHO it's not required straight away, let's leave it as-is.
Otherwise, we'll have to update the CHANGELOG headers to mirror that.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. Let's re-run make generate-helm-docs
to fix the README and we're good to go.
PR Description
Use helm chart template functions to supply
app.kubernetes.io/version
with value as configured by any possible override for the Agent image tag version to the Helm chart Values.PR Checklist