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

Where should thetrust_threshold and trusting_period configuration params live? #356

Closed
3 of 5 tasks
romac opened this issue Nov 3, 2020 · 0 comments · Fixed by #793
Closed
3 of 5 tasks

Where should thetrust_threshold and trusting_period configuration params live? #356

romac opened this issue Nov 3, 2020 · 0 comments · Fixed by #793
Labels
A: question Admin: further information is requested
Milestone
03.2021

Comments

@romac
Copy link
Member

romac commented Nov 3, 2020

Crate

Summary

Problem Definition

Question

PR #348 surfaced the question as to where should the trust_threshold and trusting_period configuration parameters live?

a) in the light client configuration, so that they can be configured per light-client using the light command
b) in the chain configuration, so that those parameters are consistent for all light clients

For Admin Use

  • Not duplicate issue
  • Appropriate labels applied
  • Appropriate milestone (priority) applied
  • Appropriate contributors tagged
  • Contributor assigned/self-assigned
@romac romac added the A: question Admin: further information is requested label Nov 3, 2020
@romac romac added this to the v0.0.5 milestone Nov 3, 2020
@romac romac mentioned this issue Nov 3, 2020
Merged
5 tasks
@adizere adizere mentioned this issue Nov 9, 2020
Closed
42 tasks
@adizere adizere modified the milestones: v0.0.5, v0.0.6 Dec 2, 2020
@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 Dec 23, 2020
@adizere adizere modified the milestones: v0.1.1, v0.1.2 Feb 9, 2021
@romac romac linked a pull request Apr 6, 2021 that will close this issue
Remove light client commands and configuration #793
Merged
5 tasks
@romac romac closed this as completed in #793 Apr 7, 2021
@adizere adizere modified the milestones: Backlog, 03.2021 Jun 24, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
A: question Admin: further information is requested
Projects
None yet
Milestone
03.2021
Development

Successfully merging a pull request may close this issue.

Remove light client commands and configuration informalsystems/hermes
3 participants
@romac @adizere @ancazamfir