Skip to content

ROX-33686: Disable dynamic plugin for OpenShift >= 4.22#19527

Draft
dvail wants to merge 1 commit intodv/ROX-33021-better-plugin-dev-via-sensor-proxyfrom
ROX-33686-disable-plugin-ocp-4.22
Draft

ROX-33686: Disable dynamic plugin for OpenShift >= 4.22#19527
dvail wants to merge 1 commit intodv/ROX-33021-better-plugin-dev-via-sensor-proxyfrom
ROX-33686-disable-plugin-ocp-4.22

Conversation

@dvail
Copy link
Contributor

@dvail dvail commented Mar 20, 2026

Description

Due to OpenShift 4.22 dropping PatternFly 5 support, the dynamic plugin causes runtime issues. This PR updates the webpack config to disable the plugin for OpenShift >= 4.21 by modifying the @console/pluginAPI dependency constraint. The version limit is applied to the plugin manifest at build time.

Note: Currently set to <4.21.0 for testing on a 4.21.4 cluster. Will update to <4.22.0 after verification.

User-facing documentation

  • CHANGELOG.md is updated OR update is not needed
  • documentation PR is created and is linked above OR is not needed

Testing and quality

  • the change is production ready: the change is GA, or otherwise the functionality is gated by a feature flag
  • CI results are inspected

Automated testing

No new tests needed - this is a webpack configuration change that affects plugin manifest generation.

How I validated my change

  • Testing on OpenShift 4.21.4 cluster to verify plugin is disabled
  • Will verify on OpenShift 4.22+ after adjusting version constraint
  • CI build will validate webpack config syntax

Due to OpenShift dropping PatternFly 5 support, disable the plugin
for versions >= 4.21 by updating the webpack config dependency constraint.

This change will be applied to the plugin manifest at build time.

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
@dvail
Copy link
Contributor Author

dvail commented Mar 20, 2026

@openshift-ci
Copy link

openshift-ci bot commented Mar 20, 2026

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

@dvail dvail changed the title ROX-33686: Disable dynamic plugin for OpenShift >= 4.21 ROX-33686: Disable dynamic plugin for OpenShift >= 4.22 Mar 20, 2026
@codecov
Copy link

codecov bot commented Mar 20, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 49.25%. Comparing base (bff801a) to head (68aec06).

Additional details and impacted files
@@                               Coverage Diff                                @@
##           dv/ROX-33021-better-plugin-dev-via-sensor-proxy   #19527   +/-   ##
================================================================================
  Coverage                                            49.25%   49.25%           
================================================================================
  Files                                                 2725     2725           
  Lines                                               205582   205582           
================================================================================
+ Hits                                                101260   101263    +3     
+ Misses                                               96784    96783    -1     
+ Partials                                              7538     7536    -2     
Flag Coverage Δ
go-unit-tests 49.25% <ø> (+<0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@rhacs-bot
Copy link
Contributor

Images are ready for the commit at 68aec06.

To use with deploy scripts, first export MAIN_IMAGE_TAG=4.11.x-364-g68aec06f9f.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants