Prysm V3 General Wishlist

[rauljordan]

🚀 Feature Request

Description

"The Merge" will mark an important event for Prysm, as it is a good time to issue a Prysm "V3" containing any breaking changes we have been holding off from during Altair. This is a tracking issue for any desired changes from the team and the community.

Ideas

• Deprecate the --http-web3provider flag to something more idiomatic. This will be the most important flag for the merge, as running an execution node is a requirement when participating in the Ethereum network
• Delete all currently deprecated flags
• Rename import paths to /v3/ to help with Go downstream imports
• Rename all the fork-related protobufs from V1, V2, V3 to phase0, altair, bellatrix (relevant to the signing request protobufs)

[terencechain]

• Delete deprecated APIs. A bunch of prysm debug APIs that's never used and becoming hard to maintain over time (i.e get forkchoice nodes)
• Revisit general logging. Some logs have useless fields and some logs miss important fields

[potuz]

• three close to my heart would be to merge clear pending atts after syncing block #9679 Adapt Doppelganger to Altair #9969 and Add USE_PRYSM_MODERN environment variable #10229.
• deprecate validator metrics and move them to the beacon node monitor

[rkapka]

Should we merge this with #10094?

[rauljordan]

Yep! Im ok with that :)

…

On Tue, Mar 8, 2022 at 5:50 PM Radosław Kapka ***@***.***> wrote: Should we merge this with #10094 <#10094>? — Reply to this email directly, view it on GitHub <#10322 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/ABKQQPOBAOFGNC3I3D6HP33U67KS7ANCNFSM5QE7XTWQ> . You are receiving this because you authored the thread.Message ID: ***@***.***>

-- *Raul E. Jordan | Co-Founder, Prysmatic Labs* prysmaticlabs.com rauljordan.com Twitter: @rauljordaneth <https://twitter.com/rauljordaneth> gpg --receive-keys 95452A701810FEDB

[rkapka]

API

• Use more idiomatic oneof and optionals for a cleaner schema
• Change the SignedBeaconBlockContainerV2 to return a oneof type for SignedBeaconBlocks instead of having the signature as a separate field in the protobuf (this change is incompatible with Beacon APIs spec)
• Standard API: remove v2 from all messages that are not request/responses (e.g. BeaconStateV2)
• Prysm API: rename StreamBlocksAltair to StreamBlocks

[yorickdowne]

• Validator client: Support REST connection to consensus client. This would allow the Prysm validator client to be used in setups that use a diverse set of consensus clients, with the validator client having a list of three or more different consensus clients in its configuration.

[rauljordan]

• Validator client: Support REST connection to consensus client. This would allow the Prysm validator client to be used in setups that use a diverse set of consensus clients, with the validator client having a list of three or more different consensus clients in its configuration.

Hi @yorickdowne , this task involves implementing a new validator client software entirely from scratch, completely different than the one we use today. We estimate this task would take at least half a year including testing and confidence boosting, as the Prysm validator client has been in production for years and is one of the riskiest pieces of our codebase. Having a brand new one incurs a lot of risk on stakers if not tested thoroughly. We have ruled this out in the short term, as it is unreasonable given our other priorities such as the merge and sharding

[rkapka]

• Signing Request OneOf naming for blocks is wrong #10853

[rkapka]

• Deprecate BeaconBlocksByRange.step #10827 (can we simply remove it for v3?)

[nisdas]

• Deprecate BeaconBlocksByRange.step #10827 (can we simply remove it for v3?)

@rkapka This is already done in #10914

[james-prysm]

• delete unused web APIs after switching to keymanager APIs

[nisdas]

Feature Flags

• Remove Deprecated Feature Flags ( Remove Deprecated Feature Flags #11125)

Networking

• Penalize Bad Payloads ( Penalize Bad Payloads Properly #11126)

[prestonvanloon]

• Bazel cross compile binaries for mac M1 (darwin_arm64) Add cross compilation for OSX ARM64 (Apple Silicon) #10981

[terencechain]

Closing this. For the remaining items that we wish to carry to V4 (breaking changes only), we can open a separate issue for it