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
Embed Hubble #10238
Embed Hubble #10238
Conversation
Release note label not set, please set the appropriate release note. |
70ffdfc
to
de770f7
Compare
waiting on #9924 to be merged |
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.
This is amazing. Most comments are minor nits that can be sorted out pretty quickly. The main discussion point is likely how we serve the API without introducing a security concern.
thanks for your feedback @tgraf , i'll rebase and then address your comments. |
2f0b4a6
to
d6e18c8
Compare
bc4ec4b
to
7edb371
Compare
test-me-please |
7edb371
to
12ea5c4
Compare
test-me-please |
12ea5c4
to
9e4f0c9
Compare
test-me-please |
9e4f0c9
to
be24fd4
Compare
test-me-please |
be24fd4
to
6542c71
Compare
test-me-please |
test-me-please |
1 similar comment
test-me-please |
Ref #9925 Signed-off-by: Michi Mutsuzaki <michi@isovalent.com>
Hubble needs to access this function to implement the IPGetter interface: https://github.com/cilium/hubble/blob/04ab72591faca62a305ce0715108876167182e04/pkg/parser/getters/getters.go#L47 Ref #9925 Signed-off-by: Michi Mutsuzaki <michi@isovalent.com>
Hubble needs to be able to look up service namespace/name by address to implement the ServiceGetter interface: https://github.com/cilium/hubble/blob/04ab72591faca62a305ce0715108876167182e04/pkg/parser/getters/getters.go#L52 Ref #9925 Signed-off-by: Michi Mutsuzaki <michi@isovalent.com>
- Implement a monitor listener that sends monitor events to Hubble. - Export RegisterNewListener and Monitor so that Daemon can register Hubble as a monitor listener. Ref #9925 Signed-off-by: Michi Mutsuzaki <michi@isovalent.com>
Implement all the getter interfaces that Hubble uses to access Cilium state: https://github.com/cilium/hubble/blob/04ab72591faca62a305ce0715108876167182e04/pkg/parser/getters/getters.go Ref #9925 Signed-off-by: Michi Mutsuzaki <michi@isovalent.com>
Add options to launch Hubble with the following configuration parameters: - hubble-listen-addresses: List of IP addresses for Hubble server to listen to. Hubble is disabled if this list is empty. It also limits the addresses to be Unix domain socket for now. - hubble-flow-buffer-size: Maximum number of flows in Hubble's buffer. - hubble-event-queue-size: Buffer size of the channel to receive monitor events. - hubble-metrics-server: Address to serve Hubble metrics on. The metrics server is disabled if this string is empty. - hubble-metrics: List of Hubble metrics to enable. Ref #9925 Signed-off-by: Michi Mutsuzaki <michi@isovalent.com>
Ref #9925 Signed-off-by: Michi Mutsuzaki <michi@isovalent.com>
6542c71
to
8dffb1f
Compare
test-me-please |
fixed the etcd version in go.mod |
7 commits:
listener refactoring, there is a separate PR here.RebasedThis change is