Skip to content

test: prevent duplicate metrics collector registration attempted#2957

Merged
miparnisari merged 1 commit intomainfrom
test-flaky-dup-metrics
Mar 6, 2026
Merged

test: prevent duplicate metrics collector registration attempted#2957
miparnisari merged 1 commit intomainfrom
test-flaky-dup-metrics

Conversation

@miparnisari
Copy link
Copy Markdown
Contributor

@miparnisari miparnisari commented Mar 6, 2026

This PR fixes a test flake.

The other way to prevent this flake is to do what #2829 says, but this is more work and riskier.

@github-actions github-actions bot added area/cli Affects the command line area/tooling Affects the dev or user toolchain (e.g. tests, ci, build tools) labels Mar 6, 2026
Copy link
Copy Markdown
Contributor

@tstirrat15 tstirrat15 left a comment

Choose a reason for hiding this comment

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

Yeah, this makes sense to me.

@miparnisari miparnisari marked this pull request as ready for review March 6, 2026 00:47
@miparnisari miparnisari requested a review from a team as a code owner March 6, 2026 00:47
@miparnisari miparnisari force-pushed the test-flaky-dup-metrics branch from 678b439 to 779bb08 Compare March 6, 2026 00:47
@codecov
Copy link
Copy Markdown

codecov bot commented Mar 6, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 74.46%. Comparing base (04b003c) to head (779bb08).
⚠️ Report is 1 commits behind head on main.

❌ Your project status has failed because the head coverage (74.46%) is below the target coverage (75.00%). You can increase the head coverage or adjust the target coverage.

Additional details and impacted files
@@             Coverage Diff             @@
##             main    #2957       +/-   ##
===========================================
+ Coverage   48.71%   74.46%   +25.75%     
===========================================
  Files         427      494       +67     
  Lines       54556    60626     +6070     
===========================================
+ Hits        26571    45138    +18567     
+ Misses      25263    12318    -12945     
- Partials     2722     3170      +448     

☔ View full report in Codecov by Sentry.
📢 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.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@miparnisari miparnisari enabled auto-merge (squash) March 6, 2026 03:33
@miparnisari miparnisari merged commit 0509498 into main Mar 6, 2026
75 of 78 checks passed
@miparnisari miparnisari deleted the test-flaky-dup-metrics branch March 6, 2026 03:37
@github-actions github-actions bot locked and limited conversation to collaborators Mar 6, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

area/cli Affects the command line area/tooling Affects the dev or user toolchain (e.g. tests, ci, build tools) Skip-Changelog

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants