Skip to content

Remove next epoch lookahead from PTC duties - #586

Merged
nflaig merged 1 commit into
masterfrom
nflaig/remove-ptc-lookahead
Mar 17, 2026
Merged

Remove next epoch lookahead from PTC duties#586
nflaig merged 1 commit into
masterfrom
nflaig/remove-ptc-lookahead

Conversation

@nflaig

@nflaig nflaig commented Mar 11, 2026

Copy link
Copy Markdown
Member

The PTC assignments are not stable for the next epoch so we should not be required to serve these duties via the api as we will need to re-compute them for the current epoch anyways. This PR effectively removes the requirement for clients to serve next epoch PTC duties.

Related ethereum/consensus-specs#4992

@rolfyone rolfyone left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

github-merge-queue Bot pushed a commit to OffchainLabs/prysm that referenced this pull request Mar 12, 2026
**What does this PR do? Why is it needed?**

Implemented the spec change:
ethereum/beacon-APIs#586

(Removed next epoch lookahead from PTC duties)
syjn99 pushed a commit to syjn99/prysm that referenced this pull request Mar 13, 2026
**What does this PR do? Why is it needed?**

Implemented the spec change:
ethereum/beacon-APIs#586

(Removed next epoch lookahead from PTC duties)
rolfyone added a commit to rolfyone/teku that referenced this pull request Mar 16, 2026
ptc duties can no longer lookahead, with PR ethereum/beacon-APIs#586 - updated description.
@nflaig
nflaig merged commit 6463cee into master Mar 17, 2026
3 checks passed
@nflaig
nflaig deleted the nflaig/remove-ptc-lookahead branch March 17, 2026 10:07
nflaig added a commit that referenced this pull request Mar 23, 2026
nflaig added a commit that referenced this pull request Mar 25, 2026
* Revert "Remove next epoch lookahead from PTC duties (#586)"

This reverts commit 6463cee.

* Apply suggestion from @nflaig

* update epoch param description

---------

Co-authored-by: Paul Harris <paul.harris@consensys.net>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Gloas api's needed in Gloas fork.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants