[DON'T MERGE] Migrating PeerDAS to Pectra + soft-forking #163
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Partially addresses Consensys#7762 Signed-off-by: Paul Harris <[email protected]>
…#8485) Partially addresses Consensys#7762 Signed-off-by: Paul Harris <[email protected]>
partially addresses Consensys#7762 Signed-off-by: Paul Harris <[email protected]>
* wip array type definition * tests added, fixed error Signed-off-by: Paul Harris <[email protected]> * Add post attestation v2 interface * add missing final * rename json files * refactoring * add integration tests * refactoring --------- Signed-off-by: Paul Harris <[email protected]> Co-authored-by: Paul Harris <[email protected]>
Signed-off-by: Paul Harris <[email protected]>
) The concept of `? extends Attestation` got a little messy... wondering if we need to make it closer to the way blocks are, where Attestation is a class, but also not sure how much 'better' this would look without spiking it. partially addresses Consensys#7762 Signed-off-by: Paul Harris <[email protected]>
Signed-off-by: Gabriel Fukushima <[email protected]>
* refactor create aggregate attestation request * refactor
* Remove -X prefix Signed-off-by: Gabriel Fukushima <[email protected]> * add changelog Signed-off-by: Gabriel Fukushima <[email protected]> --------- Signed-off-by: Gabriel Fukushima <[email protected]>
* remove deprecated GetNewBlindedBlock Signed-off-by: Gabriel Fukushima <[email protected]> * add changelog Signed-off-by: Gabriel Fukushima <[email protected]> * removed json schema and response Signed-off-by: Gabriel Fukushima <[email protected]> --------- Signed-off-by: Gabriel Fukushima <[email protected]>
* Cleaned up interfaces for remote request Also added some tests for minimal typedef functions which were missing. I'd like to ultimately move everything to this model where we're testing per class, but it's a larger cleanup, so here's a good sanity check point. partially addresses Consensys#7762 Signed-off-by: Paul Harris <[email protected]>
This meant handling our failure list responses also, which will be needed for a couple of other interfaces also, so i've put the handling into a spot we can get to. partially addresses Consensys#7762 Signed-off-by: Paul Harris <[email protected]>
* Remove BlocksV2 Signed-off-by: Gabriel Fukushima <[email protected]> * remove API resources and tests from GetBlockV2 Signed-off-by: Gabriel Fukushima <[email protected]> * add changelog Signed-off-by: Gabriel Fukushima <[email protected]> --------- Signed-off-by: Gabriel Fukushima <[email protected]>
- moved the last 2 endpoints to typedef - cleaned up interface ValidatorRestApiClient interface - Renamed SendSyncCommitteeRequest This removes the last usages of the old schema based interfaces in the VC. fixes Consensys#7762 Signed-off-by: Paul Harris <[email protected]>
…ys#8487) * support schema for oneOf electra and phase0 Signed-off-by: Gabriel Fukushima <[email protected]> * fix tests Signed-off-by: Gabriel Fukushima <[email protected]> --------- Signed-off-by: Gabriel Fukushima <[email protected]>
* Moved slashing protection interchange format away from using JsonProvider. Signed-off-by: Paul Harris <[email protected]>
…ader (Consensys#8516) * add Eth-Consensus-Version header to getBlobSidecars response Signed-off-by: Gabriel Fukushima <[email protected]> * spotless Signed-off-by: Gabriel Fukushima <[email protected]> --------- Signed-off-by: Gabriel Fukushima <[email protected]>
This reverts commit 89dd8fa.
* remove experimental flag and useBlindedBlock variable now that V3 can handle blind and unblinded flows. Signed-off-by: Gabriel Fukushima <[email protected]> --------- Signed-off-by: Gabriel Fukushima <[email protected]>
also restored some deserializers for JsonProvider that are likely used by external signer... Signed-off-by: Paul Harris <[email protected]>
Signed-off-by: Paul Harris <[email protected]>
- migrated TekuBeaconNode towards using mostly ObjectMapper where possible, though i did use a couple of typedefs. - Added ObjectMapper to Node, but haven't yet removed JsonProvider because its still heavily used in AT. - Several functions now return more specific things, which simplifies using ObjectMapper. Signed-off-by: Paul Harris <[email protected]>
* add get aggregate attestation request * fix integration test * refactoring * rename test * spotless * refactor aggregattion api * refactoring * refactoring * rename aggregate attestation request * rename request * apply spotless * use schema definition cache * refactoring * use spec when submit * refactor create aggregate request
* update blob sidecars gossip topics * use misc helpers to get blob sidecars subnet count * remove unused method in gossip filter test
* update blobs rpc methods
Co-authored-by: Enrico Del Fante <[email protected]>
* migrate beacon state * fix historical summary schema * fix ref test compilation * fix ref tests
* migrate-block-content-schemas * deleted too much * lookup cache first
Fix incorrect variable syntax in state download request Co-authored-by: Paul Harris <[email protected]>
* use attestation bits aggregator in performance tracker * add electra tests * fix spotless * fix unit test * always clear objects * refactor bits aggregator or operation * revert bits aggregator or operation return type
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
WIP