-
Notifications
You must be signed in to change notification settings - Fork 166
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
Bootstrapping Generates Root Snapshot File #424
Bootstrapping Generates Root Snapshot File #424
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
For this lint error, needs to do serializable.Encodable()
. inmem.Snapshot
implements protocol.Snapshot
but is not directly serializable, inmem.EncodableSnapshot
has all the exported fields needed to be serialized from another package.
For this error, it is fixed in this PR #374
We will also need to update the I have added some more detail to the issue regarding the final step of this ticket:
|
…-root-snapshot-bootstrap
…-root-snapshot-bootstrap
…-root-snapshot-bootstrap
…om:onflow/flow-go into danu/5286/generate-root-snapshot-bootstrap
…om:onflow/flow-go into danu/5286/generate-root-snapshot-bootstrap
…om:onflow/flow-go into danu/5286/generate-root-snapshot-bootstrap
This commit is a reference point for reproducing inconsistent sealing behaviour between integration tests (TestMVP_Bootstrap) and localnet. Localnet can be started, then restarted, and sealing will continue. (This is without submitting any transactions.) make init && make start make stop && make start The TestMVP_Bootstrap integration test (which does the same network restart automatically) fails to seal any blocks after the network is restarted.
This reverts commit 6a3bb95.
…/5286-bootstrap-test
Update bootstrap MVP integration test
Resolves https://github.com/dapperlabs/flow-go/issues/5286.