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
Remove hubble-cli sub-chart #11806
Remove hubble-cli sub-chart #11806
Conversation
this is blocked on #11784. once it's merged, test/k8sT/hubble.go needs to be updated a bit more. for example, https://github.com/cilium/cilium/blob/7a113cbb5d2fe446ccfae392ddc68d6247623465/test/k8sT/hubble.go#L179 needs to exec into the cilium pod to run the hubble command. |
… the cilium-agent container Fixes: #11805 Signed-off-by: Sean Winn <sean@isovalent.com>
The Hubble CLI is now part of the Cilium container image. Therefore we can invoke all Hubble commands from the Cilium container and remove the hubble-cli helpers. Co-authored-by: Sean Winn <sean@isovalent.com> Signed-off-by: Sean Winn <sean@isovalent.com> Signed-off-by: Sebastian Wicki <sebastian@isovalent.com>
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.
love it
test-me-please |
retest-runtime (hit |
Remove the hubble-cli sub-chart since the binary was packaged with the cilium-agent container and will not be released
Fixes: #11805
Signed-off-by: Sean Winn sean@isovalent.com