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
Documentation: Fix Envoy LB docs incorrect supported annotation values #26867
Conversation
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.
LGTM for docs
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.
Good catch! LGTM this is indeed the correct value
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.
The mistake probably comes from the fact that the Helm value loadBalancer.l7.backend
currently accepts either envoy
or disabled
. But the correct value for the lb-l7
annotation is indeed enabled
, as can be seen here.
Thanks!
Could you please rebase on top of the current |
Commit d1d7abc3fde7ec625eff90fe90733fd4a4d0637e does not contain "Signed-off-by". Please follow instructions provided in https://docs.cilium.io/en/stable/contributing/development/contributing_guide/#developer-s-certificate-of-origin |
1 similar comment
Commit d1d7abc3fde7ec625eff90fe90733fd4a4d0637e does not contain "Signed-off-by". Please follow instructions provided in https://docs.cilium.io/en/stable/contributing/development/contributing_guide/#developer-s-certificate-of-origin |
I messed it up with the merge commit, that's what you get for using github UI. Will try to fix the tree. |
bbce54b
to
fb90c4f
Compare
Signed-off-by: Rauan Mayemir <rauan@mayemir.io>
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.
All good, thank you!
There is a one-liner example that advices adding correct
service.cilium.io/lb-l7=enabled
annotation, but the table with supported annotations has a typo with incorrectenvoy
option instead ofenabled
.