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
Only set ipv4/ipv6 routes when there is corresponding CIDR. #82800
Conversation
Signed-off-by: Lantao Liu <lantaol@google.com>
I'm not sure whether we need this in 1.16.0, but we should at least cherry-pick this into patch releases. |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: Random-Liu The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
routes []string | ||
hasV4, hasV6 bool | ||
) | ||
for _, thisCIDR := range plugin.podCIDRs { |
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.
is plugin.podCIDRs available if dual-stack is disabled? should we check if it exists and fall back to podCIDR if it doesn't ?
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.
it is available either ways..
/lgtm kubenet will run fine even if ipv6 is not enabled on old settings, we have green signal across the board for that. However new settings are better. |
/test pull-kubernetes-e2e-gce-100-performance |
@khenidak I tried to disable the ipv6 sysctl myself, and the ipv6 route worked after that. However, based on kubernetes-sigs/kind#626, it seems that it won't work if ipv6 is disabled at boot time. |
…2800-upstream-release-1.16 Automated cherry pick of #82800: Only set ipv4/ipv6 routes when there is corresponding CIDR.
We should only set ipv6 and ipv4 routes when there are corresponding CIDRs.
On system which doesn't enable IPv6 at boot time, Kubernetes 1.16 with kubenet will not be able to run even if users don't actually use IPv6. (See kubernetes-sigs/kind#626)
/cc @kubernetes/sig-network-bugs @khenidak @aojea
Signed-off-by: Lantao Liu lantaol@google.com