Skip to content
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

Merged
merged 7 commits into from
Nov 15, 2023

Conversation

tristanburgess
Copy link
Contributor

@tristanburgess tristanburgess commented Nov 14, 2023

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

  • CHANGELOG.md updated

@tpaschalis tpaschalis self-assigned this Nov 14, 2023
@@ -2,7 +2,7 @@ apiVersion: v2
name: grafana-agent
description: 'Grafana Agent'
type: application
version: 0.27.2
Copy link
Member

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.

Copy link
Member

@tpaschalis tpaschalis left a 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.

@tpaschalis tpaschalis enabled auto-merge (squash) November 15, 2023 14:58
@tpaschalis tpaschalis merged commit 0b03f0b into main Nov 15, 2023
10 checks passed
@tpaschalis tpaschalis deleted the tristanburgess/helm-chart-k8s-version-label branch November 15, 2023 15:02
@github-actions github-actions bot added the frozen-due-to-age Locked due to a period of inactivity. Please open new issues or PRs if more discussion is needed. label Feb 21, 2024
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Feb 21, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
frozen-due-to-age Locked due to a period of inactivity. Please open new issues or PRs if more discussion is needed.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants