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
Fix testing if an interface is the loopback #74082
Conversation
We'll need to backport this to the same releases as #73721 |
/assign @frapposelli |
@frapposelli With the same loopback setup, before that PR, our linux nodes were working correctly. With the PR and without this fix, they fail to register correctly (they don't show an IP address). While the PR did not edit the loopback check on the IP, it did change the way the mac addresses are checked. Before, any non-matching mac address (including an empty one) was just ignored. Now, an empty mac address produces an error. |
@astrieanna thanks for the feedback, looks like the problem is caused by the ineffective check that this PR is fixing and by the forced return if MAC address is not long enough. |
/retest |
It's not guaranteed that the loopback interface only has the loopback IP, in our environments our loopback interface is also assigned a 169 address as well.
/assign @divyenpatel |
/approve |
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: benmoss, divyenpatel 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 |
/test pull-kubernetes-e2e-gce |
@andrewsykim can you backport this to 1.11/1.12/1.13 like the other MAC address patch? They go hand in hand |
…2-upstream-release-1.12 Automated cherry pick of #74082: Fix testing if an interface is the loopback
…2-upstream-release-1.11 Automated cherry pick of #74082: Fix testing if an interface is the loopback
…2-upstream-release-1.13 Automated cherry pick of #74082: Fix testing if an interface is the loopback
What type of PR is this?
/kind bug
What this PR does / why we need it:
#73721 introduced a change that causes an error to occur when a non-loopback interface is found that doesn't have a valid MAC address. The problem is that the method for determining whether the interface is loopback or not is flawed: interfaces can have multiple IP addresses, and in our case our loopback interface has a 169 address assigned to as well as the 127 one.
Which issue(s) this PR fixes:
Special notes for your reviewer:
Does this PR introduce a user-facing change?:
If we can make it into the same release as #73721 we shouldn't need to amend the release notes
/sig cloud-provider
/sig vmware
cc @astrieanna