TRT-2560: Revert #638 "OCPBUGS-65634: add service account to verify pod"#647
TRT-2560: Revert #638 "OCPBUGS-65634: add service account to verify pod"#647petr-muller wants to merge 1 commit intoopenshift:mainfrom
Conversation
|
@petr-muller: This pull request references TRT-2560 which is a valid jira issue. Warning: The referenced jira issue has an invalid target version for the target branch this PR targets: expected the ticket to target the "4.22.0" version, but no target version was set. 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. |
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: petr-muller The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
|
/payload-aggregate periodic-ci-openshift-release-main-ci-4.22-upgrade-from-stable-4.21-e2e-azure-ovn-upgrade 10 |
|
@petr-muller: 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/82ce5630-1309-11f1-9d45-a4fe0be5e7cf-0 |
|
/hold wait for #647 (comment) |
07e48ef to
aacce72
Compare
|
@petr-muller, #638 broke our downstream bumper, so we have to manually revert it using our bumper process anyways... I'm working on it now. |
|
See #648 for a bumper fix. |
|
No signs of the issue in https://pr-payload-tests.ci.openshift.org/runs/ci/82ce5630-1309-11f1-9d45-a4fe0be5e7cf-0 which confirms the revert addresses TRT-2560 |
|
/verified by https://pr-payload-tests.ci.openshift.org/runs/ci/82ce5630-1309-11f1-9d45-a4fe0be5e7cf-0 Not sure if #648 is the thing that needs to merge instead of the revert here, asking on Slack. Leaving the hold here for now. |
|
@petr-muller: 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. |
|
/close We need #648 |
|
@petr-muller: Closed this PR. 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. |
Reverts #638 ; tracked by TRT-2560
Per OpenShift policy, we are reverting this breaking change to get CI and/or nightly payloads flowing again.
The
verify-all-openshiftredhatoperatorspod introduced by this PR triggers theKubePodNotReadyalert (alert/KubePodNotReady should not be at or above info in ns/default) in 4.22 upgrade-from-4.21 CI jobs across multiple platforms (AWS, Azure with OVN). This is causing aggregated job failures and blocking nightly payloads.Example aggregated failure: https://prow.ci.openshift.org/view/gs/test-platform-results/logs/aggregated-aws-ovn-upgrade-4.22-minor-release-openshift-release-analysis-aggregator/2026880209889792000
Example individual failure: https://prow.ci.openshift.org/view/gs/test-platform-results/logs/periodic-ci-openshift-release-main-ci-4.22-upgrade-from-stable-4.21-e2e-azure-ovn-upgrade/2026898179982626816
To unrevert this, revert this PR, and layer an additional separate commit on top that addresses the problem. Before merging the unrevert, please run these jobs on the PR and check the result of these jobs to confirm the fix has corrected the problem:
CC: @ehearne-redhat