-
Notifications
You must be signed in to change notification settings - Fork 4.8k
SPLAT-2337: CCM-AWS OTE binary and skips #30747
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
SPLAT-2337: CCM-AWS OTE binary and skips #30747
Conversation
|
Pipeline controller notification For optional jobs, comment This repository is configured in: automatic mode |
|
Skipping CI for Draft Pull Request. |
|
/payload-job periodic-ci-openshift-release-master-ci-4.22-e2e-aws-upgrade-ovn-single-node |
|
@mtulio: trigger 1 job(s) for the /payload-(with-prs|job|aggregate|job-with-prs|aggregate-with-prs) command
See details on https://pr-payload-tests.ci.openshift.org/runs/ci/c526d0e0-0038-11f1-8671-8cc451fbd60c-0 |
|
Previous job failed, skip statement didn't work as skip label was expecting to match /payload-job periodic-ci-openshift-release-master-ci-4.22-e2e-aws-upgrade-ovn-single-node |
|
@mtulio: trigger 1 job(s) for the /payload-(with-prs|job|aggregate|job-with-prs|aggregate-with-prs) command
See details on https://pr-payload-tests.ci.openshift.org/runs/ci/bbf5f710-0062-11f1-82e4-770c08b5ef46-0 |
|
Added a new statement to prevent /payload-job periodic-ci-openshift-release-master-ci-4.22-e2e-aws-upgrade-ovn-single-node |
|
@mtulio: trigger 1 job(s) for the /payload-(with-prs|job|aggregate|job-with-prs|aggregate-with-prs) command
See details on https://pr-payload-tests.ci.openshift.org/runs/ci/1087c530-006b-11f1-8db1-33dbe3ad21f2-0 |
|
/payload-job periodic-ci-openshift-release-master-ci-4.22-e2e-aws-ovn-upgrade-out-of-change |
|
@mtulio: trigger 1 job(s) for the /payload-(with-prs|job|aggregate|job-with-prs|aggregate-with-prs) command
See details on https://pr-payload-tests.ci.openshift.org/runs/ci/4e53c710-006b-11f1-83b3-8a3c785cffe9-0 |
|
Reviewing the last change, which propose to fix the topology env setup in SNO, Although, the skips for CCM-AWS' OTE e2e wasn't working in SNO job. I will investigate the next steps. [1] cc @neisw |
|
I just found one interesting behavior on run-upgrade: it isn't setting up the cluster-config. My last commit(e71e227) is proposing to set it and passing it through the life cycle. /payload-job periodic-ci-openshift-release-master-ci-4.22-e2e-aws-upgrade-ovn-single-node |
|
@mtulio: trigger 1 job(s) for the /payload-(with-prs|job|aggregate|job-with-prs|aggregate-with-prs) command
See details on https://pr-payload-tests.ci.openshift.org/runs/ci/30d4c1a0-0096-11f1-9682-765365ce9bf7-0 |
|
/test ? |
|
/payload-job ? |
|
/testwith openshift/origin/main/e2e-aws-ovn-single-node-upgrade openshift/cloud-provider-aws#125 |
timeout on build, re-trying: |
|
/testwith openshift/origin/main/e2e-aws-ovn-single-node-upgrade openshift/cloud-provider-aws#125 |
|
/payload-job-with-prs periodic-ci-openshift-release-master-ci-4.22-e2e-aws-upgrade-ovn-single-node openshift/cloud-provider-aws#125 |
|
/payload-job-with-prs periodic-ci-openshift-release-master-ci-4.22-e2e-aws-upgrade-ovn-single-node openshift/cloud-provider-aws#125 |
|
@mtulio: trigger 1 job(s) for the /payload-(with-prs|job|aggregate|job-with-prs|aggregate-with-prs) command
See details on https://pr-payload-tests.ci.openshift.org/runs/ci/1ffc4210-0116-11f1-9819-cd56774c6083-0 |
|
Checking if failures are related to cross build or specifically with missing rhcos image in the pipeline affecting any build method: /payload-job periodic-ci-openshift-release-master-ci-4.22-e2e-aws-upgrade-ovn-single-node |
|
@mtulio: trigger 1 job(s) for the /payload-(with-prs|job|aggregate|job-with-prs|aggregate-with-prs) command
See details on https://pr-payload-tests.ci.openshift.org/runs/ci/72340150-0123-11f1-8a1f-c489fed6372e-0 |
/test e2e-aws-ovn-fips |
|
/test images |
|
/payload-job-with-prs periodic-ci-openshift-release-master-ci-4.22-e2e-aws-upgrade-ovn-single-node openshift/cloud-provider-aws#125 |
|
/lgtm |
✍🏽 /verified by @mtulio SNO jobs are now passing with skipped tests: https://prow.ci.openshift.org/view/gs/test-platform-results/logs/openshift-cloud-provider-aws-125-openshift-origin-30747-ci-4.22-e2e-aws-upgrade-ovn-single-node/2019041978443894784 |
|
@mtulio: This PR has been marked as verified by 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 openshift-eng/jira-lifecycle-plugin repository. |
Re-add OTE binary extraction for CCM-AWS after skips added to the binary[1], and successful jobs taken to SNO[2]. [1] openshift/cloud-provider-aws#125 [2] https://prow.ci.openshift.org/view/gs/test-platform-results/logs/openshift-cloud-provider-aws-125-openshift-origin-30747-ci-4.22-e2e-aws-upgrade-ovn-single-node/2019041978443894784
9129b35 to
4f6dfae
Compare
|
PR rebased, re-stamping same verify label: /verified by @mtulio SNO jobs are now passing with skipped tests: https://prow.ci.openshift.org/view/gs/test-platform-results/logs/openshift-cloud-provider-aws-125-openshift-origin-30747-ci-4.22-e2e-aws-upgrade-ovn-single-node/2019041978443894784 |
|
@mtulio: This PR has been marked as verified by 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 openshift-eng/jira-lifecycle-plugin repository. |
|
/lgtm |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: jcpowermac, mtulio, neisw 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 |
|
Scheduling required tests: |
|
/test e2e-aws-ovn-fips |
1 similar comment
|
/test e2e-aws-ovn-fips |
|
Job e2e-aws-ovn-fips are failing caused by CI infrastructure And job e2e-aws-ovn-microshift is caused by |
|
/override ci/prow/e2e-aws-ovn-microshift unrelated flakes and failures |
|
@neisw: Overrode contexts on behalf of neisw: ci/prow/e2e-aws-ovn-fips, ci/prow/e2e-aws-ovn-microshift 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-sigs/prow repository. |
|
@mtulio: all tests passed! 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-sigs/prow repository. I understand the commands that are listed here. |
Re-adding the OTE binary extraction for CCM AWS previously added in #30525 and reverted #30744 after SNO job failing with tests exposed by it:
job aggregated-aws-ovn-single-node-upgrade-4.22-micro failed blocking job for 4.22.0-0.nightly-2026-02-01-095950.
Skips were added to the OTE implementation by openshift/cloud-provider-aws#125
SNO jobs are now passing with skipped tests: https://prow.ci.openshift.org/view/gs/test-platform-results/logs/openshift-cloud-provider-aws-125-openshift-origin-30747-ci-4.22-e2e-aws-upgrade-ovn-single-node/2019041978443894784