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

[3.2] Add command line option --snapshot-to-json #710

Closed
wants to merge 3 commits into from

Conversation

ClaytonCalabrese
Copy link
Contributor

Add new command line option --snapshot-to-json which reads given snapshot.bin and generates a JSON equivalent file.

Resolves #151

@ClaytonCalabrese ClaytonCalabrese added 3.2 Candidate OCI OCI working this issue... labels Jul 21, 2022
@ClaytonCalabrese ClaytonCalabrese requested a review from heifner July 21, 2022 15:16
@ClaytonCalabrese ClaytonCalabrese self-assigned this Jul 21, 2022
Copy link
Member

@spoonincode spoonincode left a comment

Choose a reason for hiding this comment

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

The linked issue mentions that this should not be part of nodeos, and that a separate tool should be created instead.

@heifner
Copy link
Member

heifner commented Jul 21, 2022

The linked issue mentions that this should not be part of nodeos, and that a separate tool should be created instead.

Completely agree. Do you want that done as part of this PR or as a separate task?

@spoonincode
Copy link
Member

Let's first decide if we want to jump on to an eosio-snapshot like application or do a more through integrated solution by removing eosio-blocklog and have a single eosio-util. If we just go with an eosio-snapshot it's pretty easy to bang that out unlike more broad changes.

@ClaytonCalabrese
Copy link
Contributor Author

ClaytonCalabrese commented Aug 22, 2022

closing due to migration to leap AntelopeIO/leap#24

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3.2 Candidate OCI OCI working this issue...
Projects
None yet
Development

Successfully merging this pull request may close these issues.

snapshot json support
3 participants