Skip to content

Chore: fix flake in non_blocking_minority_configured_to_favour_...#7215

Open
aaronb-stacks wants to merge 3 commits into
stacks-network:developfrom
aaronb-stacks:test/flake-tenure_extend
Open

Chore: fix flake in non_blocking_minority_configured_to_favour_...#7215
aaronb-stacks wants to merge 3 commits into
stacks-network:developfrom
aaronb-stacks:test/flake-tenure_extend

Conversation

@aaronb-stacks

Copy link
Copy Markdown
Contributor

This test failed 3x in a row for me on a PR recently, so I tried to dig into the flakiness a little bit.

There was a race between the stale Pre-Miner-2-winning-sortition global state and the expected Miner-2-won-sortition-and-timed-out state. This change removes that race by clearing the observer.

@coveralls

coveralls commented May 19, 2026

Copy link
Copy Markdown

Coverage Report for CI Build 27029018508

Coverage increased (+0.1%) to 85.852%

Details

  • Coverage increased (+0.1%) from the base build.
  • Patch coverage: No coverable lines changed in this PR.
  • 7235 coverage regressions across 125 files.

Uncovered Changes

No uncovered changes found.

Coverage Regressions

7235 previously-covered lines in 125 files lost coverage.

Top 10 Files by Coverage Loss Lines Losing Coverage Coverage
stackslib/src/config/mod.rs 376 68.96%
stackslib/src/net/mod.rs 310 78.12%
stackslib/src/chainstate/stacks/index/storage.rs 286 82.28%
clarity/src/vm/database/clarity_db.rs 263 82.36%
stackslib/src/chainstate/stacks/miner.rs 253 83.4%
stackslib/src/chainstate/stacks/db/transactions.rs 222 97.16%
stackslib/src/net/inv/epoch2x.rs 215 79.84%
stackslib/src/chainstate/nakamoto/mod.rs 203 84.75%
stackslib/src/net/chat.rs 200 93.03%
stackslib/src/chainstate/stacks/db/mod.rs 195 86.33%

Coverage Stats

Coverage Status
Relevant Lines: 224161
Covered Lines: 192446
Line Coverage: 85.85%
Coverage Strength: 18867631.48 hits per line

💛 - Coveralls

@CLAassistant

CLAassistant commented May 20, 2026

Copy link
Copy Markdown

CLA assistant check
All committers have signed the CLA.

@aaronb-stacks aaronb-stacks self-assigned this May 21, 2026
…g_miner

There was a race between the stale Pre-Miner-2-winning-sortition global state and the expected
Miner-2-won-sortition-and-timed-out state. This change removes that race by clearing the observer.
@aaronb-stacks aaronb-stacks force-pushed the test/flake-tenure_extend branch from 9487bf8 to fa22f58 Compare May 28, 2026 19:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants