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

QGB sign attestations only if part of the valset #480

Closed
Tracked by #301
rach-id opened this issue Jun 10, 2022 · 0 comments · Fixed by #523
Closed
Tracked by #301

QGB sign attestations only if part of the valset #480

rach-id opened this issue Jun 10, 2022 · 0 comments · Fixed by #523
Assignees
@rach-id

Comments

@rach-id
Copy link
Member

rach-id commented Jun 10, 2022 •
edited
Loading

Currently, the QGB orchestrator signs all the attestations up to the unbonding period. This needs to be updated to only sign the valsets/data commitments when the orchestrator is part of the validator set.
@rach-id rach-id added the C: QGB label Jun 10, 2022
@rach-id rach-id moved this to TODO in Celestia Node Jun 10, 2022
@rach-id rach-id mentioned this issue Jun 10, 2022
Closed
90 tasks
@rach-id rach-id mentioned this issue Jul 5, 2022
Merged
@rach-id rach-id linked a pull request Jul 5, 2022 that will close this issue
QGB Orchestrator only signs if part of valset #523
Merged
@rach-id rach-id closed this as completed Jul 12, 2022
Repository owner moved this from TODO to Done in Celestia Node Jul 12, 2022
@rach-id rach-id self-assigned this Jul 12, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@rach-id rach-id

Labels
None yet
Projects
No open projects
Celestia Node
Archived in project
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

QGB Orchestrator only signs if part of valset rach-id/celestia-app
1 participant
@rach-id