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

[WIP] EIP-7732 (ePBS) #13917

Open
wants to merge 51 commits into
base: develop
Choose a base branch
from
Open

[WIP] EIP-7732 (ePBS) #13917

wants to merge 51 commits into from

Conversation

potuz
Copy link
Contributor

@potuz potuz commented Apr 25, 2024

This is a placeholder PR for ePBS so that the long standing branch runs CI and we can check tests are continuously passing.

@potuz potuz requested review from prestonvanloon and a team as code owners April 25, 2024 11:37
@potuz potuz requested review from kasey and rauljordan April 25, 2024 11:37
@potuz potuz force-pushed the epbs branch 6 times, most recently from 9895c89 to e795c7d Compare May 1, 2024 12:26
@potuz potuz force-pushed the epbs branch 9 times, most recently from 84524de to aecf4cb Compare May 9, 2024 22:52
@potuz potuz force-pushed the epbs branch 3 times, most recently from c7b3709 to e1e7f27 Compare May 16, 2024 06:46
@potuz potuz force-pushed the epbs branch 6 times, most recently from a97480c to 966e322 Compare May 24, 2024 19:14
terencechain and others added 25 commits September 13, 2024 08:07
@terencechain @Redidacove @potuz
Read only payload attestation message with Verifier (#14222)
3f1e652 
* Read only payload attestation message with verifier

* Payload attestation tests (#14242)

* Payload attestation in verification package

* Feedback #1

---------

Co-authored-by: Md Amaan <114795592+Redidacove@users.noreply.github.com>
@terencechain @potuz
Broadcast signed execution payload header to peer (#14300)
7e7773d
@jihoonsong @potuz
Add execution_payload and payload_attestation_message topics (#14304
33e9097 
)

* Add `execution_payload` and `payload_attestation_message` topics

* Set `SourcePubkey` to 48 bytes long

* Add randomly populated `PayloadAttestationMessage` object

* Add tests for `execution_payload` and `payload_attestation_message` topics
@Redidacove @potuz
Indexed paylaod attestation test (#14299)
0433f62 
* test-added

* nil check fix

* randomized inputs

* hardcoded inputs

* suggestions applied

* minor-typo fixed

* deleted
@potuz
Process Execution Payload Envelope in Chain Service (#14295)
3bca472 
Adds the processing of execution payload envelope
Corrects the protos for attestations and slashings in Electra versions
Adds generators of full blocks for Electra
@potuz @terencechain
Payload Attestation Sync package changes (#13989)
2807c96 
* Payload Attestation Sync package changes

* With verifier

* change idx back to uint64

* subscribe to topic

* add back error

---------

Co-authored-by: terence tsao <terence@prysmaticlabs.com>
@terencechain @potuz
Add getter for payload attestation cache (#14328)
e1943b7 
* Add getter for payload attestation cache

* Check against status

* Feedback #1
@terencechain @potuz
Initialize payload att message verfier in sync (#14323)
fa944ad
@potuz
Enable validator client to sign execution header (#14333)
548526d 
* Enable validator client to sign execution header

* Update proto/prysm/v1alpha1/validator-client/keymanager.proto

---------

Co-authored-by: Potuz <potuz@prysmaticlabs.com>
@Redidacove @potuz
Process withdrawal (#14297)
8092e70 
* process_withdrawal_fn and isParentfull test

* suggestions applied

* minor change

* removed

* lint

* lint fix

* removed Latestheader

* test added with nil error

* tests passing

* IsParentNode Test added

* lint

* fix test

* updated godoc

* fix in godoc

* comment removed

* fixed braces

* removed var

* removed var

* Update beacon-chain/core/blocks/withdrawals.go

* Update beacon-chain/core/blocks/withdrawals_test.go

* gazelle

* test added and removed previous changes in Testprocesswithdrawal

* added check for nil state

* decrease chromatic complexity

---------

Co-authored-by: Potuz <potuz@potuz.net>
Co-authored-by: Potuz <potuz@prysmaticlabs.com>
@potuz
Sync changes to process execution payload envelopes
f9dcfb6
@terencechain @jihoonsong @potuz
Enable validator client to sign execution payload envelope (#14346)
6a0b646 
* Enable validator client to sign execution payload envelope

* Update comment

Co-authored-by: JihoonSong <jihoonsong@users.noreply.github.com>

---------

Co-authored-by: JihoonSong <jihoonsong@users.noreply.github.com>
@jihoonsong @potuz
Add unit tests of ExecutionPayloadEnvelope verification (#14373)
2ed94ea 
* Correct requirement list of EnvelopeVerifier

* Add unit tests of ExecutionPayloadEnvelope verification
@potuz
Refactor currentlySyncingPayload cache (#14350)
9b73bec
@potuz
Signed execution payload header for sync (#14363)
0de42c1 
* Signed execution payload header for sync

* Use RO state

* SignedExecutionPayloadHeader by hash and root

* Fix execution headers cache
@potuz
[ePBS] implement UpdateVotesOnPayloadAttestation (#14308)
88a57ba
@potuz
Add ePBS fork schedule to config (#14383)
58f1e5c
@potuz
Receive ptc message (#14394)
0ef1e9c 
* Handle incoming ptc attestation messages in the chain package

* fix double import

* remove unused error
@terencechain @potuz
Prysm rpc: Submit payload attestation data (#14381)
1fcdae3
@potuz
Remove Changelog workflow
f798480
@terencechain @potuz
Prysm rpc: submit execution payload envelope (#14395)
65cbc35
@terencechain @potuz
Add wait until PTC duty helper function (#14419)
ecbd1c8 
Add wait until PTC duty
@potuz
Add GetPTCVote helpers (#14420)