Skip to content

Upgrade to React 18 (#9323) - #773

Open
hhovsepy wants to merge 1 commit into
kiali:mainfrom
hhovsepy:issue9323
Open

hhovsepy wants to merge 1 commit into
kiali:mainfrom
hhovsepy:issue9323

Conversation

@hhovsepy

Copy link
Copy Markdown
Contributor

Summary

  • Sync vendored Kiali frontend from kiali/kiali#10045 (commit feed22b38), including the React 18 migration and Graph/Mesh first-load layout fix.
  • Upgrade OSSMC plugin dependencies to React 18 (react/react-dom ^18.3.1, updated @types/*).
  • Disable ConsoleRemotePlugin shared-module validation so the plugin can build before the OpenShift Console SDK is upgraded to 4.22+ (which also requires Redux 5 / React Router 7).

Companion PR for upstream Kiali: kiali/kiali#10045

Test plan

  • cd plugin && yarn install
  • cd plugin && yarn test (75 unit tests pass)
  • cd plugin && yarn build
  • Deploy plugin image to OCP and verify OSSMC loads with React 18 Console
  • Smoke-test Graph and Mesh pages (first-load layout fix)
  • Run OSSMC Cypress core tests on OCP

OCP note: Runtime React is provided by the OpenShift Console via module federation. Test on a cluster whose Console ships React 18 (OCP 4.22+). Clusters with React 17 Console (4.19–4.21) are not compatible with this change.

Made with Cursor

@hhovsepy
hhovsepy force-pushed the issue9323 branch 2 times, most recently from 7d222e7 to fb29cb7 Compare July 20, 2026 09:27
Refresh plugin deps/webpack for Console 4.22, keep pluginAPI *, and
vendored Kiali frontend from the latest PR 10045 tip.

Co-authored-by: Cursor <cursoragent@cursor.com>
@jmazzitelli

jmazzitelli commented Jul 28, 2026

Copy link
Copy Markdown
Contributor

BTW: I'm keeping an eye on this and will need to sync up with this PR (if we do end up going this route with the new fleet service mesh plugin): #785

I already merged that PR with this one so I am trying to track the changes being made here.

@hhovsepy hhovsepy mentioned this pull request Jul 29, 2026
7 tasks
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