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 vero vc_type #827

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

feat: add vero vc_type #827

wants to merge 2 commits into from

Conversation

eth2353
Copy link
Contributor

@eth2353 eth2353 commented Nov 5, 2024

Adds the Vero validator client as a supported vc_type.

The Vero VC doesn't handle keys and requires a remote signer to be used (use_remote_signer: true). It can handle the minimal preset, as well as custom seconds_per_slot values.

I've added the "Vero - Detailed" Grafana dashboard. I'm not sure if it's a good fit for this repo, let me know if you'd like me to remove it.

@eth2353 eth2353 changed the title feat: add Vero VC type feat: add vero vc_type Nov 5, 2024
@eth2353 eth2353 marked this pull request as ready for review November 5, 2024 17:40
@barnabasbusa
Copy link
Contributor

Nice PR, seems to be working as described.

Few nitpicks, could you modify the grafana dash to show the name (job/service) of the container instead of the instance which is IP:, and possibly have an option to enable overview of all validator processes at the same time.

@eth2353
Copy link
Contributor Author

eth2353 commented Nov 8, 2024

Grafana dashboard modified to show the proper name/job.

possibly have an option to enable overview of all validator processes at the same time.

This is not too easy to adjust in this dashboard, it's really quite instance-specific.

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.

2 participants