-
Notifications
You must be signed in to change notification settings - Fork 807
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 by Bors] - Shutdown after sync #2519
Conversation
bac3aaf
to
384807a
Compare
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.
This looks fine to me. Re-running tests as windows failed.
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.
Nice one! There's a cargo-audit fail in CI but I think that's been fixed in unstable
, therefore bors should work.
bors r+
Merge conflict. |
Sorry @pawanjay176, looks like there's been a new merge conflict introduced :( |
Fixed @paulhauner |
bors r+ |
## Issue Addressed Resolves #2033 ## Proposed Changes Adds a flag to enable shutting down beacon node right after sync is completed. ## Additional Info Will need modification after weak subjectivity sync is enabled to change definition of a fully synced node.
Pull request successfully merged into unstable. Build succeeded: |
Issue Addressed
Resolves #2033
Proposed Changes
Adds a flag to enable shutting down beacon node right after sync is completed.
Additional Info
Will need modification after weak subjectivity sync is enabled to change definition of a fully synced node.