Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[Merge] Update to interop Engine API #4346

Closed
17 tasks done
ajsutton opened this issue Sep 13, 2021 · 1 comment
Closed
17 tasks done

[Merge] Update to interop Engine API #4346

ajsutton opened this issue Sep 13, 2021 · 1 comment
Assignees
@tbenr

Comments

@ajsutton
Copy link
Contributor

ajsutton commented Sep 13, 2021 •
edited by tbenr
Loading

Description

Update the engine API calls to use the latest standard (at least those identified for the interop subset).

beacon state

  • ExecutionPayload and ExecutionPayloadHeader

core

  • engine_preparePayload

    • client implementation
    • api usage

  • engine_getPayload

    • client implementation
    • api usage

  • engine_executePayload

    • client implementation
    • api usage

  • engine_consensusValidated

    • client implementation
    • api usage

  • engine_forkchoiceUpdated

    • client implementation
    • api usage

endpoint

  • introduce a new executionEngineEndpoint param (optional - fallback to eth1Endpoints.get(0)) to route there the engine_* APIs only. We will continue using eth1Endpoints.get(0) for retrieving the powHead but we may start using the new endpoint for all if it will be standardized)
@ajsutton ajsutton mentioned this issue Sep 13, 2021
Closed
20 tasks
tbenr added a commit to tbenr/teku that referenced this issue Sep 15, 2021
@tbenr
starts the work for engine_preparePayload API call (Consensys#4346)
70363d5 
introduces Duty preparation
uses Duty preparation in BlockProductionDuty to handle the call to the execution engine to prepare the block
starts extending other required classes to be able to call the new method
tbenr added a commit to tbenr/teku that referenced this issue Sep 15, 2021
@tbenr
requestId -> payloadId and toVersionMerge() (Consensys#4346)
1370791
tbenr added a commit to tbenr/teku that referenced this issue Sep 16, 2021
@tbenr
ensure BlockProductionDuty preparation, start integrating getPayload …
6d50c6c 
…by bringing executionPayloadId to block production (Consensys#4346)
tbenr added a commit to tbenr/teku that referenced this issue Sep 16, 2021
@tbenr
fixing unit tests (Consensys#4346)
47b5a9d
tbenr added a commit to tbenr/teku that referenced this issue Sep 17, 2021
@tbenr
use slot as payloadId, better prepare handling (Consensys#4346)
be2a5b1
tbenr added a commit to tbenr/teku that referenced this issue Sep 17, 2021
@tbenr
fixes unit tests (Consensys#4346)
6a3dfc2
tbenr added a commit to tbenr/teku that referenced this issue Sep 17, 2021
@tbenr
remove unnecessary payloadId handling (Consensys#4346)
9c4ed7e
tbenr added a commit to tbenr/teku that referenced this issue Sep 17, 2021
@tbenr
fix acceptance tests not throwing in RemoteValidatorApiHandler::prepa…
de67e49 
…reExecutionPayload (Consensys#4346)
tbenr added a commit to tbenr/teku that referenced this issue Sep 17, 2021
@tbenr
fix createUnsignedBlock_shouldCreateBlock unit test (Consensys#4346)
dcd077c
tbenr added a commit to tbenr/teku that referenced this issue Sep 17, 2021
@tbenr
fix BlockProductionDutyTest unit tests (Consensys#4346)
ad4c03b
tbenr added a commit that referenced this issue Sep 17, 2021
@tbenr
starts engine_preparePayload and engine_getPayload API call imple…
57fe174 
…mentation (#4371)

* starts the work for `engine_preparePayload` and `engine_getPayload` API call (#4346)

introduces Duty preparation
uses Duty preparation in BlockProductionDuty to handle the call to the execution engine to prepare the block
starts extending other required classes to be able to call the new method
currently we are using slot as payloadId
tbenr added a commit to tbenr/teku that referenced this issue Sep 20, 2021
@tbenr
Updated ExecutionPayload and ExecutionPayloadHeader Ssz (Consensy…
7d96293 
…s#4346)
tbenr added a commit to tbenr/teku that referenced this issue Sep 20, 2021
@tbenr
fixes unit test (Consensys#4346)
c3d9c97
tbenr added a commit to tbenr/teku that referenced this issue Sep 20, 2021
@tbenr
rename miner->coinbase in client schema and zeroed extra_data in Gene…
a823995 
…sisGenerator (Consensys#4346)
tbenr added a commit to tbenr/teku that referenced this issue Sep 21, 2021
@tbenr
implement engine_preparePayload and engine_getPayload (Consensys#…
9d44980 
…4346)
tbenr added a commit that referenced this issue Sep 22, 2021
@tbenr
[Merge] implements engine_preparePayload and engine_getPayload (#…
ac8fb1c 
…4399)

(#4346)

* implements `engine_preparePayload` and `engine_getPayload`
This was referenced Sep 22, 2021
Merged
Merged
This was referenced Sep 22, 2021
Merged
Merged
tbenr added a commit to tbenr/teku that referenced this issue Sep 27, 2021
@tbenr
Updated ExecutionPayload and ExecutionPayloadHeader Ssz (Consensy…
6d86a20 
…s#4346)
tbenr added a commit to tbenr/teku that referenced this issue Sep 27, 2021
@tbenr
fixes unit test (Consensys#4346)
0b2e0d6
tbenr added a commit to tbenr/teku that referenced this issue Sep 27, 2021
@tbenr
rename miner->coinbase in client schema and zeroed extra_data in Gene…
095315c 
…sisGenerator (Consensys#4346)
tbenr added a commit that referenced this issue Sep 27, 2021
@tbenr
[Merge] add extra_data field in ExecutionPayload and `ExecutionPa…
8381c48 
…yloadHeader` Ssz (#4392)

* Updated `ExecutionPayload` and `ExecutionPayloadHeader` Ssz (#4346)

* rename miner->coinbase in client schema and zeroed extra_data in GenesisGenerator  (#4346)
@tbenr
Copy link
Contributor

tbenr commented Sep 27, 2021

all PRs relative to the task list have been merged

@tbenr tbenr closed this as completed Sep 27, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@tbenr tbenr

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@ajsutton @tbenr