fix(ci): stop including broken catalog index defaults, disable affected tests#5133
fix(ci): stop including broken catalog index defaults, disable affected tests#5133jonkoops wants to merge 2 commits into
Conversation
Replace all ./dynamic-plugins/dist/ wrapper paths in CI configs, E2E tests, and scripts with explicit OCI image tags from the plugin export overlays metadata. Bump the catalog index image from quay.io/rhdh/plugin-catalog-index:1.10 to quay.io/rhdh/plugin-catalog-index:2.1 to match the current main branch target. Remove dead OCM plugin entries from the GKE config and the corresponding stale ocm.spec.ts test ignore. Refs: RHDHPLAN-256, RHDHPLAN-934 Signed-off-by: Jon Koops <jonkoops@gmail.com>
|
Skipping CI for Draft Pull Request. |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #5133 +/- ##
==========================================
- Coverage 56.03% 55.48% -0.55%
==========================================
Files 123 111 -12
Lines 2404 2186 -218
Branches 553 546 -7
==========================================
- Hits 1347 1213 -134
+ Misses 1051 972 -79
+ Partials 6 1 -5
Continue to review full report in Codecov by Harness.
🚀 New features to boost your workflow:
|
|
/test all |
877df16 to
d4e2d39
Compare
|
/test all |
…ed tests The :1.11 / :next catalog index's dynamic-plugins.default.yaml references OCI images on registry.access.redhat.com that are not publicly accessible. When included, the init container fails to install all 9 default plugins, causing the app to start without core UI plugins and making tests fail. Stop including dynamic-plugins.default.yaml from the catalog index. Add the dynamic-home-page plugin explicitly as an OCI ref (required by the smoke test's "Welcome back!" heading check and the home-page-customization tests). All other plugins needed by CI tests are already listed explicitly as OCI refs in the values files. Disable verify-redis-cache.spec.ts (needs techdocs frontend plugin which is not available as an OCI ref yet). Remove http-request.spec.ts (migrated to overlay repo's roadie-backstage-plugins workspace, source-verified 1:1 match). Clean up stale plugins/rbac/** testIgnore references (directory removed during RBAC migration). Refs: RHIDP-14348 Signed-off-by: Jon Koops <jonkoops@gmail.com>
d4e2d39 to
2ba225a
Compare
|
|
/test e2e-ocp-helm |
|
@jonkoops: The following test failed, say
Full PR test history. Your PR dashboard. DetailsInstructions 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. |
1 participant
Summary
dynamic-plugins.default.yamlfrom the catalog index in CI value files — the:1.11/:nextindex references OCI images onregistry.access.redhat.comthat are not publicly accessible, causing all 9 default plugins to fail to installhome-page-customization,verify-redis-cache)http-request.spec.ts(migrated to overlay repo'sroadie-backstage-pluginsworkspace, source-verified 1:1 functional match)plugins/rbac/**testIgnore references (directory was removed during RBAC plugin migration)Root Cause
The
:next/:1.11catalog index image (quay.io/rhdh/plugin-catalog-index) ships adynamic-plugins.default.yamlthat is only 126 lines (vs 1233 in:1.10). All 9 remaining plugin entries referenceregistry.access.redhat.com/rhdh/...OCI digests that are not accessible:When CI deploys RHDH with
includes: ["dynamic-plugins.default.yaml"], the init container tries to pull these images, fails, and the app starts without any default plugins. This causes theMuiCircularProgressloading spinner to never settle, making all tests flaky (10 flaky, 2 permanently failed in #5122's CI run).Test plan
e2e-ocp-helmCI passes — this is the blockerBuilds on #5113 (OCI ref migration). Unblocks #5122 (dynamic catalog index resolution).
Refs: RHIDP-14348