Skip to content

Comments

Gloas attestation verification #8705

Merged
mergify[bot] merged 4 commits intosigp:unstablefrom
eserilev:gloas-attestation-verification
Feb 13, 2026
Merged

Gloas attestation verification #8705
mergify[bot] merged 4 commits intosigp:unstablefrom
eserilev:gloas-attestation-verification

Conversation

@eserilev
Copy link
Member

Issue Addressed

https://github.com/ethereum/consensus-specs/blob/master/specs/gloas/p2p-interface.md#attestation-subnets

Implements attestation verification logic for Gloas and adds a few gloas related tests. Note that a few of these tests rely on gloas test harness block production which hasn't been built out yet. So for now those tests are ignored.

@eserilev eserilev requested a review from jxs as a code owner January 27, 2026 19:54
@eserilev eserilev added the gloas label Jan 27, 2026
@eserilev
Copy link
Member Author

@jimmygchen sorry i just realized you may have been assigned to this. accidentally jumped the gun here, my bad

@eserilev eserilev mentioned this pull request Jan 27, 2026
24 tasks
@eserilev eserilev added the ready-for-review The code is ready for review label Jan 31, 2026
/// This test only runs when `FORK_NAME=gloas` is set with `fork_from_env` feature.
// TODO(EIP-7732): Enable this test once gloas block production works in test harness.
// `state.latest_execution_payload_header()` not available in Gloas.
#[ignore]
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Strange to merge ignored tests

@mergify
Copy link

mergify bot commented Feb 13, 2026

Some required checks have failed. Could you please take a look @eserilev? 🙏

@mergify mergify bot added waiting-on-author The reviewer has suggested changes and awaits thier implementation. and removed ready-for-review The code is ready for review labels Feb 13, 2026
@eserilev eserilev added ready-for-review The code is ready for review and removed waiting-on-author The reviewer has suggested changes and awaits thier implementation. labels Feb 13, 2026
Copy link
Collaborator

@dapplion dapplion left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Gossip logic aligns with the spec, let's review the tests they are enabled

Copy link
Member

@pawanjay176 pawanjay176 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@pawanjay176 pawanjay176 added ready-for-merge This PR is ready to merge. and removed ready-for-review The code is ready for review labels Feb 13, 2026
@mergify mergify bot added the queued label Feb 13, 2026
@mergify
Copy link

mergify bot commented Feb 13, 2026

Merge Queue Status

Rule: default


This pull request spent 31 minutes 23 seconds in the queue, including 29 minutes 2 seconds running CI.

Required conditions to merge
  • check-success=local-testnet-success
  • check-success=test-suite-success

mergify bot added a commit that referenced this pull request Feb 13, 2026
@mergify mergify bot merged commit 68ad975 into sigp:unstable Feb 13, 2026
36 checks passed
@mergify mergify bot removed the queued label Feb 13, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

gloas ready-for-merge This PR is ready to merge.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants