Skip to content

feat: stackerdb bandwidth config#7358

Open
federico-stacks wants to merge 4 commits into
stacks-network:developfrom
federico-stacks:feat/stackerdb-bandwith-config
Open

feat: stackerdb bandwidth config#7358
federico-stacks wants to merge 4 commits into
stacks-network:developfrom
federico-stacks:feat/stackerdb-bandwith-config

Conversation

@federico-stacks

@federico-stacks federico-stacks commented Jun 26, 2026

Copy link
Copy Markdown
Contributor

Description

This PR improve stackerdb messages management along this lines:

  • Exposed connection options max_stackerdb_push_bandwidth, max_nakamoto_block_push_bandwidth, max_transaction_push_bandwidth as TOML configuration to set bandwidth cap on message reception
  • Set connection options max_stackerdb_push_bandwidth default to 4 MB/seconds.
  • Enforced StackerDB message chunk-size check against replica configuration instead of statically against STACKERDB_MAX_CHUNK_SIZE
  • Improved StackerDB configuration creation for miner to set chunk-size to STACKERDB_MAX_CHUNK_SIZE instead of MAX_PAYLOAD_LEN

Applicable issues

  • fixes #

Additional info (benefits, drawbacks, caveats)

Checklist

  • Test coverage for new or modified code paths
  • For new Clarity features or consensus changes, add property tests (see docs/property-testing.md)
  • Changelog fragment(s) or "no changelog" label added (see changelog.d/README.md)
  • Required documentation changes (e.g., rpc/openapi.yaml for RPC endpoints, event-dispatcher.md for new events)
  • New clarity functions have corresponding PR in clarity-benchmarking repo

@coveralls

coveralls commented Jun 26, 2026

Copy link
Copy Markdown

Coverage Report for CI Build 28231693161

Warning

Build has drifted: This PR's base is out of sync with its target branch, so coverage data may include unrelated changes.
Quick fix: rebase this PR. Learn more →

Coverage increased (+0.3%) to 86.029%

Details

  • Coverage increased (+0.3%) from the base build.
  • Patch coverage: 54 of 54 lines across 6 files are fully covered (100%).
  • 8903 coverage regressions across 129 files.

Uncovered Changes

No uncovered changes found.

Coverage Regressions

8903 previously-covered lines in 129 files lost coverage.

Top 10 Files by Coverage Loss Lines Losing Coverage Coverage
stackslib/src/chainstate/burn/db/sortdb.rs 504 90.2%
stackslib/src/chainstate/stacks/db/blocks.rs 502 89.93%
stackslib/src/chainstate/nakamoto/mod.rs 403 84.62%
stackslib/src/config/mod.rs 376 69.35%
stackslib/src/net/mod.rs 310 78.12%
stackslib/src/chainstate/stacks/index/storage.rs 277 82.41%
clarity/src/vm/database/clarity_db.rs 268 82.11%
stackslib/src/chainstate/stacks/miner.rs 259 83.06%
stackslib/src/chainstate/stacks/db/transactions.rs 239 97.15%
stackslib/src/net/inv/epoch2x.rs 229 79.03%

Coverage Stats

Coverage Status
Relevant Lines: 226471
Covered Lines: 194830
Line Coverage: 86.03%
Coverage Strength: 18707876.51 hits per line

💛 - Coveralls

@federico-stacks federico-stacks force-pushed the feat/stackerdb-bandwith-config branch from 4aa4610 to 9a66fa2 Compare June 26, 2026 09:08
@federico-stacks federico-stacks changed the title feat: stackerdb bandwith config feat: stackerdb bandwidth config Jun 26, 2026
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.

4 participants