| === frankban|afk is now known as frankban | ||
| cjwatson | wgrant: Job leases confuse me. Do you know of a particular reason why queueing a job shouldn't just release the lease? AFAICT retry_delay should be sufficient to arrange for it to be queued a reasonable time in the future, and having the lease still held just confuses things. | 09:13 |
|---|---|---|
| wgrant | cjwatson: retry_delay is only for celery, isn't it? | 09:14 |
| wgrant | I agree it's confusing, but it's not *just* confusing. | 09:14 |
| cjwatson | Oh, does nothing else honour scheduled_start? Hmm. | 09:14 |
| wgrant | Oh, it's possible | 09:15 |
| cjwatson | Job.ready_job checks scheduled_start | 09:15 |
| cjwatson | ready_jobs | 09:15 |
| cjwatson | So surely a job won't show up in iterReady() until the retry delay has elapsed. | 09:15 |
| wgrant | You are probably not wrong. | 09:16 |
| wgrant | It was all slightly strange, and I ripped out some strangeness a few years ago, but not all of it. | 09:16 |
| cjwatson | (I'm trying to fix the weird snap store upload retry behaviour. I have it mostly fixed, but it's a bit of a hairball.) | 09:16 |
| === frankban is now known as frankban|afk | ||
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!