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

[LSM] Remove slash query admin gate #902

Merged
merged 1 commit into from
Aug 24, 2023

Conversation

sampocs
Copy link
Collaborator

@sampocs sampocs commented Aug 23, 2023

Closes: #XXX

Context and purpose of the change

No reason to have the slash query admin-gated

Brief Changelog

  • Removed admin address check in validate basic

Testing

  • Start up network
  • Confirm validator sharesToTokensRate is 1.0 in state.log
  • Wait for delegation to land by checking state.log
  • Slash the validator
docker pause dockernet-gaia2-1
sleep 30
docker unpause dockernet-gaia2-1
  • Process the slash with a non-admin address
stridedl tx stakeibc update-delegation GAIA cosmosvaloper17kht2x2ped6qytr2kklevtvmxpw7wq9rarvcqz --from val1 -y
  • Confirm the sharesToTokensRate was update in state.log

@sampocs sampocs requested a review from riley-stride August 23, 2023 18:03
@sampocs sampocs merged commit f1afdab into lsm Aug 24, 2023
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