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
docs: Fix Helm instructions for BGP #16263
Conversation
Commit 0c34921dacd0a2600080c07a8298f180f7ae75f7 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 |
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!
(Please make sure to sign-off your commit as explained at the link above.)
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.
Please state what the commit is changing. So a commit title such as "docs: Fix Helm instructions for BGP" can work and please make sure the commit is signed off as Quentin mentioned
Commits 0c34921dacd0a2600080c07a8298f180f7ae75f7, d3b8f1bd992af5011574c12b81a377eb3cfb987a do 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 |
The sign off I think needs to have the email within angle brackets, |
Signed-off-by: Tobias Mose <tobias.mose@xentom.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.
Thanks for the PR!
Helm value "bgp.announce.lbIP" is invalid, it should be "bgp.announce.loadbalancerIP".
(Source: https://github.com/cilium/cilium/tree/v1.10.0/install/kubernetes/cilium#values)
Signed-off-by: Tobias Mose tobias.mose@xentom.com