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

fix(cilium-cli/status): add helmReleaseName parameter #34748

Merged
merged 1 commit into from
Sep 17, 2024

Conversation

pgils
Copy link
Contributor

@pgils pgils commented Sep 7, 2024

The status command did not use the helmReleaseName set by the cli, causing it to do a lookup of a helm release with an empty string as helmReleaseName. This results in error and no version reported by the status command.

This commit adds the helmReleaseName parameter so the global value is inherited by the status command.

Fixes: #34747

Fix missing Helm chart version for status command

@pgils pgils requested a review from a team as a code owner September 7, 2024 10:53
@pgils pgils requested a review from nathanjsweet September 7, 2024 10:53
@maintainer-s-little-helper maintainer-s-little-helper bot added the dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. label Sep 7, 2024
@github-actions github-actions bot added cilium-cli This PR contains changes related with cilium-cli kind/community-contribution This was a contribution made by a community member. labels Sep 7, 2024
The status command did not use the helmReleaseName set by the cli,
causing it to do a lookup of a helm release with an empty string as
helmReleaseName. This results in error and no version reported by the
status command.

This commit adds the helmReleaseName parameter so the global value is
inherited by the status command.

Fixes: cilium#34747

Signed-off-by: Pelle van Gils <[email protected]>
@pgils pgils force-pushed the fix/status-helm-version branch from cea2dcd to 4e74c15 Compare September 7, 2024 11:05
@dylandreimerink
Copy link
Member

/test

@dylandreimerink dylandreimerink added the release-note/bug This PR fixes an issue in a previous release of Cilium. label Sep 10, 2024
@maintainer-s-little-helper maintainer-s-little-helper bot removed the dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. label Sep 10, 2024
@maintainer-s-little-helper maintainer-s-little-helper bot added the ready-to-merge This PR has passed all tests and received consensus from code owners to merge. label Sep 17, 2024
@ldelossa ldelossa added this pull request to the merge queue Sep 17, 2024
Merged via the queue into cilium:main with commit 2291c5f Sep 17, 2024
63 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cilium-cli This PR contains changes related with cilium-cli kind/community-contribution This was a contribution made by a community member. ready-to-merge This PR has passed all tests and received consensus from code owners to merge. release-note/bug This PR fixes an issue in a previous release of Cilium.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

CLI: status command does not show helm chart version
4 participants