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

User-facing Hermes CLIs to initiate channel & connection handshakes #1116

Closed
5 of 6 tasks
adizere opened this issue Jun 22, 2021 · 1 comment
Closed
5 of 6 tasks

User-facing Hermes CLIs to initiate channel & connection handshakes #1116

adizere opened this issue Jun 22, 2021 · 1 comment
Labels
I: CLI Internal: related to the relayer's CLI O: usability Objective: cause to improve the user experience (UX) and ease using the product
Milestone

Comments

@adizere
Copy link
Member

adizere commented Jun 22, 2021

Crate

ibc-relayer-cli

Problem Definition

With the introduction of the connection worker (#1019) and earlier the channel worker (#924) Hermes is able to complete the conn open handshakes while running passively. The only action a user needs to take is to initiate each handshake(s). Currently we only have tx raw CLIs to initiate handshakes, which are not designed to be user-facing.

Proposal

Add a CLI

  • hermes tx open-init [channel/connection] or
  • hermes init [channel/connection] or similar

to allow the user to trigger the first step of the open handshake protocol. Thereafter, hermes start will run the rest of the handshake on its own.

Acceptance Criteria

  • two new user-facing CLIs, similar in functionality to tx raw chan-open-init and tx raw conn-init.

For Admin Use

  • Not duplicate issue
  • Appropriate labels applied
  • Appropriate milestone (priority) applied
  • Appropriate contributors tagged
  • Contributor assigned/self-assigned
@adizere adizere added I: CLI Internal: related to the relayer's CLI O: usability Objective: cause to improve the user experience (UX) and ease using the product labels Jun 22, 2021
@adizere adizere modified the milestones: 07.2021, 08.2021 Jun 22, 2021
@adizere adizere modified the milestones: 08.2021, Backlog Aug 3, 2021
@adizere adizere modified the milestones: Backlog, v1.1 Sep 30, 2022
@adizere
Copy link
Member Author

adizere commented Sep 30, 2022

This was resolved with the removal of the raw suffix (ref #2315), allowing us to have commands:

  • hermes tx conn-init
  • hermes tx chan-open-init

Copying @yaruwangway here, as she might be interested in these CLIs. We recently (likely as part of work in cosmos/gaia#1766) noticed the need to have exactly a commands such as the ones above as part of the CI/CD testing of gaiad.

@adizere adizere closed this as completed Sep 30, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
I: CLI Internal: related to the relayer's CLI O: usability Objective: cause to improve the user experience (UX) and ease using the product
Projects
None yet
Development

No branches or pull requests

1 participant