OCPBUGS-13547: Promote Azure CCM from TPNU to default#307
Conversation
openshift-ci-robot
added
jira/valid-reference
|
@JoelSpeed: This pull request references Jira Issue OCPBUGS-13547, which is valid. 3 validation(s) were run on this bug
Requesting review from QA contact: The bug has been updated to refer to the pull request using the external bug tracker. 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. |
openshift-ci
Bot
added
the
do-not-merge/hold
openshift-ci
Bot
added
the
approved
|
/test ? |
|
@JoelSpeed: The following commands are available to trigger required jobs:
The following commands are available to trigger optional jobs:
Use
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-azure |
JoelSpeed
changed the title
openshift-ci
Bot
added
the
do-not-merge/work-in-progress
openshift-merge-robot
added
the
needs-rebase
JoelSpeed
force-pushed
the
promote-azure-ccm-feature-gate
branch
from
c7dbdea to
9f129da
Compare
openshift-merge-robot
removed
the
needs-rebase
|
/test e2e-azure |
3 similar comments
|
/test e2e-azure |
|
/test e2e-azure |
|
/test e2e-azure |
|
/test e2e-azure Seems there's some infrastructure related issue causing failures |
|
Network operator is going degraded because of some panic in the internals of the http server in the last run, doesn't look like that's desirable |
|
Azure E2E failed this time in cluster teardown, but the tests themselves passed, lets try some payload tests |
|
@JoelSpeed: trigger 4 job(s) for the /payload-(job|aggregate) command
See details on https://pr-payload-tests.ci.openshift.org/runs/ci/ed5779e0-0511-11ee-85e1-27a2f38f2b56-0 |
|
Having reviewed the previous payload jobs, looks like a couple of minor/unrelated flakes caused failures, lets run some payload blocking tests to see if they find any issues |
|
@JoelSpeed: trigger 7 job(s) of type blocking for the nightly release of OCP 4.14
See details on https://pr-payload-tests.ci.openshift.org/runs/ci/477145d0-0537-11ee-8da9-2869c796f213-0 trigger 4 job(s) of type blocking for the ci release of OCP 4.14
See details on https://pr-payload-tests.ci.openshift.org/runs/ci/477145d0-0537-11ee-8da9-2869c796f213-1 |
openshift-merge-robot
added
the
needs-rebase
JoelSpeed
force-pushed
the
promote-azure-ccm-feature-gate
branch
from
9f129da to
0e78099
Compare
|
The upgrade aggregation went green!
Going to try that again |
|
@JoelSpeed: trigger 1 job(s) for the /payload-(job|aggregate) command
See details on https://pr-payload-tests.ci.openshift.org/runs/ci/5659ebc0-165c-11ee-9fae-e683f19d8364-0 |
|
All went green this time, I think this is good to go |
|
I tested this pr and enable TPNU then ran e2e all passed. |
|
Spoke to TRT, they would like us to hold on this until the next EC build has been cut, will merge next week |
JoelSpeed
force-pushed
the
promote-azure-ccm-feature-gate
branch
from
c398578 to
c850a0d
Compare
|
/payload-job periodic-ci-openshift-release-master-ci-4.14-e2e-azure-ovn |
|
@JoelSpeed: trigger 1 job(s) for the /payload-(job|aggregate) command
See details on https://pr-payload-tests.ci.openshift.org/runs/ci/8ac5b010-1c10-11ee-91ec-cac1a43eded5-0 |
|
@JoelSpeed: 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. |
|
/payload-job periodic-ci-openshift-release-master-ci-4.14-e2e-azure-ovn Test pod failed to schedule |
|
@JoelSpeed: trigger 1 job(s) for the /payload-(job|aggregate) command
See details on https://pr-payload-tests.ci.openshift.org/runs/ci/60578bc0-1c22-11ee-85af-1380f3933a85-0 |
|
/payload-aggregate periodic-ci-openshift-release-master-ci-4.14-e2e-azure-ovn 10 The canary I ran is using the latest CCM, so let's do this one more time to double check my latest fix hasn't broken anything |
|
@JoelSpeed: trigger 1 job(s) for the /payload-(job|aggregate) command
See details on https://pr-payload-tests.ci.openshift.org/runs/ci/49610930-1c33-11ee-80af-91c4c0cbd798-0 |
JoelSpeed
force-pushed
the
promote-azure-ccm-feature-gate
branch
from
c850a0d to
f93b1a7
Compare
JoelSpeed
changed the title
openshift-ci
Bot
removed
the
do-not-merge/work-in-progress
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: damdo, JoelSpeed 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 |
|
/hold cancel API PR has merged and test runs from previous job show that the CCM is working as expected and is not causing the disruption that previously caused it to be reverted |
openshift-ci
Bot
removed
the
do-not-merge/hold
|
/override ci/prow/e2e-aws-ovn-techpreview This PR does not affect AWS in any way, and especially not in techpreview. Failures were minor and related to OLM, so I'm confident not caused by this PR |
|
@JoelSpeed: Overrode contexts on behalf of JoelSpeed: ci/prow/e2e-aws-ovn-techpreview 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. |
|
@JoelSpeed: Jira Issue OCPBUGS-13547 is in an unrecognized state (ON_QA) and will not be 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. |
5 participants
This promotes the Azure CCM from TechPreviewNoUpgrade to default
/hold
We need to thoroughly test this and make sure all prior known fixes are merged before this can be merged