Skip to content

Partial data column containers for Gloas - #17348

Open
aarshkshah1992 wants to merge 51 commits into
spectest-alpha-12-compliancefrom
feat/gloas-partial-column-container-changes
Open

Partial data column containers for Gloas #17348
aarshkshah1992 wants to merge 51 commits into
spectest-alpha-12-compliancefrom
feat/gloas-partial-column-container-changes

Conversation

@aarshkshah1992

@aarshkshah1992 aarshkshah1992 commented Aug 13, 2026

Copy link
Copy Markdown
Contributor

What

Stacks the Gloas partial data column container work on top of #17310 (spectest-alpha-12-compliance).

Bundles the in-progress Gloas partial-column feature with the broadcaster
incoming-message changes (group-id parse, fork-aware decode, Gloas drop) for
kurtosis devnet testing.
Adds ColumnCallbacks.ValidatePartialColumnGroupID so the broadcaster downscores
peers whose Gloas group id slot does not match the slot of the seen block at the
group's beacon block root ([REJECT]); unseen blocks are ignored ([IGNORE]).
Conflicts resolved:
- execution/testing/mock_engine_client.go: union of struct fields
  (our PartialColumnsEnabledForSlotResult + develop's FetchedAttributes).
- rpc/eth/beacon/handlers_gloas.go: took develop's delegation refactor
  (#16960/#16943); our inline REST blob->column->broadcast logic is
  superseded by the v1alpha1 publish path.
- rpc/prysm/v1alpha1/validator/proposer_payload_envelope.go: kept develop's
  stateless+stateful sidecar resolution, preserved our partial columns in the
  stateful (cache) path, and fixed storeExecutionPayloadEnvelope returns for
  develop's new (*ExecutionPayloadEnvelope, error) signature.

Other merge resolutions:
- sync: adopted develop's Gloas data-column reconstruction (#17053) by
  removing our now-obsolete IsGloas reconstruction guards in
  processDataColumnSidecarsFromReconstruction and verifiedRODataColumnSubscriber.
- spectest engineMock: implemented PartialColumnsEnabledForSlot to satisfy the
  EngineCaller interface (our addition) against develop's new mock.
…com:prysmaticlabs/prysm into feat/gloas-partial-column-container-changes
…ner-changes' into feat/gloas-partial-column-container-changes
@aarshkshah1992 aarshkshah1992 changed the title Partial data column containers for Gloas (stacked on spectest-alpha-12-compliance) Partial data column containers for Gloas Aug 13, 2026
@aarshkshah1992
aarshkshah1992 force-pushed the feat/gloas-partial-column-container-changes branch from 8fc0aa2 to 86df8fd Compare August 13, 2026 15:59
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.

1 participant