Skip to content

Add target_gas_limit to gloas PayloadAttributes#16792

Merged
terencechain merged 2 commits into
developfrom
payload-attrs-target-gas-limit
May 19, 2026
Merged

Add target_gas_limit to gloas PayloadAttributes#16792
terencechain merged 2 commits into
developfrom
payload-attrs-target-gas-limit

Conversation

@terencechain

Copy link
Copy Markdown
Collaborator
  • Add target_gas_limit field to PayloadAttributesV4 to align engine_forkchoiceUpdatedV4.
  • Thread val.GasLimit from TrackedValidator through every PayloadAttributesV4 construction site (proposer RPC, blockchain FCU, late-payload FCU, payload_attributes SSE event)
  • Implements consensus-specs#5235 and execution-apis#796.

Threads val.GasLimit through PayloadAttributesV4 so the EL receives
target_gas_limit on engine_forkchoiceUpdatedV4 per consensus-specs#5235
and execution-apis#796.
@terencechain terencechain force-pushed the payload-attrs-target-gas-limit branch from 9675b73 to 6b0a5cf Compare May 19, 2026 13:48
return payloadAttributesGloas(uint64(t.Unix()), prevRando, val.FeeRecipient[:], headRoot, withdrawals, slot)
targetGasLimit := val.GasLimit
if targetGasLimit == 0 {
targetGasLimit = params.BeaconConfig().DefaultBuilderGasLimit

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.

I created a helper for these checks will fix it on my pr when this merges

@terencechain terencechain enabled auto-merge May 19, 2026 14:20
@terencechain terencechain added this pull request to the merge queue May 19, 2026
Merged via the queue into develop with commit f5066f1 May 19, 2026
23 checks passed
@terencechain terencechain deleted the payload-attrs-target-gas-limit branch May 19, 2026 15:23
@github-project-automation github-project-automation Bot moved this from Unassigned to Done in Gloas May 19, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants