Skip to content

blocked-edges: add risk for Portworx on 4.22.9 - #10591

Open
simkam wants to merge 1 commit into
openshift:masterfrom
simkam:PortworxKernelModuleBuildFailure
Open

simkam wants to merge 1 commit into
openshift:masterfrom
simkam:PortworxKernelModuleBuildFailure

Conversation

@simkam

@simkam simkam commented Sep 8, 2026

Copy link
Copy Markdown

Clusters running Portworx may lose storage after upgrading to 4.22.9. The RHCOS kernel (5.14.0-687.35.1.el9_8) shipped with that version is missing from RHEL package repositories, preventing Portworx from building its kernel module.

Spike: https://redhat.atlassian.net/browse/OTA-2119
Bug: https://redhat.atlassian.net/browse/OCPBUGS-121207

@coderabbitai

coderabbitai Bot commented Sep 8, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Repository YAML (base), Central YAML (inherited)

Review profile: CHILL

Plan: Advanced

Run ID: 535f7f8a-8134-453c-858c-49c43e9e802d

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

Comment @coderabbitai help to get the list of available commands.

@openshift-ci
openshift-ci Bot requested review from fao89 and hongkailiu September 8, 2026 10:47
@openshift-ci

openshift-ci Bot commented Sep 8, 2026

Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: simkam
Once this PR has been reviewed and has the lgtm label, please assign bennerv 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

@simkam
simkam force-pushed the PortworxKernelModuleBuildFailure branch from 686cda5 to 307b73f Compare September 8, 2026 11:02
@simkam

simkam commented Sep 8, 2026

Copy link
Copy Markdown
Author

/hold ongoing discussion

@openshift-ci openshift-ci Bot added the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Sep 8, 2026
@simkam

simkam commented Sep 8, 2026

Copy link
Copy Markdown
Author

/hold cancel

@openshift-ci openshift-ci Bot removed the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Sep 8, 2026
@@ -0,0 +1,17 @@
to: 4.22.9
from: .*
url: https://redhat.atlassian.net/browse/OCPBUGS-121207

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

we usually point at the impact statement ticket, in this case OTA-2119, and not at the OCPBUGS. Reasons include:

  • The OCPBUGS is often written from a reporter to the component developer, to help them understand what's broken, so they can work up a fix. In this case, with OCPBUGS-121207, I'm not sure what the fix would be. 4.22.10 has already shipped, so there's no change to make in the shipped product, right? The impact statement, on the other hand, is written to the cluster admin considering an update into an impacted release. So it can cover things like "How involved is remediation?" that a cluster-admin likely cares about, but which a developer trying to fix a bug likely does not care about.
  • OCPBUGS are tied to a specific target release, and one issue impacting multiple branches will have a separate OCPBUGS tracking a backport into each branch. Impact statements can talk about the issue as it impacts updates, without being pinned to a single target release branch.

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! Updated.

- type: PromQL
promql:
promql: |
group by (_id) (csv_succeeded{_id="",name=~"portworx-operator[.]v.*"})

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It doesn't impact cluster-version operator functionality, but personally, I find this kind of PromQL easier to debug if we keep name around instead of group aggregating it away. This is a recent example

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! Updated.

Clusters running Portworx may lose storage after upgrading to 4.22.9.
The RHCOS kernel (5.14.0-687.35.1.el9_8) shipped with that version is
missing from RHEL package repositories, preventing Portworx from
building its kernel module.

Spike: https://redhat.atlassian.net/browse/OTA-2119
Bug: https://redhat.atlassian.net/browse/OCPBUGS-121207

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@simkam
simkam force-pushed the PortworxKernelModuleBuildFailure branch from 307b73f to 317e078 Compare September 9, 2026 09:02
@openshift-ci

openshift-ci Bot commented Sep 9, 2026

Copy link
Copy Markdown
Contributor

@simkam: 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.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants