Skip to content

Set parent payload availability bit for attestation packing and the attestation simulator - #9828

Merged
mergify[bot] merged 4 commits into
unstablefrom
gloas-deferred-payload-availability-fixes
Aug 25, 2026
Merged

Set parent payload availability bit for attestation packing and the attestation simulator#9828
mergify[bot] merged 4 commits into
unstablefrom
gloas-deferred-payload-availability-fixes

Conversation

@eserilev

Copy link
Copy Markdown
Member

Payload processing is deferred to the next block, so a state from before process_parent_execution_payload doesn't have the parent's availability bit set.

This PR impls the following fixes:

  • During attestation packing, set the parent's availability bit if the block builds on the parent's payload
  • In the attestation simulator use the attested block's slot as parent_slot.

Only non same slot attestations read the availability bit, so these fixes are only relevant for attestations made on a skipped slot.

Depends on #9764

@eserilev
eserilev force-pushed the gloas-deferred-payload-availability-fixes branch from 510c5d6 to 6d1e258 Compare August 17, 2026 10:30
@eserilev eserilev added ready-for-review The code is ready for review gloas labels Aug 17, 2026
@eserilev
eserilev force-pushed the gloas-deferred-payload-availability-fixes branch from 6d1e258 to 2486be8 Compare August 17, 2026 12:33
@pawanjay176
pawanjay176 force-pushed the gloas-deferred-payload-availability-fixes branch from 2486be8 to 939fb34 Compare August 18, 2026 02:06
@mergify

mergify Bot commented Aug 18, 2026

Copy link
Copy Markdown

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. ready-for-review The code is ready for review and removed ready-for-review The code is ready for review waiting-on-author The reviewer has suggested changes and awaits thier implementation. labels Aug 18, 2026
@pawanjay176
pawanjay176 force-pushed the gloas-deferred-payload-availability-fixes branch from 939fb34 to cb4017f Compare August 18, 2026 18:06
@mergify

mergify Bot commented Aug 18, 2026

Copy link
Copy Markdown

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 Aug 18, 2026
@michaelsproul
michaelsproul changed the base branch from ef-tests-alpha-13 to unstable August 25, 2026 04:45
@michaelsproul
michaelsproul requested a review from jxs as a code owner August 25, 2026 04:45
@michaelsproul
michaelsproul force-pushed the gloas-deferred-payload-availability-fixes branch from cb4017f to 742c672 Compare August 25, 2026 04:48
@mergify mergify Bot 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 Aug 25, 2026
@mergify

mergify Bot commented Aug 25, 2026

Copy link
Copy Markdown

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 Aug 25, 2026
@mergify

mergify Bot commented Aug 25, 2026

Copy link
Copy Markdown

Queued — the merge queue status continues in this comment ↓.

@eserilev eserilev added the ready-for-merge This PR is ready to merge. label Aug 25, 2026
@mergify mergify Bot added the queued label Aug 25, 2026
@mergify

mergify Bot commented Aug 25, 2026

Copy link
Copy Markdown

Merge Queue Status

This pull request spent 29 minutes 45 seconds in the queue, including 27 minutes 21 seconds running CI.

Required conditions to merge

@mergify
mergify Bot merged commit f50f8c7 into unstable Aug 25, 2026
37 of 38 checks passed
@mergify mergify Bot removed the queued label Aug 25, 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. waiting-on-author The reviewer has suggested changes and awaits thier implementation.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants