Skip to content

feat: move blobs from consensus txs to p2p so they can be pruned properly - #3149

Open
fridrik01 wants to merge 1 commit into
mainfrom
blobreactor-v2
Open

feat: move blobs from consensus txs to p2p so they can be pruned properly#3149
fridrik01 wants to merge 1 commit into
mainfrom
blobreactor-v2

feat: move blobs from consensus txs to p2p so they can be pruned prop…

be843f8
Select commit
Loading
Failed to load commit list.
Codecov / codecov/project succeeded Aug 12, 2026 in 0s

64.26% (target 0.00%)

View this Pull Request on Codecov

64.26% (target 0.00%)

Details

Codecov Report

❌ Patch coverage is 72.08528% with 419 lines in your changes missing coverage. Please review.
✅ Project coverage is 64.26%. Comparing base (2059e2a) to head (be843f8).

Files with missing lines Patch % Lines
da/blobreactor/reactor.go 69.28% 53 Missing and 29 partials ⚠️
da/blobreactor/requests.go 71.88% 40 Missing and 21 partials ⚠️
beacon/blockchain/blob_queue.go 63.15% 29 Missing and 20 partials ⚠️
beacon/blockchain/blob_fetcher.go 74.82% 25 Missing and 11 partials ⚠️
da/blobreactor/push.go 80.35% 24 Missing and 9 partials ⚠️
da/blobreactor/messages.go 85.95% 14 Missing and 3 partials ⚠️
node-api/handlers/beacon/blobs.go 15.00% 13 Missing and 4 partials ⚠️
beacon/blockchain/blob_retrieval.go 75.00% 14 Missing and 1 partial ⚠️
beacon/blockchain/process_proposal.go 48.27% 12 Missing and 3 partials ⚠️
da/blobreactor/serve.go 76.27% 8 Missing and 6 partials ⚠️
... and 16 more
Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #3149      +/-   ##
==========================================
+ Coverage   63.11%   64.26%   +1.15%     
==========================================
  Files         372      385      +13     
  Lines       15549    16965    +1416     
==========================================
+ Hits         9814    10903    +1089     
- Misses       4775     4993     +218     
- Partials      960     1069     +109
Files with missing lines Coverage Δ
beacon/blockchain/common.go 70.58% <100.00%> (+28.28%) ⬆️
beacon/blockchain/service.go 84.00% <100.00%> (+3.51%) ⬆️
chain/spec.go 75.47% <100.00%> (+3.39%) ⬆️
config/config.go 85.00% <100.00%> (+0.38%) ⬆️
config/spec/devnet.go 100.00% <100.00%> (ø)
config/spec/mainnet.go 100.00% <100.00%> (ø)
config/spec/testnet.go 100.00% <100.00%> (ø)
consensus/cometbft/service/blobconsensus/config.go 100.00% <100.00%> (ø)
consensus/cometbft/service/cache/cache.go 88.23% <ø> (ø)
consensus/cometbft/service/process_proposal.go 85.36% <100.00%> (+15.36%) ⬆️
... and 33 more

... and 4 files with indirect coverage changes

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