Skip to content

Conversation

@dashrews78
Copy link
Contributor

@dashrews78 dashrews78 commented Jan 9, 2026

Description

Adds a benchmark to remove by Pod.

User-facing documentation

Testing and quality

  • the change is production ready: the change is GA, or otherwise the functionality is gated by a feature flag
  • CI results are inspected

Automated testing

  • added unit tests
  • added e2e tests
  • added regression tests
  • added compatibility tests
  • modified existing tests

How I validated my change

Simply adds a benchmark test.

@openshift-ci
Copy link

openshift-ci bot commented Jan 9, 2026

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

@dashrews78
Copy link
Contributor Author

dashrews78 commented Jan 9, 2026

This change is part of the following stack:

Change managed by git-spice.

@rhacs-bot
Copy link
Contributor

rhacs-bot commented Jan 9, 2026

Images are ready for the commit at 9e207f7.

To use with deploy scripts, first export MAIN_IMAGE_TAG=4.10.x-753-g9e207f765e.

@codecov
Copy link

codecov bot commented Jan 9, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 48.93%. Comparing base (0041570) to head (9e207f7).

Additional details and impacted files
@@           Coverage Diff           @@
##           master   #18424   +/-   ##
=======================================
  Coverage   48.93%   48.93%           
=======================================
  Files        2631     2631           
  Lines      197964   197964           
=======================================
+ Hits        96872    96877    +5     
+ Misses      93703    93701    -2     
+ Partials     7389     7386    -3     
Flag Coverage Δ
go-unit-tests 48.93% <ø> (+<0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

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

@dashrews78 dashrews78 force-pushed the dashrews/plop-bench-32081 branch from 417611f to fe43010 Compare January 9, 2026 18:44
@dashrews78 dashrews78 marked this pull request as ready for review January 12, 2026 10:55
@dashrews78 dashrews78 force-pushed the dashrews/plop-bench-32081 branch from fe087f1 to d227d54 Compare January 12, 2026 10:55
@dashrews78 dashrews78 force-pushed the dashrews/plop-bench-32081 branch from d227d54 to 9e207f7 Compare January 12, 2026 15:24
@dashrews78 dashrews78 merged commit 382defc into master Jan 12, 2026
100 of 102 checks passed
@dashrews78 dashrews78 deleted the dashrews/plop-bench-32081 branch January 12, 2026 17:40
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.

3 participants