[chrisdutz]

Description

SonarCloud's quality gate currently fails for every pull request with new_reliability_rating = 3 (C, the limit is 1). The rating comes from two bugs in recently merged code, not from the pull requests themselves (e.g. #19104, #19107, #19108):

File	Rule	Fix
PipeTaskInfo (#19056)	pipeMetaExceptionPersistencePending |= hasException on a volatile boolean is not atomic	AtomicBoolean, set only when there is an exception
PipeResourceMetrics (#19106)	a Map published through a volatile field is not thread-safe	AtomicReference holding an unmodifiable map; bindTo/unbindFrom replace it as a whole

Behavior is unchanged: the flag is still set on a recorded exception and cleared after persistence, and the reject counters are still looked up by reason with a no-op default.

Tests

PipeResourceMetricsTest, PipeMemoryManagerTest, PipeMemoryManagerResizeTest, PipeTaskInfoAutoRestartTest, PipeTaskInfoConsensusPipeTest, PipeMetaSyncProcedureTest, AbstractOperatePipeProcedureV2Test: 67 tests pass; spotless:check passes.

🤖 Generated with Claude Code

[sonarqubecloud]

Quality Gate passed

Issues
427 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
2.7% Duplication on New Code

See analysis details on SonarQube Cloud

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 48.23%. Comparing base (814b770) to head (5df911a).

Additional details and impacted files

@@             Coverage Diff              @@
##             master   #19109      +/-   ##
============================================
- Coverage     48.23%   48.23%   -0.01%     
  Complexity      625      625              
============================================
  Files          5572     5572              
  Lines        411672   411675       +3     
  Branches      54225    54226       +1     
============================================
- Hits         198558   198557       -1     
- Misses       213114   213118       +4     

☔ 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.

[Caideyipi]

Reviewed 5df911a; no actionable issues found. Checked the exception-persistence call paths and the metrics bind/unbind lifecycle. Setting the flag only on failures preserves its pending-state semantics, and the map is fully populated before publication and never modified afterward, with the no-op fallback preserved. All 42 CI checks on this commit passed. No additional tests were run for this review.