Skip to content

fix: address a name collision risk in the MCS handling logic - #397

Open
Chen Yu (michaelawyu) wants to merge 1 commit into
Azure:mainfrom
michaelawyu:fix/name-collision
Open

fix: address a name collision risk in the MCS handling logic#397
Chen Yu (michaelawyu) wants to merge 1 commit into
Azure:mainfrom
michaelawyu:fix/name-collision

Fixed a name collision issue

4898673
Select commit
Loading
Failed to load commit list.
Codecov / codecov/project succeeded Aug 4, 2026 in 1s

67.53% (+0.42%) compared to 556170b

View this Pull Request on Codecov

67.53% (+0.42%) compared to 556170b

Details

Codecov Report

❌ Patch coverage is 70.50360% with 41 lines in your changes missing coverage. Please review.
✅ Project coverage is 67.53%. Comparing base (556170b) to head (4898673).

Files with missing lines Patch % Lines
pkg/controllers/multiclusterservice/controller.go 61.11% 29 Missing and 6 partials ⚠️
pkg/controllers/multiclusterservice/uniquename.go 87.75% 4 Missing and 2 partials ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #397      +/-   ##
==========================================
+ Coverage   67.11%   67.53%   +0.42%     
==========================================
  Files          33       34       +1     
  Lines        3783     5012    +1229     
==========================================
+ Hits         2539     3385     +846     
- Misses       1070     1444     +374     
- Partials      174      183       +9     

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

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