-
Notifications
You must be signed in to change notification settings - Fork 2.8k
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
clarify some docs around the kubeProxyReplacement=partial mode #19831
clarify some docs around the kubeProxyReplacement=partial mode #19831
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.
Looks good to me, thanks! Just one minor comment below.
Can you please squash the two commits? (And drop the |
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!
97301f7
to
f1edcdf
Compare
Done (with apologies for the long delay as I've just returned from a long vacation) |
This pull request has been automatically marked as stale because it |
@aecay it looks there are conflicting files. Could you rebase your PR against the latest changes on master branch? Thank you |
- change how the `enableHealthCheckNodeport` option is spelled - itʼs actually `nodePort.enableHealthCheck` [ref](https://github.com/cilium/cilium/blob/dfa6b157e8e9484c65fd938b2b45a4f5f50c61f9/install/kubernetes/cilium/templates/cilium-configmap.yaml#L591-L593) - change the scope of the recommendation to enable this option - AFAICS itʼs only relevant when `nodePort.enabled` is true - a double-check of this would be welcome (but weʼve been running with `kubeProxyReplacement=partial nodePort.enableHealthCheck=true` with no apparent ill effect) - include a brief explanation of why this setting is required Signed-off-by: Aaron Ecay <aaron.ecay@sky.uk>
f1edcdf
to
a23a634
Compare
@aanm - thanks. Rebased 👍 |
enableHealthCheckNodeport
option is spellednodePort.enableHealthCheck
refnodePort.enabled
is truekubeProxyReplacement=partial nodePort.enabled=false nodePort.enableHealthCheck=true
with no apparent ill effect...)