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

Feature: One password file for all validator #8731

Open
maximus321 opened this issue Oct 15, 2024 · 1 comment
Open

Feature: One password file for all validator #8731

maximus321 opened this issue Oct 15, 2024 · 1 comment

Comments

@maximus321
Copy link

maximus321 commented Oct 15, 2024

Description

It would be nice to have a way to only use one validator password file for all my validator.
With a high number of validators, that is easy to achieve now with lido csm, we can have an overwheliming number of .txt file that contains the same information (not optimal)

It would be nice to have an option like
--single-password-file:pwd.txt

As discussed on discord here
https://discord.com/channels/697535391594446898/697539289042649190/1295781894008471604

Steps to Reproduce (Bug)

[Please be as specific as possible]

Expected behavior: [What you expect to happen]

Actual behavior: [What actually happens]

Frequency: [How regularly does it occur?]

Versions (Add all that apply)

  • Software version: [teku --version or look for log lines starting with Teku version:]
  • Java version: [java -version]
  • OS Name & Version: [cat /etc/*release]
  • Docker Version: [docker version]
  • Cloud VM, type, size: [Amazon Web Services I3-large]
@rolfyone
Copy link
Contributor

Hey thanks for raising this. We'll see what we can do to improve the situation.

We do similar too for our validators so I feel like it's a use-case we have worked around and could probably improve, we just may need to specify the keys differently for this new scenario or something, but we'll schedule some time to do some design.

Just for context from others browsing, currently teku supports basically a list of key:password entries
https://docs.teku.consensys.io/reference/cli#validator-keys
which may be pairs of directories or pairs of files,
and we'd be changing to a semantic of specifying the keys in one parameter (files or directory most likely) and taking the password from a separate entry.

The validator-keys cli arg is very commonly used currently, so a solution where current behaviour remains valid would be required. This may mean we end up using a different flag to specify just the keys, and a flag for the one password file, but I won't preclude the design process by 'solutioning' just yet.

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

No branches or pull requests

3 participants
@rolfyone @maximus321 and others