Skip to content

Conversation

@jmencak
Copy link
Contributor

@jmencak jmencak commented Dec 15, 2025

This change improves user's upgrade experience as outlined by OCPSTRAT-2484 by fixing false alarms in ClusterOperator status.

The following new rule is adhered to:

  • Operators should not report Progressing only because DaemonSets owned by them are adjusting to a new node from cluster scaleup or a node rebooting from cluster upgrade.

This is achieved by adding helper annotation
tuned.openshift.io/last-stable-daemonset-generation to the tuned daemonset.

Other minor fixes:

  • Minor logging fix (updateTunedProfileStatus())
  • Removed non-functional code from the operator initializing ClusterOperator status

Resolves: OCPBUGS-62632

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Dec 15, 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

@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 15, 2025
@openshift-ci-robot openshift-ci-robot added jira/valid-reference Indicates that this PR references a valid Jira ticket of any type. jira/invalid-bug Indicates that a referenced Jira bug is invalid for the branch this PR is targeting. labels Dec 15, 2025
@openshift-ci-robot
Copy link
Contributor

openshift-ci-robot commented Dec 15, 2025

@jmencak: This pull request references Jira Issue OCPBUGS-62632, 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:

This change improves user's upgrade experience as outlined by OCPSTRAT-2484 by fixing false alarms in ClusterOperator status.

The following new rule is adhered to:

  • Operators should not report Progressing only because DaemonSets owned by them are adjusting to a new node from cluster scaleup or a node rebooting from cluster upgrade.

This is achieved by adding helper annotation
tuned.openshift.io/stable-generation to the tuned daemonset.

Other minor fixes:

  • Minor logging fix (updateTunedProfileStatus())
  • Update profile.Status also when profile.Status.ObservedGeneration != profile.Generation
  • Removed non-functional code from the operator initializing ClusterOperator status

Resolves: OCPBUGS-62632

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 openshift-ci bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Dec 15, 2025
@jmencak
Copy link
Contributor Author

jmencak commented Dec 15, 2025

/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 15, 2025
@openshift-ci-robot
Copy link
Contributor

@jmencak: This pull request references Jira Issue OCPBUGS-62632, 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 ASSIGNED, which is one of the valid states (NEW, ASSIGNED, POST)

No GitHub users were found matching the public email listed for the QA contact in Jira ([email protected]), skipping review request.

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.

@jmencak
Copy link
Contributor Author

jmencak commented Dec 15, 2025

/test all

@jmencak
Copy link
Contributor Author

jmencak commented Dec 15, 2025

/payload-job periodic-ci-openshift-release-master-ci-4.22-upgrade-from-stable-4.21-e2e-aws-ovn-upgrade

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Dec 15, 2025

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

  • periodic-ci-openshift-release-master-ci-4.22-upgrade-from-stable-4.21-e2e-aws-ovn-upgrade

See details on https://pr-payload-tests.ci.openshift.org/runs/ci/7bda56e0-d9b5-11f0-8a9c-d6494fa18d6f-0

@jmencak
Copy link
Contributor Author

jmencak commented Dec 15, 2025

/test e2e-aws-ovn-techpreview

@jmencak jmencak marked this pull request as ready for review December 15, 2025 17:59
@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 15, 2025
@openshift-ci openshift-ci bot requested review from Tal-or and swatisehgal December 15, 2025 17:59
@jmencak
Copy link
Contributor Author

jmencak commented Dec 15, 2025

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

* periodic-ci-openshift-release-master-ci-4.22-upgrade-from-stable-4.21-e2e-aws-ovn-upgrade

See details on https://pr-payload-tests.ci.openshift.org/runs/ci/7bda56e0-d9b5-11f0-8a9c-d6494fa18d6f-0

Upgrade test looks good, no legacy-cvo-invariants test issues from co/node-tuning.

@jmencak
Copy link
Contributor Author

jmencak commented Jan 8, 2026

/test all

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Jan 8, 2026

@jmencak: all tests passed!

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.

@jmencak
Copy link
Contributor Author

jmencak commented Jan 8, 2026

/payload-job periodic-ci-openshift-release-master-ci-4.22-upgrade-from-stable-4.21-e2e-aws-ovn-upgrade

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Jan 8, 2026

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

  • periodic-ci-openshift-release-master-ci-4.22-upgrade-from-stable-4.21-e2e-aws-ovn-upgrade

See details on https://pr-payload-tests.ci.openshift.org/runs/ci/80d32230-ec98-11f0-8481-3c171578c72f-0

@jmencak
Copy link
Contributor Author

jmencak commented Jan 9, 2026

After running periodic-ci-openshift-release-master-ci-4.22-upgrade-from-stable-4.21-e2e-aws-ovn-upgrade job twice, I'm not seeing any issues with [Monitor:legacy-cvo-invariants][bz-Node Tuning Operator].
/verified by periodic-ci-openshift-release-master-ci-4.22-upgrade-from-stable-4.21-e2e-aws-ovn-upgrade

@openshift-ci-robot openshift-ci-robot added the verified Signifies that the PR passed pre-merge verification criteria label Jan 9, 2026
@openshift-ci-robot
Copy link
Contributor

@jmencak: This PR has been marked as verified by periodic-ci-openshift-release-master-ci-4.22-upgrade-from-stable-4.21-e2e-aws-ovn-upgrade.

Details

In response to this:

After running periodic-ci-openshift-release-master-ci-4.22-upgrade-from-stable-4.21-e2e-aws-ovn-upgrade job twice, I'm not seeing any issues with [Monitor:legacy-cvo-invariants][bz-Node Tuning Operator].
/verified by periodic-ci-openshift-release-master-ci-4.22-upgrade-from-stable-4.21-e2e-aws-ovn-upgrade

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
Copy link
Contributor

