Partial data column containers for Gloas - #17348
Open
aarshkshah1992 wants to merge 18 commits into
Open
Conversation
aarshkshah1992
force-pushed
the
feat/gloas-partial-column-container-changes
branch
from
August 13, 2026 15:59
8fc0aa2 to
86df8fd
Compare
kasey
force-pushed
the
spectest-alpha-12-compliance
branch
from
August 14, 2026 20:23
9a4debc to
3e89799
Compare
aarshkshah1992
force-pushed
the
feat/gloas-partial-column-container-changes
branch
from
August 19, 2026 07:32
86df8fd to
bab7dc6
Compare
kasey
force-pushed
the
spectest-alpha-12-compliance
branch
10 times, most recently
from
August 25, 2026 19:28
c3c534c to
ce6a7a9
Compare
kasey
force-pushed
the
spectest-alpha-12-compliance
branch
4 times, most recently
from
August 27, 2026 19:10
2f9751a to
c6049a2
Compare
The alpha.14 parent_execution_payload vectors set latest_execution_payload_bid.slot without latest_block_header.slot, so the header slot only becomes usable here with the alpha.15 vectors (consensus-specs#5554).
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Squash of the feat/gloas-partial-column-container-changes work (origin tip bab7dc6, i.e. everything on top of the spectest-alpha-12 chain 3e89799, which is already in develop via #17310) rebased onto alpha13. Merge notes vs alpha13: - proposer_payload_envelope.go: partial columns ride along the cache-first publish path from #17391 (cachedDataColumns returns them too); sidecarsFromContents returns sidecars plus partial columns. - two_node_test.go: the Gloas copy of TestTwoNodePartialColumnExchange gets the connect-early fix from #17361, same as the Fulu test. - blockchain/testing/mock.go: only RecentBlockSlotErr is added. - encoding/ssz/BUILD.bazel: stale fastssz test dep dropped by gazelle. - proto/prysm/v1alpha1/gloas.yaml: partial column types re-added and SSZ regenerated with make gen (no drift). Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
The minimal.tar.gz asset of the v1.7.0-beta.0 consensus-specs release was re-uploaded at 2026-09-04T02:24Z, after alpha13 pinned it, so the pinned sha256 no longer matches (mainnet and general are unchanged). Belongs in #17382; kept as a separate commit here so this branch builds and runs the spectests. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
broadcastDataColumnSidecars joined each column topic lazily inside the batch full-sidecar path while the partial publish for the same column ran concurrently. When the partial publish won the race on a topic the node is not subscribed to, the pubsub router had no local topic state yet, MeshPeers selected no peers, and the column was never sent. A full-node proposer hit this on its first proposal after a fork: peers ended the slot holding exactly the proposer's subscribed columns and the payload went empty network-wide. Join every column topic up front, before dispatching either publish path. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
aarshkshah1992
force-pushed
the
feat/gloas-partial-column-container-changes
branch
from
September 4, 2026 11:07
bab7dc6 to
6c384e3
Compare
|
|
||
| // Field 2: KzgProofs | ||
| dst = ssz.WriteOffset(dst, offset) | ||
| offset += len(c.KzgProofs) * 48 |
terencechain
force-pushed
the
alpha13
branch
from
September 7, 2026 16:24
7572794 to
f795409
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What
Stacks the Gloas partial data column container work on top of #17382.
Kurtosis Testing
Cherry-pick commit 9b36b8f on top of this branch which simulates private blobs/blobs missing in the EL.
Kurtosis config file:
Hey ! Ofcourse. Something like this works :