-
Notifications
You must be signed in to change notification settings - Fork 557
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
[Rosetta] use mina missing block guardian script instead of download missing blocks #16375
Conversation
!ci-build-me |
!ci-nightly-me |
@@ -109,7 +116,7 @@ MINA_DAEMON_PID=$! | |||
sleep 30 | |||
|
|||
echo "========================= POPULATING MISSING BLOCKS ===========================" | |||
./download-missing-blocks.sh --network ${MINA_NETWORK} --archive-uri ${PG_CONN} & | |||
/etc/mina/archive/missing-blocks-guardian.sh single-run & |
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.
Thoughts: The missing-blocks-guardian
is not only used once to populate the missing blocks between the daily SQL dump and the current latest available block. The missing-blocks-guardian
should run every hour to ensure no blocks are missed in the long run.
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.
Oki, so we should use different mode of missing-blocks-guardian which will run as daemon
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.
It looks good to me.
!ci-build-me |
!ci-nightly-me |
!ci-build-me |
Porting fix only from PR: #16346, while waiting for test stabilization
How i tested it: