-
Notifications
You must be signed in to change notification settings - Fork 2.7k
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
feat(hubble-ui): Add probes for Hubble-ui #27028
feat(hubble-ui): Add probes for Hubble-ui #27028
Conversation
9e27872
to
6088472
Compare
6088472
to
f760510
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.
Looks good to me.
Please note that you will have to update the values.yaml.tmpl
file instead of the values.yaml
file and run make -C install/kubernetes
and make -C Documentation
to update all generated files relating to Helm.
f760510
to
6717de5
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.
Helm-wise looks good to me! Thanks! Let's also get a review from a Hubble UI code owner
/test |
LGTM! |
Hi @mkilchhofer, sorry, but with #27002 merging, there's been a change to the required jobs to pass CI, so I recommend rebasing and then one of us can re-trigger the testing. |
Signed-off-by: Marco Kilchhofer <mkilchhofer@users.noreply.github.com>
6717de5
to
ed6ef52
Compare
@youngnick @gandro rebased as requested :-) |
/test |
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.
As this is a common pattern for all Kubernetes pods, I think it doesn't require additional use cases.
But: I contributed the
/healthz
endpoint only a few days ago. The feature for the backend requires hubble-ui >=0.12:Resolves: cilium/hubble-ui#586
/cc: @geakstr