Individual cron job runs should be marked as timed out in the cron log

[ayust]

If a cron job run times out, currently it winds up just displaying with a yellow background (as if it were still running), and no duration / result.

It should display the duration after which it timed out, and a result of 'timeout'

[7sempra]

"timeout" should probably be a separate flag from result. A task that timed out might still finish successfully, or it might fail in a particular way that matters.

[ayust]

Hm. What is your expectation for what happens when something times out? I would expect a task that times out to be killed, and thus generally be a failure.

[7sempra]

So, we currently can't kill tasks that exceed their timeout window (at least, not in a reliable way). So we have to wait for them to finish, or not.

However, we could get around this by actually moving the tasks onto individual Web Workers. Those can be terminated, but it probably makes other orchestration a bit harder. Worth some investigation.

[7sempra]

...or just move to an actual job management framework like Kue: https://github.com/Automattic/kue

[ayust]

yeah but let's not introduce another entire database just to be able to kill cron jobs