-
Notifications
You must be signed in to change notification settings - Fork 2.1k
ci-operator/config/openshift/cluster-version-operator: Fix B->A rollback config #31580
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
ci-operator/config/openshift/cluster-version-operator: Fix B->A rollback config #31580
Conversation
…ack config My initial approach from acd81b7 (ci-operator/config/openshift/cluster-version-operator: Separate A->B and B->A jobs (openshift#31518), 2022-08-22) doesn't seem to have worked. Follow up by pattern-matching from ebbee6d (Add AWS and Azure upgrade jobs and AWS selfupgrade for o/k, 2020-10-09, openshift#12618) from openshift/kubernetes, which was renamed to e2e-{platform}-downgrade in 6841f1c (All 4.7+ yearly should be given an interval, 2021-03-24, openshift#17131), but is otherwise unchanged.
18fa6fe to
7da7d9a
Compare
|
@wking: The following tests 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. |
$ curl -s https://gcsweb-ci.apps.ci.l2s4.p1.openshiftapps.com/gcs/origin-ci-test/pr-logs/pull/openshift_release/31580/rehearse-31580-pull-ci-openshift-cluster-version-operator-master-e2e-agnostic-upgrade-out-of-change/1561907517615771648/artifacts/e2e-agnostic-upgrade-out-of-change/gather-extra/artifacts/clusterversion.json | jq -r '.items[].status.history[] | .startedTime + " " + .completionTime + " " + .state + " " + .version'
2022-08-23T03:51:45Z 2022-08-23T05:02:24Z Completed 4.12.0-0.ci.test-2022-08-23-024755-ci-op-k2m849xk-initial
2022-08-23T03:12:06Z 2022-08-23T03:48:19Z Completed 4.12.0-0.ci.test-2022-08-23-025711-ci-op-k2m849xk-latestLooks plausible for latest -> initial. |
LalatenduMohanty
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: LalatenduMohanty, wking 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 |
|
@wking: Updated the following 2 configmaps:
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. |
My initial approach from acd81b7 (#31518) doesn't seem to have worked. Follow up by pattern-matching from ebbee6d (#12618) from openshift/kubernetes, which was renamed to
e2e-{platform}-downgradein 6841f1c (#17131), but is otherwise unchanged.