-
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
Move cilium/hubble code to cilium/cilium repo #31893
Conversation
a131712
to
6eed257
Compare
/test |
69f511d
to
7676ddb
Compare
/test |
821bead
to
b91e823
Compare
/test |
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 the PR @michi-covalent 🙏
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.
Thank you! I have some concerns around the CI workflow, but not necessarily blocking
b91e823
to
3a46baa
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.
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.
🚢
so long years and years of git blame 😢 |
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 @michi-covalent, maybe we should consider adding a /hubble/README.md
file to clarify that the directory contains the Hubble CLI code (i.e. not the Hubble server part) to make it easier to discover for new contributors.
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.
:rubber-stamp:
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.
Let's merge it ✔️
- Copy hubble CLI code at cilium/hubble@10258183. - Include locally-built Hubble CLI in cilium-agent container image. - Add a GitHub workflow to build Hubble CLI release binaries and run integration tests. See CFP-033 [^1] for details. [^1]: cilium/design-cfps#33 Signed-off-by: Michi Mutsuzaki <michi@isovalent.com>
81b0cfd
to
d849e1d
Compare
/test |
Hubble CLI code recently got moved into cilium/cilium repo [^1]. Import github.com/cilium/cilium/hubble instead of github.com/cilium/hubble to remove cilium/hubble dependency. [^1]: cilium/cilium#31893 Signed-off-by: Michi Mutsuzaki <michi@isovalent.com>
Hubble CLI code recently got moved into cilium/cilium repo [^1]. Import github.com/cilium/cilium/hubble instead of github.com/cilium/hubble to remove cilium/hubble dependency. [^1]: cilium/cilium#31893 Signed-off-by: Michi Mutsuzaki <michi@isovalent.com>
integration tests.
See CFP-033 1 for details.
For reviewers:
Please review the following files carefully:
.github/workflows/hubble-cli.yaml
CODEOWNERS
go.mod
hubble/Makefile
images/cilium/Dockerfile
The following files don't have any logic changes except for fixing import paths and lint errors:
hubble/cmd/*
hubble/main.go
hubble/pkg/*
Footnotes
https://github.com/cilium/design-cfps/pull/33 ↩