humanmade / Cavalcade-Runner

Daemon for Cavalcade, a scalable WordPress jobs system.

https://engineering.hmn.md/projects/cavalcade/

54 stars 26 forks source link

set database timezone as php one #59

Closed edubacco closed 4 years ago

edubacco commented 5 years ago

Cavalcade-Runner force db timezone to UTC. If PHP is configured to use a different timezone, jobs will be inserted and executed according to different timezones. This will cause spikes in cron executions.

edubacco commented 5 years ago

Checks fail on a not modified file

discoinfiltrator commented 5 years ago

This is not the best solution to this issue. We should make sure that any date/time logic is done in UTC instead. See:

https://github.com/humanmade/Cavalcade/issues/74

And the proposed fix: https://github.com/humanmade/Cavalcade-Runner/pull/64

roborourke commented 4 years ago

64 is now merged so I'm closing this out. Thanks for the PR @edubacco!