Skip to content

fix(fab): remove fab plugin completely#4889

Open
ciiay wants to merge 2 commits into
redhat-developer:mainfrom
ciiay:rhidp-13680-remove-fab-plugin
Open

fix(fab): remove fab plugin completely#4889
ciiay wants to merge 2 commits into
redhat-developer:mainfrom
ciiay:rhidp-13680-remove-fab-plugin

Conversation

@ciiay
Copy link
Copy Markdown
Member

@ciiay ciiay commented May 26, 2026

Description

For RHIDP-13680

PR acceptance criteria

Please make sure that the following steps are complete:

  • 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

@openshift-ci openshift-ci Bot requested review from rm3l and teknaS47 May 26, 2026 18:09
@codecov
Copy link
Copy Markdown

codecov Bot commented May 26, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 22.07%. Comparing base (9266e28) to head (af22fb2).
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@             Coverage Diff             @@
##             main    #4889       +/-   ##
===========================================
- Coverage   55.82%   22.07%   -33.76%     
===========================================
  Files         121      120        -1     
  Lines        2350     2025      -325     
  Branches      562      539       -23     
===========================================
- Hits         1312      447      -865     
- Misses       1032     1573      +541     
+ Partials        6        5        -1     
Flag Coverage Δ
rhdh 22.07% <ø> (-33.76%) ⬇️

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 9266e28...af22fb2. 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:

@ciiay
Copy link
Copy Markdown
Member Author

ciiay commented May 27, 2026

/test e2e-ocp-helm

@ciiay
Copy link
Copy Markdown
Member Author

ciiay commented May 27, 2026

/retest

@ciiay
Copy link
Copy Markdown
Member Author

ciiay commented May 28, 2026

/test e2e-ocp-helm

1 similar comment
@ciiay
Copy link
Copy Markdown
Member Author

ciiay commented May 28, 2026

/test e2e-ocp-helm

@sanketpathak
Copy link
Copy Markdown
Contributor

sanketpathak commented May 28, 2026

The 1.10 tag might be causing the failure. CI still uses catalog-index 1.10, which was built before that removal

catalogIndex:
image:
registry: quay.io
repository: rhdh/plugin-catalog-index
tag: "1.10"

Copy link
Copy Markdown
Member

@its-mitesh-kumar its-mitesh-kumar left a comment

Choose a reason for hiding this comment

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

/lgtm

Copy link
Copy Markdown
Member

@teknaS47 teknaS47 left a comment

Choose a reason for hiding this comment

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

/lgtm

ciiay added 2 commits June 3, 2026 16:54
Signed-off-by: Yi Cai <yicai@redhat.com>
Signed-off-by: Yi Cai <yicai@redhat.com>
@ciiay ciiay force-pushed the rhidp-13680-remove-fab-plugin branch from cf67943 to af22fb2 Compare June 3, 2026 21:14
@openshift-ci openshift-ci Bot removed the lgtm label Jun 3, 2026
@openshift-ci
Copy link
Copy Markdown

openshift-ci Bot commented Jun 3, 2026

New changes are detected. LGTM label has been removed.

@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented Jun 3, 2026

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jun 3, 2026

The container image build workflow finished with status: failure.

@openshift-ci
Copy link
Copy Markdown

openshift-ci Bot commented Jun 3, 2026

@ciiay: 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-helm af22fb2 link true /test e2e-ocp-helm

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.

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.

4 participants