[release-4.8] Bug 1975938: Fix skipping of Multi-AZ Cluster Volumes test on GCP with k8s 1.21#26267
Conversation
|
@yselkowitz: This pull request references Bugzilla bug 1975938, which is invalid:
Comment 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. |
|
Skipping CI for Draft Pull Request. |
|
/test all |
|
/bugzilla refresh |
|
@yselkowitz: This pull request references Bugzilla bug 1975938, which is valid. The bug has been updated to refer to the pull request using the external bug tracker. 6 validation(s) were run on this bug
No GitHub users were found matching the public email listed for the QA contact in Bugzilla (wduan@redhat.com), skipping review request. 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. |
In order to update the Multi-AZ Cluster Volumes test which was renamed in k8s 1.21: openshift/kubernetes#829
In order to update the Multi-AZ Cluster Volumes test which was renamed in k8s 1.21: openshift/kubernetes#829
| k8s.io/sample-apiserver => github.com/openshift/kubernetes/staging/src/k8s.io/sample-apiserver v0.0.0-20210603185452-2dfc46b23003 | ||
| k8s.io/sample-cli-plugin => github.com/openshift/kubernetes/staging/src/k8s.io/sample-cli-plugin v0.0.0-20210603185452-2dfc46b23003 | ||
| k8s.io/sample-controller => github.com/openshift/kubernetes/staging/src/k8s.io/sample-controller v0.0.0-20210603185452-2dfc46b23003 | ||
| github.com/opencontainers/runc => github.com/openshift/opencontainers-runc v1.0.0-rc95.0.20210608002938-1f5126fe967e |
There was a problem hiding this comment.
Can you please add a comment like this? This helps for next person doing k8s bumps.
There was a problem hiding this comment.
Given this is a stable branch, do you anticipate openshift/kubernetes reverting to upstream runc for the duration of 4.8? Perhaps master, where this is much more likely, would be a better place for such a comment?
There was a problem hiding this comment.
We might do minor bumps for 4.8 as well, so better to add it for both master and 4.8 branches.
|
/retest |
|
Besides the builds/jenkins failures which are being worked on, is 4.8 CI just that bad right now or is there a real issue here? |
|
/retest The proxy seems to have unrelated issue. Can you fix #26267 (comment)? |
|
proxy and metal-ipi-ovn both seem to be generally broken, we can override those if/when you get green elsewhere and have lgtm/approval |
|
/retest |
1 similar comment
|
/retest |
|
/test e2e-gcp-builds |
|
All tests are green except for those known to be broken. |
|
/lgtm |
|
/retest |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: jsafrane, sttts, yselkowitz The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
|
We're ready for those overrides now. |
|
/retest Please review the full test history for this PR and help us cut down flakes. |
|
@bparees: Overrode contexts on behalf of bparees: ci/prow/e2e-aws-proxy, ci/prow/e2e-metal-ipi 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. |
|
/override ci/prow/e2e-aws-proxy |
|
@bparees: Overrode contexts on behalf of bparees: ci/prow/e2e-aws-proxy, ci/prow/e2e-metal-ipi 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. |
|
/test e2e-aws |
|
/retest Please review the full test history for this PR and help us cut down flakes. |
2 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. |
|
@yselkowitz: 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. |
1 similar comment
|
/retest Please review the full test history for this PR and help us cut down flakes. |
|
@yselkowitz: All pull requests linked via external trackers have merged: Bugzilla bug 1975938 has been moved to the MODIFIED state. 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. |
Backport of #26264 for 4.8.