akvo / akvo-rsr

Akvo Really Simple Reporting

http://rsr.akvo.org

GNU Affero General Public License v3.0

39 stars 20 forks source link

[#5248] Disable catch-up for django-q #5250

Closed MichaelAkvo closed 1 year ago

MichaelAkvo commented 1 year ago

TODO / Done

Summarize what has been changed / what has to be done in order to finalize the PR.

[x] Disable catchup
We do not need it as we schedule tasks with cron and they will eventually be executed. Especially tasks that are executed every minute to not need catching up.

Test plan

What tests are necessary to ensure this works or doesn't break anything working

[x] Manual test

Manual test

Start RSR
Wait until everything is up
Stop the worker for a few minutes
Start the worker
Ensure that aggregation job is only scheduled once instead of the number of minutes the worker was down

Related to #5248

coveralls commented 1 year ago

Coverage: 68.217%. Remained the same when pulling d8a99b4ca99231d32825661be96203a82ce8e5aa on 5248-bug-django-q-plays-catch-up-with-tasks-and-reschules-many-after-a-deployment into 8e7b9e798c135a79131745f12ef595c08d14e527 on master.