Skip to content

[system test] [perf] Remove redundancy on setting finalizers within T…#11977

Merged
see-quick merged 1 commit intostrimzi:mainfrom
see-quick:update-perf-scala
Oct 6, 2025
Merged

[system test] [perf] Remove redundancy on setting finalizers within T…#11977
see-quick merged 1 commit intostrimzi:mainfrom
see-quick:update-perf-scala

Conversation

@see-quick
Copy link
Copy Markdown
Member

@see-quick see-quick commented Oct 6, 2025

…O scallability perf test

Type of change

  • Bugfix
  • Refactoring
  • Documentation

Description

This PR removes a redundancy in the test cases. Currently, we are setting STRIMZI_USE_FINALIZERS in our templates, so there is no need to perform this operation twice.

Checklist

  • Write tests
  • Make sure all tests pass

…O scallability perf test

Signed-off-by: see-quick <maros.orsak159@gmail.com>
@see-quick see-quick added this to the 0.49.0 milestone Oct 6, 2025
@see-quick see-quick requested a review from a team October 6, 2025 11:12
@see-quick see-quick self-assigned this Oct 6, 2025
@codecov
Copy link
Copy Markdown

codecov bot commented Oct 6, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 74.66%. Comparing base (66ccc4e) to head (8075444).
⚠️ Report is 5 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff            @@
##               main   #11977   +/-   ##
=========================================
  Coverage     74.66%   74.66%           
- Complexity     6399     6401    +2     
=========================================
  Files           342      342           
  Lines         24228    24228           
  Branches       3190     3190           
=========================================
+ Hits          18090    18091    +1     
  Misses         4866     4866           
+ Partials       1272     1271    -1     

see 1 file with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@see-quick see-quick merged commit 9e775e7 into strimzi:main Oct 6, 2025
22 checks passed
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.

2 participants