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.

Sign up for GitHub

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

Jump to bottom

Improve error messages for "release not found" #1907

Open
asauber opened this issue Aug 14, 2023 · 3 comments · May be fixed by #2597
Open

Improve error messages for "release not found" #1907

asauber opened this issue Aug 14, 2023 · 3 comments · May be fixed by #2597
Labels
area/helm-mode Relates to Helm-based implementation good first issue Good for newcomers kind/bug Something isn't working

Comments

@asauber
Copy link
Member

asauber commented Aug 14, 2023

In codepaths where the CLI might fail with release not found, such as

Error: Unable to enable ClusterMesh: release: not found

we should include more information in this message to allow for a resolution.

In most cases, the user is working with a cluster without a Helm-based Cilium install, or they haven't specified the --namespace.

Instructing a retry with CILIUM_CLI_MODE=classic can also be included until full removal of classic mode.
@asauber asauber added kind/bug Something isn't working good first issue Good for newcomers area/helm-mode Relates to Helm-based implementation labels Aug 14, 2023
@dhanusaputra
Copy link

hi, can I help with this please?

@network-charles
Copy link

I’m currently having this issue while attempting to enable clustermesh. What namespace are you referring to, “kube-system”?

@cnvergence
Copy link

I would like to help with that, as I encountered the issue already fixed in #2274
I will work on this PR: #2597

@cnvergence cnvergence linked a pull request Jun 13, 2024 that will close this issue
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
area/helm-mode Relates to Helm-based implementation good first issue Good for newcomers kind/bug Something isn't working
Projects
Good First Issues
Status: No status
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

chore: improve error messages for Helm commands cnvergence/cilium-cli
4 participants
@asauber @cnvergence @network-charles @dhanusaputra