Skip to content

fix(ci): ingress resource visibility check in topology tests#4774

Merged
openshift-merge-bot[bot] merged 1 commit into
redhat-developer:mainfrom
imykhno:fix/ci/topology-tab-test
May 11, 2026
Merged

fix(ci): ingress resource visibility check in topology tests#4774
openshift-merge-bot[bot] merged 1 commit into
redhat-developer:mainfrom
imykhno:fix/ci/topology-tab-test

Conversation

@imykhno
Copy link
Copy Markdown
Contributor

@imykhno imykhno commented May 7, 2026

Description

The fix for Verify pods visibility in the Topology tab e2e tests. The error occurred during the step to verify the visibility of the code block. The test checked for the presence of /apiVersion:|kind:|metadata:/, but these were missing. Please see the attached screenshot for the actual data displayed in the code block.

Screenshot 2026-05-11 at 9 45 30

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

@openshift-ci openshift-ci Bot requested review from kadel and sanketpathak May 7, 2026 08:12
@codecov
Copy link
Copy Markdown

codecov Bot commented May 7, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 69.49%. Comparing base (e35bc61) to head (9841035).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@             Coverage Diff             @@
##             main    #4774       +/-   ##
===========================================
+ Coverage   40.88%   69.49%   +28.60%     
===========================================
  Files         119      109       -10     
  Lines        2228     4710     +2482     
  Branches      563      513       -50     
===========================================
+ Hits          911     3273     +2362     
- Misses       1312     1437      +125     
+ Partials        5        0        -5     
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 e35bc61...9841035. 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

github-actions Bot commented May 7, 2026

The container image build workflow finished with status: cancelled.

Signed-off-by: Ihor Mykhno <imykhno@redhat.com>
@imykhno imykhno force-pushed the fix/ci/topology-tab-test branch from 1d0fbd4 to 9841035 Compare May 11, 2026 06:58
@sonarqubecloud
Copy link
Copy Markdown

@github-actions
Copy link
Copy Markdown
Contributor

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

Copy link
Copy Markdown
Contributor

@HusneShabbir HusneShabbir left a comment

Choose a reason for hiding this comment

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

/lgtm
Thanks for the fix @imykhno.

@openshift-ci openshift-ci Bot added the lgtm label May 11, 2026
@openshift-merge-bot openshift-merge-bot Bot merged commit 2a77e36 into redhat-developer:main May 11, 2026
22 checks passed
@imykhno imykhno deleted the fix/ci/topology-tab-test branch May 11, 2026 08:04
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.

2 participants