@jmencak: This pull request references Jira Issue OCPBUGS-62632, which is valid.

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 POST, which is one of the valid states (NEW, ASSIGNED, POST)

No GitHub users were found matching the public email listed for the QA contact in Jira ([email protected]), skipping review request.

Details

In response to this:

This change improves user's upgrade experience as outlined by OCPSTRAT-2484 by fixing false alarms in ClusterOperator status.

The following new rule is adhered to:

  • Operators should not report Progressing only because DaemonSets owned by them are adjusting to a new node from cluster scaleup or a node rebooting from cluster upgrade.

This is achieved by adding helper annotation
tuned.openshift.io/last-stable-daemonset-generation to the tuned daemonset.

Other minor fixes:

  • Minor logging fix (updateTunedProfileStatus())
  • Removed non-functional code from the operator initializing ClusterOperator status

Resolves: OCPBUGS-62632

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.

@jmencak jmencak force-pushed the 4.22-OCPBUGS-62632-co-progressing branch from 2183008 to a0fdb4a Compare January 12, 2026 10:16
@openshift-ci-robot openshift-ci-robot removed the verified Signifies that the PR passed pre-merge verification criteria label Jan 12, 2026
@jmencak
Copy link
Contributor Author

jmencak commented Jan 12, 2026

/payload-job periodic-ci-openshift-release-master-ci-4.22-upgrade-from-stable-4.21-e2e-aws-ovn-upgrade

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Jan 12, 2026

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

  • periodic-ci-openshift-release-master-ci-4.22-upgrade-from-stable-4.21-e2e-aws-ovn-upgrade

See details on https://pr-payload-tests.ci.openshift.org/runs/ci/6ecf5750-efb0-11f0-91d6-4442dbea1efe-0

…or node reboot

This change improves user's upgrade experience as outlined by
OCPSTRAT-2484 by fixing false alarms in ClusterOperator status.

The following new rule is adhered to:
  * Operators should not report Progressing only because DaemonSets
    owned by them are adjusting to a new node from cluster scaleup or a node
    rebooting from cluster upgrade.

This is achieved by adding helper annotation
"tuned.openshift.io/last-stable-daemonset-generation" to the tuned daemonset.

Other minor fixes:
  * Minor logging fix (updateTunedProfileStatus())
  * Removed non-functional code from the operator initializing ClusterOperator status

Resolves: OCPBUGS-62632
@jmencak jmencak force-pushed the 4.22-OCPBUGS-62632-co-progressing branch from a0fdb4a to 06457f7 Compare January 12, 2026 16:50
@jmencak
Copy link
Contributor Author

jmencak commented Jan 12, 2026

/payload-job periodic-ci-openshift-release-master-ci-4.22-upgrade-from-stable-4.21-e2e-aws-ovn-upgrade

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Jan 12, 2026

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

  • periodic-ci-openshift-release-master-ci-4.22-upgrade-from-stable-4.21-e2e-aws-ovn-upgrade

See details on https://pr-payload-tests.ci.openshift.org/runs/ci/14e4ee40-efd7-11f0-8d58-25f10bcff945-0

@jmencak
Copy link
Contributor Author

jmencak commented Jan 13, 2026

No longer seeing any issues with [Monitor:legacy-cvo-invariants][bz-Node Tuning Operator].
/verified by periodic-ci-openshift-release-master-ci-4.22-upgrade-from-stable-4.21-e2e-aws-ovn-upgrade

@openshift-ci-robot openshift-ci-robot added the verified Signifies that the PR passed pre-merge verification criteria label Jan 13, 2026
@openshift-ci-robot
Copy link
Contributor

@jmencak: This PR has been marked as verified by periodic-ci-openshift-release-master-ci-4.22-upgrade-from-stable-4.21-e2e-aws-ovn-upgrade.

Details

In response to this:

No longer seeing any issues with [Monitor:legacy-cvo-invariants][bz-Node Tuning Operator].
/verified by periodic-ci-openshift-release-master-ci-4.22-upgrade-from-stable-4.21-e2e-aws-ovn-upgrade

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 openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Jan 13, 2026
@openshift-ci
Copy link
Contributor

openshift-ci bot commented Jan 13, 2026

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: jmencak, MarSik

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

The pull request process is described 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

@openshift-merge-bot openshift-merge-bot bot merged commit 7a7311b into openshift:main Jan 13, 2026
18 checks passed
@openshift-ci-robot
Copy link
Contributor

@jmencak: Jira Issue Verification Checks: Jira Issue OCPBUGS-62632
✔️ This pull request was pre-merge verified.
✔️ All associated pull requests have merged.
✔️ All associated, merged pull requests were pre-merge verified.

Jira Issue OCPBUGS-62632 has been moved to the MODIFIED state and will move to the VERIFIED state when the change is available in an accepted nightly payload. 🕓

Details

In response to this:

This change improves user's upgrade experience as outlined by OCPSTRAT-2484 by fixing false alarms in ClusterOperator status.

The following new rule is adhered to:

  • Operators should not report Progressing only because DaemonSets owned by them are adjusting to a new node from cluster scaleup or a node rebooting from cluster upgrade.

This is achieved by adding helper annotation
tuned.openshift.io/last-stable-daemonset-generation to the tuned daemonset.

Other minor fixes:

  • Minor logging fix (updateTunedProfileStatus())
  • Removed non-functional code from the operator initializing ClusterOperator status

Resolves: OCPBUGS-62632

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.

@jmencak jmencak deleted the 4.22-OCPBUGS-62632-co-progressing branch January 13, 2026 09:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

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.

4 participants