Skip to content

chore(e2e): remove migrated OCI plugin tests and CI infrastructure#4763

Merged
openshift-merge-bot[bot] merged 6 commits intoredhat-developer:mainfrom
teknaS47:e2e-cleanup-oci
May 8, 2026
Merged

chore(e2e): remove migrated OCI plugin tests and CI infrastructure#4763
openshift-merge-bot[bot] merged 6 commits intoredhat-developer:mainfrom
teknaS47:e2e-cleanup-oci

Conversation

@teknaS47
Copy link
Copy Markdown
Member

@teknaS47 teknaS47 commented May 6, 2026

Summary

  • Remove E2E tests for OCI-based plugins migrated to the rhdh-plugin-export-overlays repository (Phase 3 — RHIDP-8841)
  • Removes 7 test suites, their exclusive utilities/page objects, OCI plugin entries from Helm values files, and related CI config
  • 31 files changed, ~1,462 lines removed

Jira: RHIDP-13448

Tests Removed

Test Files Deleted Config Removed
Custom Theme custom-theme.spec.ts, theme-verifier.ts, custom-theme.ts, theme-constants.ts testMatch entries in localization projects in playwright.config.ts; @pataknight/rhdh-qe-theme OCI entry from values_showcase.yaml; theme block from dynamic-plugins-config.yaml
Scorecard scorecard.spec.ts, scorecard-page.ts 4 OCI entries from values_showcase-rbac.yaml; 6 catalog locations + permission from app-config-rhdh-rbac.yaml; scorecard.metric.read from rbac-policy.csv and rbac-constants.ts
Quay quay.spec.ts, quay-actions.spec.ts, quay.ts, quay-client.ts 3 OCI entries from values_showcase.yaml, 2 from values_showcase-rbac.yaml, 1 from diff-values_showcase-sanity-plugins.yaml; quay-actions template from app-config-rhdh.yaml; Quay FAB entry from FAB config; /quay/api proxy preserved (required by http-request.spec.ts scaffolder test)
GitLab Discovery gitlab-discovery.spec.ts immobiliarelabs-backstage-plugin-gitlab-backend OCI entry from values_showcase.yaml
Global Header (test) header-mount-points.spec.ts global-header-test OCI entry from values_showcase.yaml; fixture removed from wrappers.test.ts
Dynamic Home Page dynamic-home-page-customization.spec.ts, home-page-customization.ts dynamic-home-page OCI entry + disabled: true companion from values_showcase-rbac.yaml; testIgnore entry from playwright.config.ts
GitHub Issues (partial removal from github-happy-path.spec.ts) github-issues OCI entry from values_showcase.yaml + values_showcase-rbac.yaml
GitHub Actions (partial removal from github-happy-path.spec.ts) github-actions OCI entry from values_showcase.yaml + values_showcase-rbac.yaml

Shared Dependencies Preserved

  • github-pull-requests OCI plugin (still used by PR tab tests in github-happy-path.spec.ts)
  • All shared utilities (UIhelper, Common, CatalogImport, etc.)
  • Tekton/Topology shared CI infrastructure (deferred to RHIDP-13449)

Dead Code Cleanup

  • Removed getGithubOpenIssues() and getWorkflowRuns() from BackstageShowcase class
  • Removed unused GITHUB_API_ENDPOINTS.issues, GITHUB_API_ENDPOINTS.workflowRuns, and backstageShowcaseAPI
  • Removed QUAY_NAMESPACE, QUAY_TOKEN, JIRA_TOKEN, and GITLAB_TOKEN from env_variables.sh and secrets-rhdh-secrets.yaml
  • Deleted orphaned pipelines-operator.yaml OLM Subscription manifest (operator installed programmatically via lib/operators.sh)

Test plan

  • OCP Helm PR job (showcase + showcase-rbac) — validates deployment with removed OCI plugins and remaining tests pass
  • OCP Helm Nightly job — validates showcase-sanity-plugins, localization, and runtime
  • OCP Operator job — validates showcase-operator + showcase-operator-rbac

@openshift-ci openshift-ci Bot requested review from HusneShabbir and ciiay May 6, 2026 06:42
@codecov
Copy link
Copy Markdown

codecov Bot commented May 6, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 69.49%. Comparing base (12f20c1) to head (2051fe7).

Additional details and impacted files
@@             Coverage Diff             @@
##             main    #4763       +/-   ##
===========================================
+ Coverage   40.88%   69.49%   +28.60%     
===========================================
  Files         119      109       -10     
  Lines        2228     4710     +2482     
  Branches      563      537       -26     
===========================================
+ Hits          911     3273     +2362     
- Misses       1311     1437      +126     
+ Partials        6        0        -6     
Flag Coverage Δ
install-dynamic-plugins 92.44% <ø> (?)
rhdh 38.65% <ø> (-2.24%) ⬇️

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 12f20c1...2051fe7. Read the comment docs.

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

@teknaS47
Copy link
Copy Markdown
Member Author

