Skip to content

Conversation

@shane-moore
Copy link

Changes

Super small PR to add off protocol payments field to ExecutionPayloadBid per spec PR

Reasoning for now field discussed in discord. TLDR is that adding the off protocol payment field in protocol enables verifying that the off protocol builder actually pays the proposer

Additionally, we add the prev_randao field to ExecutionPayloadBid per spec PR

Additionally, we set the latest_execution_payload_bid's block_hash field to latest_execution_payload_header.block_hash during upgrade_state_to_gloas per spec PR

@ethDreamer
@eserilev

Note that these are the same commits from PR #8466, which was closed since the PR was pointing at gloas-containers, which has now been merged to unstable

@chong-he chong-he added gloas ready-for-review The code is ready for review labels Dec 17, 2025
Copy link
Member

@jimmygchen jimmygchen left a comment

Choose a reason for hiding this comment

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

LGTM, thanks for linking the spec PRs!

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

mergify bot commented Jan 6, 2026

Merge Queue Status

✅ The pull request has been merged at c7375e3

This pull request spent 39 minutes 34 seconds in the queue, including 37 minutes 51 seconds running CI.
The checks were run on draft #8627.

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

mergify bot added a commit that referenced this pull request Jan 6, 2026
@mergify mergify bot merged commit 2fe5940 into sigp:unstable Jan 6, 2026
36 checks passed
@mergify mergify bot removed the queued label Jan 6, 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.

4 participants