Skip to content

Conversation

@neisw
Copy link
Contributor

@neisw neisw commented Dec 11, 2025

Reverts #5390
This is for testing only to get back to a state prior to 4.21.0-0.nightly-2025-11-27-173427 where we have picked up disruption noted in OCPBUGS-66420 most likely case is we rule out that any of these changes are in play here.

@openshift-ci-robot
Copy link
Contributor

openshift-ci-robot commented Dec 11, 2025

@neisw: This pull request references OCPNODE-3719 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 story to target the "4.21.0" version, but no target version was set.

Details

In response to this:

Reverts #5390
This is for testing only to get back to a state prior to 4.21.0-0.nightly-2025-11-27-173427 where we have picked up disruption noted in OCPBUGS-66420 most likely case is we rule out that any of these changes are in play here.

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.

@openshift-ci-robot openshift-ci-robot added the jira/valid-reference Indicates that this PR references a valid Jira ticket of any type. label Dec 11, 2025
@openshift-ci openshift-ci bot added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Dec 11, 2025
@openshift-ci
Copy link
Contributor

openshift-ci bot commented Dec 11, 2025

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

@neisw
Copy link
Contributor Author

neisw commented Dec 11, 2025

/payload-aggregate periodic-ci-openshift-release-master-nightly-4.21-e2e-aws-ovn-upgrade-fips 10

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Dec 11, 2025

@neisw: trigger 1 job(s) for the /payload-(with-prs|job|aggregate|job-with-prs|aggregate-with-prs) command

  • periodic-ci-openshift-release-master-nightly-4.21-e2e-aws-ovn-upgrade-fips

See details on https://pr-payload-tests.ci.openshift.org/runs/ci/e2ef6e30-d6a7-11f0-86ba-8f6c48362a62-0

@neisw
Copy link
Contributor Author

neisw commented Dec 11, 2025

/payload-aggregate periodic-ci-openshift-release-master-nightly-4.21-e2e-aws-ovn-upgrade-fips 10

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Dec 11, 2025

@neisw: trigger 1 job(s) for the /payload-(with-prs|job|aggregate|job-with-prs|aggregate-with-prs) command

  • periodic-ci-openshift-release-master-nightly-4.21-e2e-aws-ovn-upgrade-fips

See details on https://pr-payload-tests.ci.openshift.org/runs/ci/cab69080-d6cc-11f0-9919-32007c53b184-0

@neisw
Copy link
Contributor Author

neisw commented Dec 12, 2025

/payload-aggregate-with-prs periodic-ci-openshift-release-master-nightly-4.21-e2e-aws-ovn-upgrade-fips 10 openshift/cluster-api#254

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Dec 12, 2025

@neisw: trigger 1 job(s) for the /payload-(with-prs|job|aggregate|job-with-prs|aggregate-with-prs) command

  • periodic-ci-openshift-release-master-nightly-4.21-e2e-aws-ovn-upgrade-fips

See details on https://pr-payload-tests.ci.openshift.org/runs/ci/03eb04e0-d6f9-11f0-8261-7255b3fcf62c-0

@neisw
Copy link
Contributor Author

neisw commented Dec 12, 2025

/payload-aggregate-with-prs periodic-ci-openshift-release-master-nightly-4.21-e2e-aws-ovn-upgrade-fips 10 openshift/cluster-api#254

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Dec 12, 2025

@neisw: trigger 1 job(s) for the /payload-(with-prs|job|aggregate|job-with-prs|aggregate-with-prs) command

  • periodic-ci-openshift-release-master-nightly-4.21-e2e-aws-ovn-upgrade-fips

See details on https://pr-payload-tests.ci.openshift.org/runs/ci/b521a1b0-d749-11f0-9c6f-49bba9ef02f5-0

@neisw neisw marked this pull request as ready for review December 12, 2025 12:01
@openshift-ci openshift-ci bot removed the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Dec 12, 2025
@neisw
Copy link
Contributor Author

neisw commented Dec 12, 2025

/payload-aggregate-with-prs periodic-ci-openshift-release-master-nightly-4.21-e2e-aws-ovn-upgrade-fips 10 openshift/cluster-api#254

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Dec 12, 2025

@neisw: trigger 1 job(s) for the /payload-(with-prs|job|aggregate|job-with-prs|aggregate-with-prs) command

  • periodic-ci-openshift-release-master-nightly-4.21-e2e-aws-ovn-upgrade-fips

See details on https://pr-payload-tests.ci.openshift.org/runs/ci/7c333c90-d755-11f0-917b-f286c81cb85b-0

@neisw
Copy link
Contributor Author

neisw commented Dec 12, 2025

/payload-aggregate-with-prs periodic-ci-openshift-release-master-nightly-4.22-e2e-aws-ovn-upgrade-fips 10 openshift/cluster-api#254

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Dec 12, 2025

@neisw: trigger 1 job(s) for the /payload-(with-prs|job|aggregate|job-with-prs|aggregate-with-prs) command

  • periodic-ci-openshift-release-master-nightly-4.22-e2e-aws-ovn-upgrade-fips

See details on https://pr-payload-tests.ci.openshift.org/runs/ci/80db55f0-d757-11f0-9eab-e3cee46cb112-0

@neisw
Copy link
Contributor Author

neisw commented Dec 12, 2025

/retest-required

@neisw neisw changed the title Revert "OCPNODE-3719: Default Enablement of Auto Sizing Reserved in OpenShift 4.21" OCPBUGS-66420: Revert "Default Enablement of Auto Sizing Reserved in OpenShift 4.21" Dec 12, 2025
@openshift-ci-robot openshift-ci-robot added the jira/invalid-bug Indicates that a referenced Jira bug is invalid for the branch this PR is targeting. label Dec 12, 2025
@openshift-ci-robot
Copy link
Contributor

@neisw: This pull request references Jira Issue OCPBUGS-66420, which is invalid:

  • expected the bug to target the "4.22.0" version, but no target version was set

Comment /jira refresh to re-evaluate validity if changes to the Jira bug are made, or edit the title of this pull request to link to a different bug.

The bug has been updated to refer to the pull request using the external bug tracker.

Details

In response to this:

Reverts #5390
This is for testing only to get back to a state prior to 4.21.0-0.nightly-2025-11-27-173427 where we have picked up disruption noted in OCPBUGS-66420 most likely case is we rule out that any of these changes are in play here.

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.

@dgoodwin
Copy link
Contributor

/lgtm

We have evidence this has cut the occurrence of the problem dramatically, though we have found a couple hits that look similar amidst the 30 job runs that have been completed. Given the PR is logically consistent with what we've observed, in the payload the trouble started, and explains the micro vs minor upgrade differences we've seen, we're going to proceed with the revert. We believe these jobs were already banging up against max master CPU at this point in testing, the change in throttling memory or cpu pushed things over the edge into failure. Whether our test jobs should be hitting CPU so hard is another question and TRT/MPEX is working on ways to better schedule tests and the concurrency with which they run. Difficult problem to solve with the scope of what tests coming from all over could be attempting.

It has to be said the case is not a slam dunk, why 2/30 runs still showed the problem, why we don't see it in fresh 4.21 installs that then run conformance (only micro upgrades on aws and gcp). If we're wrong and this does not stabilize CI, we will bring back in immediately with our sincerest apologies.

@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Dec 12, 2025
@openshift-ci
Copy link
Contributor

openshift-ci bot commented Dec 12, 2025

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: dgoodwin, neisw
Once this PR has been reviewed and has the lgtm label, please assign yuqi-zhang for approval. For more information see the Code Review Process.

The full list of commands accepted by this bot can be found here.

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@dgoodwin
Copy link
Contributor

/jira refresh

@openshift-ci-robot openshift-ci-robot added jira/valid-bug Indicates that a referenced Jira bug is valid for the branch this PR is targeting. and removed jira/invalid-bug Indicates that a referenced Jira bug is invalid for the branch this PR is targeting. labels Dec 12, 2025
@openshift-ci-robot
Copy link
Contributor

@dgoodwin: This pull request references Jira Issue OCPBUGS-66420, which is valid. The bug has been moved to the POST state.

3 validation(s) were run on this bug
  • bug is open, matching expected state (open)
  • bug target version (4.22.0) matches configured target version for branch (4.22.0)
  • bug is in the state New, which is one of the valid states (NEW, ASSIGNED, POST)
Details

In response to this:

/jira refresh

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.

@neisw
Copy link
Contributor Author

neisw commented Dec 12, 2025

/label acknowledge-critical-fixes-only

@openshift-ci openshift-ci bot added the acknowledge-critical-fixes-only Indicates if the issuer of the label is OK with the policy. label Dec 12, 2025
@dgoodwin
Copy link
Contributor

/label approved

@openshift-ci openshift-ci bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Dec 12, 2025
@dgoodwin
Copy link
Contributor

/verified by payload testing

@openshift-ci-robot openshift-ci-robot added the verified Signifies that the PR passed pre-merge verification criteria label Dec 12, 2025
@dgoodwin
Copy link
Contributor

/cherry-pick release-4.21

@openshift-ci-robot
Copy link
Contributor

@dgoodwin: This PR has been marked as verified by payload testing.

Details

In response to this:

/verified by payload testing

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.

@openshift-cherrypick-robot

@dgoodwin: once the present PR merges, I will cherry-pick it on top of release-4.21 in a new PR and assign it to you.

Details

In response to this:

/cherry-pick release-4.21

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.

@dgoodwin dgoodwin merged commit 08c94ea into openshift:main Dec 12, 2025
11 of 14 checks passed
@openshift-ci-robot
Copy link
Contributor

@neisw: Jira Issue OCPBUGS-66420: Some pull requests linked via external trackers have merged:

The following pull request, linked via external tracker, has not merged:

All associated pull requests must be merged or unlinked from the Jira bug in order for it to move to the next state. Once unlinked, request a bug refresh with /jira refresh.

Jira Issue OCPBUGS-66420 has not been moved to the MODIFIED state.

This PR is marked as verified. If the remaining PRs listed above are marked as verified before merging, the issue will automatically be moved to VERIFIED after all of the changes from the PRs are available in an accepted nightly payload.

Details

In response to this:

Reverts #5390
This is for testing only to get back to a state prior to 4.21.0-0.nightly-2025-11-27-173427 where we have picked up disruption noted in OCPBUGS-66420 most likely case is we rule out that any of these changes are in play here.

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.

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Dec 12, 2025

@neisw: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name Commit Details Required Rerun command
ci/prow/bootstrap-unit cd5c07b link false /test bootstrap-unit

Full PR test history. Your PR dashboard.

Details

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. I understand the commands that are listed here.

@openshift-cherrypick-robot

@dgoodwin: new pull request created: #5493

Details

In response to this:

/cherry-pick release-4.21

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.

@neisw
Copy link
Contributor Author

neisw commented Dec 12, 2025

/payload-aggregate periodic-ci-openshift-release-master-nightly-4.22-e2e-aws-ovn-upgrade-fips 10

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Dec 12, 2025

@neisw: trigger 1 job(s) for the /payload-(with-prs|job|aggregate|job-with-prs|aggregate-with-prs) command

  • periodic-ci-openshift-release-master-nightly-4.22-e2e-aws-ovn-upgrade-fips

See details on https://pr-payload-tests.ci.openshift.org/runs/ci/ff55a130-d79b-11f0-9a25-9f734a58f934-0

@openshift-merge-robot
Copy link
Contributor

Fix included in accepted release 4.22.0-0.nightly-2025-12-13-084344

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

acknowledge-critical-fixes-only Indicates if the issuer of the label is OK with the policy. approved Indicates a PR has been approved by an approver from all required OWNERS files. jira/valid-bug Indicates that a referenced Jira bug is valid for the branch this PR is targeting. jira/valid-reference Indicates that this PR references a valid Jira ticket of any type. lgtm Indicates that a PR is ready to be merged. verified Signifies that the PR passed pre-merge verification criteria

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants