-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
Bug 1975527: upi/vsphere: Use Afterburn guestinfo for static IP and hostname config #3533
Bug 1975527: upi/vsphere: Use Afterburn guestinfo for static IP and hostname config #3533
Conversation
@LorbusChris: The specified target(s) for
Use In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
/test e2e-vsphere-upi |
It seems the interface isn't coming up. I suspect NM's keyfile plugin is not enabled on RHCOS. Investigating. |
Try naming the file |
/test e2e-vsphere-upi |
2 similar comments
/test e2e-vsphere-upi |
/test e2e-vsphere-upi |
@dustymabe @jcpowermac any ideas why this isn't working? Are the boot images maybe not supporting keyfiles, yet? |
I've rolled back the wip commits. Name is |
/test e2e-vsphere-upi |
/retest |
/test e2e-vsphere-upi |
1 similar comment
/test e2e-vsphere-upi |
Keep in mind this is only going to affect networking in the real root, not at Ignition time. For that we need the installer to make use of openshift/enhancements#291 specifically coreos/afterburn#404 |
/test e2e-vsphere-upi |
/test e2e-vsphere-upi |
/test e2e-vsphere-upi |
/test e2e-vsphere-upi |
4 similar comments
/test e2e-vsphere-upi |
/test e2e-vsphere-upi |
/test e2e-vsphere-upi |
/test e2e-vsphere-upi |
vsphere UPI failures look like flakes to me /test e2e-vsphere-upi |
/test e2e-aws-fips |
@jcpowermac @staebler any chance you can take a look at this again soon? |
@LorbusChris upi passed |
My naive reading of this comment from @cgwalters is that the network settings will not be available at ignition time, which seems to be the same as what we had before with configuring the settings via ignition configs. /lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: staebler 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 |
/retest Please review the full test history for this PR and help us cut down flakes. |
@LorbusChris: The following test failed, say
Full PR test history. Your PR dashboard. Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here. |
/retest Please review the full test history for this PR and help us cut down flakes. |
1 similar comment
/retest Please review the full test history for this PR and help us cut down flakes. |
/cherry-pick fcos |
@LorbusChris: new pull request created: #4673 In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
Cherry-picking this to 4.7 due to issues found in vSphere UPI CI jobs. /cherry-pick release-4.7 |
@rvanderp3: new pull request created: #5025 In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
/retitle Bug 1975527: upi/vsphere: Use Afterburn guestinfo for static IP and hostname config |
@LorbusChris: Some pull requests linked via external trackers have merged: The following pull requests linked via external trackers have not merged:
These pull request must merge or be unlinked from the Bugzilla bug in order for it to move to the next state. Once unlinked, request a bug refresh with Bugzilla bug 1975527 has not been moved to the MODIFIED state. In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
/bugzilla refresh |
@sdodson: All pull requests linked via external trackers have merged: Bugzilla bug 1975527 has been moved to the MODIFIED state. In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
/test vsphere-upi