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.

Sign up for GitHub

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

Jump to bottom

Relayer CLI for client upgrade message #357

Closed
6 of 8 tasks
adizere opened this issue Nov 4, 2020 · 0 comments · Fixed by #723
Closed
6 of 8 tasks

Relayer CLI for client upgrade message #357

adizere opened this issue Nov 4, 2020 · 0 comments · Fixed by #723
Assignees
@adizere
Labels
A: question Admin: further information is requested I: CLI Internal: related to the relayer's CLI I: dependencies Internal: related to dependencies I: logic Internal: related to the relaying logic
Milestone
03.2021

Comments

@adizere
Copy link
Member

adizere commented Nov 4, 2020 •
edited
Loading

Crate

relayer, relayer-cli, ibc

Summary

The MsgUpgradeClient raw message is defined here:

https://github.com/informalsystems/ibc-rs/blob/9a930e62d715ad01d544f8c5e31af033b4649913/proto/src/prost/ibc.core.client.v1.rs#L84

We need:

  • a corresponding domain type message definition + implementation
  • clarify the exact cli syntax for this command and add support for it in relayer-cli and relayer crates:
    • relayer tx raw upgrade-client <client-id> -h <upgrade-height>
    • done with ADR 006
  • the handlers will be handled separately Handler for upgrade client #722

For Admin Use

  • Not duplicate issue
  • Appropriate labels applied
  • Appropriate milestone (priority) applied
  • Appropriate contributors tagged
  • Contributor assigned/self-assigned
@adizere adizere added A: question Admin: further information is requested I: CLI Internal: related to the relayer's CLI I: logic Internal: related to the relaying logic client labels Nov 4, 2020
@adizere adizere added this to the v0.0.6 milestone Nov 4, 2020
@adizere adizere added the I: dependencies Internal: related to dependencies label Nov 9, 2020
@adizere adizere mentioned this issue Nov 9, 2020
Closed
42 tasks
@adizere adizere modified the milestones: v0.0.6, v0.0.7 Dec 23, 2020
@ancazamfir ancazamfir modified the milestones: v0.0.7, v0.0.8 Jan 5, 2021
@ancazamfir ancazamfir mentioned this issue Sep 29, 2022
Closed
11 tasks
@ancazamfir ancazamfir modified the milestones: v0.1.1, v0.1.0 Jan 19, 2021
@adizere adizere self-assigned this Feb 9, 2021
@adizere adizere modified the milestones: v0.1.1, 03.2021 Feb 18, 2021
This was referenced Mar 3, 2021
Merged
Closed
@adizere adizere mentioned this issue Mar 18, 2021
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@adizere adizere

Labels
A: question Admin: further information is requested I: CLI Internal: related to the relayer's CLI I: dependencies Internal: related to dependencies I: logic Internal: related to the relaying logic
Projects
None yet
Milestone
03.2021
Development

Successfully merging a pull request may close this issue.

Hermes CLI for upgrading client informalsystems/hermes
2 participants
@adizere @ancazamfir