colossus: sync - longer default interval

Joystream / joystream

Joystream Monorepo

http://www.joystream.org

GNU General Public License v3.0

1.42k stars 115 forks source link

colossus: sync - longer default interval #4924

Closed mnaamani closed 9 months ago

mnaamani commented 11 months ago

As per recommendation in https://github.com/Joystream/joystream/pull/4921#issuecomment-1754633973

Changed default syncInterval to 20min if not specified on commandline
Do not wait initial interval before starting sync
Add another interval defaulting to 3min on sync failure

This is not an urgent update since it is really only changing the default, but it would be good to confirm with Storage lead if it does indeed help storage operators overall.

mnaamani commented 9 months ago

Looks good, just one small comment. And for the delay, I repeatedly missed reviewing this.

Thanks for review. Fixed the typo in c25416d