-
Notifications
You must be signed in to change notification settings - Fork 463
Move some deduped templates back to where they belong #2209
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
Move some deduped templates back to where they belong #2209
Conversation
|
/test e2e-ovirt |
An on-prem template wasn't moved correctly, and an OpenStack template was moved when it shouldn't have been.
cd7dd4f to
d407e16
Compare
|
/test e2e-ovirt |
|
/test e2e-vsphere-upi |
|
/test e2e-agnostic-upgrade |
|
/test e2e-vsphere |
|
/retest |
1 similar comment
|
/retest |
|
/test e2e-ovirt |
1 similar comment
|
/test e2e-ovirt |
|
/test e2e-openstack |
|
/test e2e-metal-ipi-ovn-dualstack |
1 similar comment
|
/test e2e-metal-ipi-ovn-dualstack |
|
/retest |
|
/lgtm |
|
/test e2e-openstack |
|
/retest |
|
This is kni/openstack, I'd like to see owners from both lgtm before approving /hold |
|
/lgtm |
|
/lgtm So that's why afterburn started showing up in my failed units lately. :-) |
|
/retest Please review the full test history for this PR and help us cut down flakes. |
|
@yuqi-zhang On this one, it is safe to skip e2e-metal-ipi-ovn-dualstack. It was added as a test for an unrelated issue. vsphere-upi should pass once 2207 is merged. |
|
/retest Please review the full test history for this PR and help us cut down flakes. |
3 similar comments
|
/retest Please review the full test history for this PR and help us cut down flakes. |
|
/retest Please review the full test history for this PR and help us cut down flakes. |
|
/retest Please review the full test history for this PR and help us cut down flakes. |
|
Given the AWS still slowly recovering, and the fact that this has passed all the other tests in the past + it should not affect cloud platforms, I will override e2e-gcp-op on this. /override ci/prow/e2e-gcp-op |
|
@yuqi-zhang: Overrode contexts on behalf of yuqi-zhang: ci/prow/e2e-gcp-op DetailsIn 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. |
|
/retest Please review the full test history for this PR and help us cut down flakes. |
14 similar comments
|
/retest Please review the full test history for this PR and help us cut down flakes. |
|
/retest Please review the full test history for this PR and help us cut down flakes. |
|
/retest Please review the full test history for this PR and help us cut down flakes. |
|
/retest Please review the full test history for this PR and help us cut down flakes. |
|
/retest Please review the full test history for this PR and help us cut down flakes. |
|
/retest Please review the full test history for this PR and help us cut down flakes. |
|
/retest Please review the full test history for this PR and help us cut down flakes. |
|
/retest Please review the full test history for this PR and help us cut down flakes. |
|
/retest Please review the full test history for this PR and help us cut down flakes. |
|
/retest Please review the full test history for this PR and help us cut down flakes. |
|
/retest Please review the full test history for this PR and help us cut down flakes. |
|
/retest Please review the full test history for this PR and help us cut down flakes. |
|
/retest Please review the full test history for this PR and help us cut down flakes. |
|
/retest Please review the full test history for this PR and help us cut down flakes. |
|
follow up from #2209 (comment) /override ci/prow/e2e-gcp-op |
|
@runcom: Overrode contexts on behalf of runcom: ci/prow/e2e-gcp-op, ci/prow/e2e-metal-ipi-ovn-dualstack DetailsIn 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. |
|
/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. |
|
@bcrochet: The following test failed, say
Full PR test history. Your PR dashboard. DetailsInstructions 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. |
An on-prem template wasn't moved correctly, and an OpenStack template was moved when it shouldn't have been.
- What I did
- How to verify it
- Description for the changelog