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
Disable the Hubble before starting with UI via cilium-cli. #25538
Disable the Hubble before starting with UI via cilium-cli. #25538
Conversation
address doc section related to #24587, which might lead the user to meet the unexpected failure to start the Hubble UI. |
a44486a
to
e39143e
Compare
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.
Thanks for looping me in! I've suggested a minor change. Please let me know if you object.
Thank you. 😎✌️
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.
Thanks for this! Minor grammar nits below.
Can you please also drop the first, unrelated commit that sneaked in into your PR? I think this is because you initially worked on the |
…cli. Fixes: cilium#24587 Signed-off-by: Tomoya Fujita <Tomoya.Fujita@sony.com>
e39143e
to
de01984
Compare
done in de01984 |
Just FYI, created follow-up feature request to |
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.
Thanks a lot!
Doc change only, no need for the full CI suite. |
thanks guys for taking the time to review. btw, do i need to rebase? technically i do not think that is required to do so for this PR, but i can see |
Please ensure your pull request adheres to the following guidelines:
description and a
Fixes: #XXX
line if the commit addresses a particularGitHub issue.
Fixes: <commit-id>
tag, thenplease add the commit author[s] as reviewer[s] to this issue.
Fixes: #24587