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

Prevent client expiration #786

Closed
5 tasks
ancazamfir opened this issue Mar 30, 2021 · 0 comments · Fixed by #872
Closed
5 tasks

Prevent client expiration #786

ancazamfir opened this issue Mar 30, 2021 · 0 comments · Fixed by #872
Assignees
@ancazamfir
Labels
E: gravity External: related to Gravity DEX I: CLI Internal: related to the relayer's CLI
Milestone
04.2021

Comments

@ancazamfir
Copy link
Collaborator

ancazamfir commented Mar 30, 2021 •
edited
Loading

Crate

relayer

Summary

Add a new hermes CLI that update clients periodically in order to prevent consensus states to expire.
Incorporate into reactive relayer (see [(#764)]

Related to #216

Problem Definition

Clients that are not active may "expire", meaning the latest on-chain trusted state may grow older than the trusting period.

Proposal

To start with, add a new CLI that monitors clients on the configured chains and updates them if required in order to prevent expiration.
Incorporate this functionality in the hermes start-multi and the passive relayer.

For Admin Use

  • Not duplicate issue
  • Appropriate labels applied
  • Appropriate milestone (priority) applied
  • Appropriate contributors tagged
  • Contributor assigned/self-assigned
@ancazamfir ancazamfir added this to the 05.2021 milestone Mar 30, 2021
@adizere adizere added the I: CLI Internal: related to the relayer's CLI label Apr 6, 2021
@ancazamfir ancazamfir self-assigned this Apr 15, 2021
This was referenced Apr 15, 2021
Closed
Closed
@ancazamfir ancazamfir mentioned this issue Apr 27, 2021
Closed
5 tasks
@adizere adizere mentioned this issue Apr 27, 2021
Closed
@ancazamfir ancazamfir modified the milestones: 05.2021, 04.2021 Apr 28, 2021
@ancazamfir ancazamfir mentioned this issue Apr 29, 2021
Merged
5 tasks
@romac romac added the E: gravity External: related to Gravity DEX label Apr 29, 2021
@romac romac closed this as completed in #872 May 4, 2021
@ancazamfir ancazamfir mentioned this issue May 6, 2021
Merged
26 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ancazamfir ancazamfir

Labels
E: gravity External: related to Gravity DEX I: CLI Internal: related to the relayer's CLI
Projects
None yet
Milestone
04.2021
Development

Successfully merging a pull request may close this issue.

Add client workers for active channels, clear packets on start informalsystems/hermes
3 participants
@romac @adizere @ancazamfir