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

Enable L1 Validator Removal Using Uptime Tracking #2394

Closed
meaghanfitzgerald opened this issue Dec 3, 2024 · 0 comments · Fixed by #2379
Closed

Enable L1 Validator Removal Using Uptime Tracking #2394

meaghanfitzgerald opened this issue Dec 3, 2024 · 0 comments · Fixed by #2379
Assignees
Labels

Comments

@meaghanfitzgerald
Copy link
Collaborator

Goal

avalanche blockchain removeValidator currently relies on P-Chain transactions to forcefully remove validators from the chosen L1. removeValidator should have this as an option, but instead default to the removing a Validator via the validation manager smart contract using valid uptime proofs. This would also enable an L1 validator to claim any validation rewards.

Assumptions and Scope

  • The CLI is able to retrieve valid uptime proofs from Subnet-EVM's uptime endpoints
  • The validator was either originally added via the CLI with avalanche blockchain addValidator or the user has the validation params available to provide
  • The CLI checks for potential rewards, and displays the estimated reward before asking "do you wish to proceed with validator removal?"

Requirements

  • The CLI checks for existing L1 configurations, and if they don't exist, prompts the user for the L1 params necessary to initiate and confirm a validator removal via the smart contract

Open Questions

NA

@arturrez arturrez moved this from Backlog 🗄️ to Paused 🧊 in Platform Engineering Group Dec 4, 2024
@arturrez arturrez moved this from Paused 🧊 to In Review 👀 in Platform Engineering Group Dec 4, 2024
@meaghanfitzgerald meaghanfitzgerald linked a pull request Dec 4, 2024 that will close this issue
@github-project-automation github-project-automation bot moved this from In Review 👀 to Done ✅ in Platform Engineering Group Dec 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

2 participants