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
SnapbackSM - Phase 1 - Recurring Sync #892
Conversation
creator-node/src/snapbackSM.js
Outdated
// Run the task every x time interval | ||
// */5 * * * *, every 5 minutes | ||
// 0 * * * *, every hour at minute 0 | ||
this.stateMachineQueue.add({}, { repeat: { cron: '0 * * * *' } }) |
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 to have, switch cron to * */n * * *
with n
as a config hourInterval
or smth per @dmanjunath
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.
So far everything looks good. First set of review comments. i got meetings for the next few hours but i'll pick back up right after
Trello Card Link
https://trello.com/c/g7NuhQwd/1543-snapbacksm-master
Description
Enable recurring sync operation separate from UserReplicaSetManager contract through SnapbackSM
Services
Does it touch a critical flow like Discovery indexing, Creator Node track upload, Creator Node gateway, or Creator Node file system?
No, but it has the potential to interfere with all of the above - so we must be very careful
How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide repro instructions & any configuration.
Include log analysis if applicable.
New mad dog test added to confirm sync progress, developer mode functionality for snapback, all unit tests for repo subcomponents passing as well