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

chore(snapshots): export block 18010657 #65

Merged
merged 5 commits into from
Dec 11, 2023

Conversation

albttx
Copy link
Contributor

@albttx albttx commented Dec 6, 2023

No description provided.

@albttx albttx requested a review from jaekwon as a code owner December 6, 2023 17:27
Copy link
Contributor

@moul moul left a comment

Choose a reason for hiding this comment

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

LGTM, thank you!

@moul
Copy link
Contributor

moul commented Dec 6, 2023

Addresses #18

@albttx albttx marked this pull request as draft December 6, 2023 17:55
@albttx
Copy link
Contributor Author

albttx commented Dec 6, 2023

Update, in the block 18010659 we don't have the information we want, updatin to block 18010658

@giunatale
Copy link
Collaborator

I was gonna ask about that. Thanks for checking @albttx.

@giunatale
Copy link
Collaborator

can we also take one for prop 82 while we are at it (same 1-block before where we were pointing, and check that votes are there)? even if we don't end up using it, it's useful data for explorations.

Also, how big is the genesis file (I am on my phone rn), can we use public storage?

@moul
Copy link
Contributor

moul commented Dec 6, 2023

3.5G

@tbruyelle
Copy link
Contributor

@albttx let's update the PR title to use the correct block height.

@albttx albttx changed the title chore(snapshots): export block 18010659 chore(snapshots): export block 18010657 Dec 11, 2023
@albttx albttx marked this pull request as ready for review December 11, 2023 14:14
@albttx
Copy link
Contributor Author

albttx commented Dec 11, 2023

Ready for review: @jaekwon @moul

Copy link
Contributor

@jaekwon jaekwon left a comment

Choose a reason for hiding this comment

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

I added a TODO note that validators that want to join must duplicate and understand these steps. So I validators will do the reviewing for us upon genesis of the chain or govno.

@jaekwon jaekwon merged commit b3d1d4e into atomone-hub:main Dec 11, 2023
@giunatale
Copy link
Collaborator

@jaekwon to clarify, the exported json is a standard export of the state from a hub node at that block height.

it's just raw, unprocessed data. But it should contain most everything we need and it should be accurate. (votes and delegations are there)

As written in the readme, due to how tally works and the fact that votes are removed once the tally is done, if we want to be completely accurate we should also look at the content of the next block, which is when the tally is actually performed (but the state exported at that height does not contain votes anymore).

@Devona0
Copy link

Devona0 commented Feb 22, 2024

yes


### Download

An export is stored in S3 accessible here: https://atomone.fra1.digitaloceanspaces.com/cosmoshub4/cosmoshub-4-export-18010657.json
Copy link

Choose a reason for hiding this comment

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

Copy link
Contributor

Choose a reason for hiding this comment

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

There was a typo that has been fixed, use the main version plz.

@tbruyelle
Copy link
Contributor

tbruyelle commented Mar 14, 2024

This document has been moved to https://github.com/atomone-hub/govbox/blob/master/SNAPSHOT-EXTRACT.md

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.

7 participants