-
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
Use actual hostname when creating network e2e test pod #47143
Use actual hostname when creating network e2e test pod #47143
Conversation
e508be0
to
fa67fde
Compare
Updated to be in line with the changes in #35937 |
/lgtm /approve |
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: ethernetdan, spiffxp, zmerlynn Associated issue: 1018 The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these OWNERS Files:
You can indicate your approval by writing |
Automatic merge from submit-queue (batch tested with PRs 47065, 47157, 47143) |
What this PR does / why we need it:
This changes a e2e framework network test Pod use the actual hostname value to match the
kubernetes.io/hostname
label in it'sNodeSelector
. Currently it assumes the Node name will match that hostname label which is not true in all environments.Which issue this PR fixes (optional, in
fixes #<issue number>(, fixes #<issue_number>, ...)
format, will close that issue when PR gets merged):Fixes coreos/tectonic-installer#1018
Release note: