Skip to content

Feature/dkg snapshot epoch #5900

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

Merged
merged 8 commits into from
Jul 21, 2025
Merged

Feature/dkg snapshot epoch #5900

merged 8 commits into from
Jul 21, 2025

Conversation

jstuczyn
Copy link
Contributor

@jstuczyn jstuczyn commented Jul 17, 2025

NET-501

In order to determine if singer quorum has been down at particular height, we need to know with certainty the dkg epoch id corresponding to given block height.

this PR makes it possible. every time epoch state is changed (due to DKG progress), snapshot is saved and can be queried. this doesn't work for past data, but given mainnet has only had a single DKG instance, that's not an issue.


This change is Reviewable

@jstuczyn jstuczyn requested a review from neacsu July 17, 2025 13:52
Copy link

vercel bot commented Jul 17, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
nym-explorer-v2 ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 17, 2025 2:23pm
nym-next-explorer ❌ Failed (Inspect) Jul 17, 2025 2:23pm
1 Skipped Deployment
Name Status Preview Comments Updated (UTC)
docs-nextra ⬜️ Ignored (Inspect) Visit Preview Jul 17, 2025 2:23pm

Copy link
Contributor

@neacsu neacsu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reviewed 19 of 20 files at r1, 4 of 4 files at r2, all commit messages.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved (waiting on @jstuczyn)

@jstuczyn jstuczyn merged commit e916576 into develop Jul 21, 2025
18 of 21 checks passed
@jstuczyn jstuczyn deleted the feature/dkg-snapshot-epoch branch July 21, 2025 16:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants