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: mainnet validator hot swaps #4203

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

Conversation

joelsmith-2019
Copy link
Collaborator

@joelsmith-2019 joelsmith-2019 commented Dec 19, 2024

This PR allows mainnet validator hot swapping. Previously, only single validator clusters allowed hot swapping for testnet purposes. This PR removes that check and introduces an integration test which validates the process of swapping an existing guardian's validator to a new validator and then back again.

@joelsmith-2019 joelsmith-2019 self-assigned this Dec 19, 2024
@joelsmith-2019 joelsmith-2019 marked this pull request as ready for review December 19, 2024 19:11
Copy link
Contributor

@mdulin2 mdulin2 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We need to make this non-consensus breaking.

@pires
Copy link
Collaborator

pires commented Jan 13, 2025

Is this truly affecting only mainnet. Please, be reminded that in testnet the guardian set size is one (1) and we should keep the ability to rotate the key. cc @andreclaro

@mdulin2
Copy link
Contributor

mdulin2 commented Jan 13, 2025

This is expanding the ability to rotate keys. Right now, there is a restriction that the hot swap is only allowed when there is a single Guardian (aka testnet). This allows for all Guardians to do the hot swap at any point.

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.

4 participants