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
doc: update Hubble/Hubble Relay guides for recent CLI changes #15981
Conversation
To ensure a more logical flow, move the section that explains how to ensure that Hubble is running before the section that explains how to observe flows with Hubble. Signed-off-by: Robin Hahling <robin.hahling@gw-computing.net>
The format of the `hubble status` and `hubble observe` subcommands vary slightly with the new Hubble CLI. Make sure the examples are consistent with the new version. While there, the new CLI also defaults to `localhost:4245` as the server address. Hence, remove notes that hinted to use `--server localhost:4245` when running Hubble CLI commands. Signed-off-by: Robin Hahling <robin.hahling@gw-computing.net>
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 @rolinh!
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.
Looks good overall, one note below.
The format of the `hubble status` and `hubble observe` subcommands vary slightly with the new Hubble CLI. Make sure the examples are consistent with the new version. While there, the new CLI also defaults to `localhost:4245` as the server address. Hence, remove notes that hinted to use `--server localhost:4245` when running Hubble CLI commands. Signed-off-by: Robin Hahling <robin.hahling@gw-computing.net>
There is already a section that explains how to enable Hubble Relay and install the Hubble CLI so link to it instead. While there, hint about using `hubble list nodes`. Signed-off-by: Robin Hahling <robin.hahling@gw-computing.net>
d4f16cd
to
9a560e6
Compare
The CI failures are flakes/unrelated (this PR only touches documentation pages). Got required approval, hence marking ready to merge. |
This PR updates the troubleshooting with Hubble/Hubble Relay sections as well as the getting started with Hubble guide to recent Hubble CLI changes. Namely, the output of
hubble status
andhubble observe
is now slightly different and the CLI defaults tolocalhost:4245
a the server address, which allows removing instructions about setting the server address when targeting Hubble Relay.This PR also includes other minor improvements, see commits for details.