teknaS47 commented May 6, 2026

/test e2e-ocp-helm-nightly
/test e2e-ocp-operator

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 6, 2026

Image was built and published successfully. It is available at:

@teknaS47
Copy link
Copy Markdown
Member Author

teknaS47 commented May 6, 2026

/test e2e-ocp-operator

@teknaS47
Copy link
Copy Markdown
Member Author

teknaS47 commented May 6, 2026

/test e2e-ocp-operator-nightly

Comment thread .ci/pipelines/resources/config_map/app-config-rhdh.yaml
@teknaS47 teknaS47 force-pushed the e2e-cleanup-oci branch from a258925 to 432e34a Compare May 6, 2026 11:18
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 6, 2026

Image was built and published successfully. It is available at:

@teknaS47 teknaS47 force-pushed the e2e-cleanup-oci branch from 9eace9c to 94beec4 Compare May 6, 2026 12:40
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 6, 2026

The container image build workflow finished with status: cancelled.

@teknaS47 teknaS47 force-pushed the e2e-cleanup-oci branch from 94beec4 to e0b514e Compare May 6, 2026 12:49
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 6, 2026

The container image build workflow finished with status: cancelled.

@teknaS47
Copy link
Copy Markdown
Member Author

teknaS47 commented May 6, 2026

/test e2e-ocp-helm-nightly
/test e2e-ocp-operator-nightly

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 6, 2026

Image was built and published successfully. It is available at:

@teknaS47
Copy link
Copy Markdown
Member Author

teknaS47 commented May 6, 2026

/test e2e-ocp-helm-nightly
/test e2e-ocp-operator-nightly

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 6, 2026

Image was built and published successfully. It is available at:

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 7, 2026

Image was built and published successfully. It is available at:

@teknaS47
Copy link
Copy Markdown
Member Author

teknaS47 commented May 7, 2026

/test e2e-ocp-helm-nightly
/test e2e-ocp-operator-nightly

@teknaS47 teknaS47 force-pushed the e2e-cleanup-oci branch from fdb1ff5 to 4dff924 Compare May 7, 2026 08:56
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 7, 2026

The container image build workflow finished with status: cancelled.

@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented May 7, 2026

❌ The last analysis has failed.

See analysis details on SonarQube Cloud

@teknaS47
Copy link
Copy Markdown
Member Author

teknaS47 commented May 7, 2026

/test e2e-ocp-helm-nightly
/test e2e-ocp-operator-nightly

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 7, 2026

Image was built and published successfully. It is available at:

@teknaS47 teknaS47 force-pushed the e2e-cleanup-oci branch from 8088550 to 87095ad Compare May 7, 2026 11:36
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 7, 2026

Image was built and published successfully. It is available at:

@teknaS47
Copy link
Copy Markdown
Member Author

teknaS47 commented May 8, 2026

/test e2e-ocp-helm-nightly
/test e2e-ocp-operator-nightly

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 8, 2026

Image was built and published successfully. It is available at:

@openshift-ci
Copy link
Copy Markdown

openshift-ci Bot commented May 8, 2026

@teknaS47: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name Commit Details Required Rerun command
ci/prow/e2e-ocp-operator-nightly 2051fe7 link false /test e2e-ocp-operator-nightly
ci/prow/e2e-ocp-helm-nightly 2051fe7 link false /test e2e-ocp-helm-nightly

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.

@subhashkhileri
Copy link
Copy Markdown
Member

/lgtm
/approve

@subhashkhileri
Copy link
Copy Markdown
Member

/hold

@teknaS47 teknaS47 force-pushed the e2e-cleanup-oci branch from 2051fe7 to b5f9707 Compare May 8, 2026 13:59
@openshift-ci openshift-ci Bot removed the lgtm label May 8, 2026
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 8, 2026

The container image build and publish workflows were skipped (either due to [skip-build] tag or no relevant changes with existing image).

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 8, 2026

The container image build and publish workflows were skipped (either due to [skip-build] tag or no relevant changes with existing image).

@subhashkhileri
Copy link
Copy Markdown
Member

/unhold

Copy link
Copy Markdown
Member

@subhashkhileri subhashkhileri left a comment

Choose a reason for hiding this comment

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

/lgtm

@openshift-ci openshift-ci Bot added the lgtm label May 8, 2026
@openshift-ci openshift-ci Bot removed the lgtm label May 8, 2026
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 8, 2026

The container image build and publish workflows were skipped (either due to [skip-build] tag or no relevant changes with existing image).

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 8, 2026

The container image build and publish workflows were skipped (either due to [skip-build] tag or no relevant changes with existing image).

Copy link
Copy Markdown
Member

@subhashkhileri subhashkhileri left a comment

Choose a reason for hiding this comment

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

/lgtm

@openshift-ci openshift-ci Bot added the lgtm label May 8, 2026
@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented May 8, 2026

@openshift-merge-bot openshift-merge-bot Bot merged commit e59cf01 into redhat-developer:main May 8, 2026
16 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants