Skip to content

chore(ci): force operator install script from rhdh-operator#3284#5175

Draft
zdrapela wants to merge 1 commit into
redhat-developer:mainfrom
zdrapela:ci/force-olm-v1-install-script-3284
Draft

chore(ci): force operator install script from rhdh-operator#3284#5175
zdrapela wants to merge 1 commit into
redhat-developer:mainfrom
zdrapela:ci/force-olm-v1-install-script-3284

Conversation

@zdrapela

Copy link
Copy Markdown
Member

Description

Temporary draft PR to validate the OLM v1 install-script fix from rhdh-operator#3284 against OCP operator CI.

Overrides the install-rhdh-catalog-source.sh download URL to refs/pull/3284/head so operator jobs pick up the readiness-wait changes (ClusterCatalog Serving / ClusterExtension Installed / Backstage CRD) instead of main.

Do not merge — revert once #3284 lands on rhdh-operator main.

Which issue(s) does this PR fix

  • Relates to OCP operator nightly CRD timeout after rhdh-operator#3047

PR acceptance criteria

  • GitHub Actions are completed and successful
  • Unit Tests are updated and passing
  • E2E Tests are updated and passing
  • Documentation is updated if necessary (requirement for new features)
  • Add a screenshot if the change is UX/UI related

How to test changes / Special notes to the reviewer

Trigger OCP operator job:

/test e2e-ocp-operator-nightly

Confirm the build log downloads the script from refs/pull/3284/head and that backstages.rhdh.redhat.com becomes available (no 300s CRD timeout).

Made with Cursor

Temporarily override install-rhdh-catalog-source.sh download to the OLM v1
readiness fix PR so OCP operator nightlies can validate the upstream change.
@openshift-ci

openshift-ci Bot commented Jul 27, 2026

Copy link
Copy Markdown

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

@zdrapela

Copy link
Copy Markdown
Member Author

/test e2e-ocp-operator-nightly

@sonarqubecloud

Copy link
Copy Markdown

@zdrapela zdrapela changed the title ci: force operator install script from rhdh-operator#3284 chore(ci): force operator install script from rhdh-operator#3284 Jul 27, 2026
@openshift-ci

openshift-ci Bot commented Jul 27, 2026

Copy link
Copy Markdown

@zdrapela: The following test 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 f4b0a61 link false /test e2e-ocp-operator-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.

@codecov

codecov Bot commented Jul 27, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 59.96%. Comparing base (6cdd57d) to head (f4b0a61).

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #5175      +/-   ##
==========================================
- Coverage   63.69%   59.96%   -3.74%     
==========================================
  Files         123      111      -12     
  Lines        2424     2198     -226     
  Branches      571      546      -25     
==========================================
- Hits         1544     1318     -226     
- Misses        878      879       +1     
+ Partials        2        1       -1     
Flag Coverage Δ
rhdh 59.96% <ø> (-3.74%) ⬇️
Components Coverage Δ
Backend plugins ∅ <ø> (∅)
Backend app 66.66% <ø> (ø)
Frontend app 58.89% <ø> (ø)
Plugin utils ∅ <ø> (∅)
Dynamic plugins utils ∅ <ø> (∅)

Continue to review full report in Codecov by Harness.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 6cdd57d...f4b0a61. 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.

@github-actions

Copy link
Copy Markdown
Contributor

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

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.

1 participant