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
envoy: Add support for exposing Envoy Admin API #30655
Conversation
f23e869
to
d05be50
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.
Looks good!
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 for docs!
I'd really like the flag to say debug somewhere in there. Maybe |
8cbf205
to
d63b350
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 Tam. Some questions and suggestions inline.
install/kubernetes/cilium/files/cilium-envoy/configmap/bootstrap-config.json
Outdated
Show resolved
Hide resolved
install/kubernetes/cilium/files/cilium-envoy/configmap/bootstrap-config.json
Outdated
Show resolved
Hide resolved
d63b350
to
0db09fe
Compare
/test |
0db09fe
to
1573ed8
Compare
/test |
1573ed8
to
c6a5bd1
Compare
/test |
c6a5bd1
to
91ae828
Compare
/test |
This is to support the capability to expose Envoy Admin API via new helm flags envoy.debug.admin.{enabled,port} for both embedded and daemonset modes. Fixes: #30647 Signed-off-by: Tam Mach <tam.mach@cilium.io>
91ae828
to
44ef5b1
Compare
/test |
This is to support the capability to expose Envoy Admin API via new helm flags envoy.debug.admin.{enabled,port} for both embeded and daemonset modes.
Fixes: #30647