Add validators manager checks for relayer#671
Conversation
There was a problem hiding this comment.
Pull request overview
This pull request adds validators manager validation for relayer configurations and consolidates the validation logic into a centralized function. Previously, the validators manager check only ran in AUTO registration mode and was duplicated in the consolidate command. The new implementation extends the validation to also work with relayer endpoints by checking that the relayer's configured validators manager matches the vault's validators manager.
Changes:
- Extended
check_validators_manager()to validate relayer validators manager against vault validators manager when a relayer endpoint is configured - Removed duplicate validators manager validation from consolidate command and replaced it with the centralized function
- Removed the ValidatorsRegistrationMode.AUTO restriction so validation runs for all registration modes
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.
| File | Description |
|---|---|
| src/common/startup_check.py | Extended check_validators_manager() to add relayer validators manager validation and removed AUTO mode restriction |
| src/commands/consolidate.py | Removed duplicate _check_validators_manager() function and replaced with call to centralized function; removed unused imports |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 4 out of 4 changed files in this pull request and generated 2 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
No description provided.