Change hour and minute intervals in APScheudler to fix incorrect triggers

What does this MR do and why?

  • Fix logical issues where cron cannot easily be programmed for every 20 minutes, but every 20th minute.
  • Use built in minute and hour intervals in APScheduler.

Release notes should include specific breaking change notification about this. Users that were using a "every 50 minutes" basic task would have that task fire "every 50th minute" instead of the intended behavior. This will change the current behavior of our hourly and minute scheduled tasks to be what we originally intended. Users that want behavior for "every 50th minute", etc should use the cron functionality to achieve that.

How to set up and validate locally

  • Setup a task for every 50 minutes.
  • It will fire at every 50th minute of every hour, not every 50 minutes.

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

  • Have you checked this doesn't interfere/conflict/duplicate someone elses work?
  • Have you fully tested your changes?
  • Have you resolved any lint issues?
  • Have you assigned a reviewer?
  • Have you applied correct labels?
Edited by Iain Powrie

Merge request reports

Loading