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

client init-validator transaction command should update config #88

Closed
Tracked by #2533
tzemanovic opened this issue Mar 15, 2022 · 4 comments
Closed
Tracked by #2533

client init-validator transaction command should update config #88

tzemanovic opened this issue Mar 15, 2022 · 4 comments
Assignees
Labels
ledger pre-mainnet Must happen before mainnet. prio:high
Milestone

Comments

@tzemanovic
Copy link
Member

the init-validator command should update the Anoma config after it successfully applies the transaction to set ledger.tendermint.tendermint_node = "Validator"

@tzemanovic
Copy link
Member Author

note, we don't yet have a e2e test case that would try to run the ledger node for the new validator, but we could extend e2e::ledger_tests::pos_init_validator to check this

@tzemanovic tzemanovic transferred this issue from anoma/anoma Jul 7, 2022
@cwgoes
Copy link
Collaborator

cwgoes commented Jan 12, 2023

@brentstone can you check on this / take this on if necessary as part of the PoS work?

@brentstone
Copy link
Collaborator

@brentstone can you check on this / take this on if necessary as part of the PoS work?

Sure.

@cwgoes cwgoes added this to the 0.15 milestone Jan 16, 2023
@cwgoes cwgoes added the pre-mainnet Must happen before mainnet. label Feb 5, 2024
@tzemanovic
Copy link
Member Author

this has been done somewhere

@github-project-automation github-project-automation bot moved this from Todo to Tested in Devnet in Namada-Old Feb 8, 2024
phy-chain pushed a commit to phy-chain/namada that referenced this issue Mar 1, 2024
* Begin changes to extension specs

* Update architecture for extension, add image

* Add integration docs to specs

* Remove outdated api docs (see integration.md)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ledger pre-mainnet Must happen before mainnet. prio:high
Projects
No open projects
Status: Tested in Devnet
Development

No branches or pull requests

3 participants