-
Notifications
You must be signed in to change notification settings - Fork 38.7k
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
Add fall-back to get node IP on host_path tests #68915
Conversation
/test pull-kubernetes-e2e-gce-device-plugin-gpu |
After applying this PR, the test can pass on my local environment which doesn't support external IPs. |
As the same as the commit[1], this adds fall-back way to get a node IP address on host_path e2e tests for environments which don't support external IPs. [1]: kubernetes@4e7c2f6#diff-5ee86aefbb33223865bc542107ea8560L81
e10adb2
to
dd9cffb
Compare
/kind bug |
/assign @MrHohn |
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
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: MrHohn, oomichi 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 |
@oomichi can you remove the dummy release note? Thanks. |
@MrHohn Oh, I missed that and removed. Thanks for your approval anyways :-) |
What this PR does / why we need it:
As the same as the commit1, this adds fall-back way to get a
node IP address on host_path e2e tests for environments which
don't support external IPs.
Which issue(s) this PR fixes (optional, in
fixes #<issue number>(, fixes #<issue_number>, ...)
format, will close the issue(s) when PR gets merged):Fixes #68914
Release note:
None