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.

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

feat: add support for remote signers - use_remote_signer #791

Merged
merged 5 commits into from
Oct 7, 2024

Conversation

eth2353
Copy link
Contributor

@eth2353 eth2353 commented Oct 5, 2024

Adds the use_remote_signer flag, outsourcing validator key management to a remote signer, which validator clients connect to.

Supported by all VCs except Lighthouse through this PR.

Slashing protection DB is not enabled for web3signer since that would require adding significantly more complexity - creating a separate postgres DB and running migrations.

I was thinking a lot what to call the remote signer service, in the end went with signer-1-geth-lodestar (this is the remote signer for vc-1-geth-lodestar).

Addresses #263 / #758

Prerequisite for adding Vero as a validator client - it exclusively uses a remote signer

@eth2353 eth2353 changed the title Add support for remote signers - use_remote_signer feat: add support for remote signers - use_remote_signer Oct 5, 2024
@TobiWo
Copy link
Contributor

TobiWo commented Oct 6, 2024

This is nice. I actually need this to better test the latest feature for eth-duties. Thanks for bringing this up 👍

@eth2353 eth2353 marked this pull request as ready for review October 6, 2024 19:10
@barnabasbusa barnabasbusa linked an issue Oct 7, 2024 that may be closed by this pull request
@barnabasbusa
Copy link
Contributor

LGTM

@barnabasbusa barnabasbusa merged commit 9f1b6e9 into ethpandaops:main Oct 7, 2024
13 checks passed
@eth2353 eth2353 deleted the remote-signer branch October 7, 2024 14:02
skylenet pushed a commit that referenced this pull request Nov 1, 2024
🤖 I have created a release *beep* *boop*
---


##
[4.4.0](4.3.0...4.4.0)
(2024-11-01)


### Features

* add checkpoint sync capabilities to nimbus
([#804](#804))
([853417e](853417e))
* add commit-boost support
([#779](#779))
([ebbbe83](ebbbe83))
* add docker authentication
([#816](#816))
([807f6aa](807f6aa))
* add peerdas metrics dashboard
([#790](#790))
([12b787d](12b787d))
* add rbuilder remove old geth builder - flashbots
([#786](#786))
([de95c61](de95c61))
* add resource configuration for prometheus and grafana
([#773](#773))
([d296c26](d296c26))
* add supernode label to every container
([#788](#788))
([43edfd5](43edfd5))
* add support for fulu/osaka fusaka
([#798](#798))
([0a9e445](0a9e445))
* add support for remote signers - `use_remote_signer`
([#791](#791))
([9f1b6e9](9f1b6e9))
* enable one parameter supernode, refactor el,cl,vc
([#778](#778))
([8513c06](8513c06))


### Bug Fixes

* add database config to assertoor
([#783](#783))
([56532cf](56532cf))
* add nimbus-eth1 net-key
([#811](#811))
([c91bbbb](c91bbbb))
* delete unnecessary port assignment for prysm
([#810](#810))
([47204c3](47204c3))
* docker login to separate action yaml
([#819](#819))
([2494022](2494022))
* get volume size recommendations when in shadowfork mode
([#820](#820))
([b1f27c6](b1f27c6))
* mixed up labels
([#799](#799))
([e2c1528](e2c1528))
* only set supernode if its true
([#796](#796))
([2110a60](2110a60))
* prefund container suffixes for k8s
([#818](#818))
([968cfbd](968cfbd))
* remove deprecated http-allow-sync-stalled from LH
([#805](#805))
([1825dbf](1825dbf))
* Remove not necessary "/api" (and bump verifier version)
([#792](#792))
([befde97](befde97))
* set default for label-maker
([#797](#797))
([5b2d234](5b2d234))
* swap http/grpc server ports and replace flags
([#802](#802))
([8c6df26](8c6df26))
* update built in assertoor tests
([#782](#782))
([d24fb2a](d24fb2a))
* update nethermind config option
([#824](#824))
([08ce034](08ce034))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Allow external-signers to be specified
3 participants