Support setting max_job_retries and retry_delay on a per-tube basis

[nesquena]

Support setting max_job_retries and retry_delay on a per-tube basis

[nesquena]

Yes if you finish and one or two others can run and give a thumbs up we are happy to merge this. Also it’d be great if you could add a section in the readme.

1. Pick the simplest worker to add tests for first
2. Yep I think that makes sense

I think you are on the right track but I’ll take another pass through later.

[nesquena]

Yeah I think you are right, would be great for this to be configurable per queue. Thanks for the suggestion!

[boncey]

I'm having a crack at implementing this.
I'd welcome any feedback on my first commit.

The parts I'm unsure about are twofold.

1. how to best test the work_one_job method.
   It seems to only be tested from each worker and I'm not sure which worker to test with.

2. There's a partial implementation of per-queue retry_limit but only for certain workers.
   I guess this would behave the same as the per-queue max_job_retries that would become part of my MR.
   Is that likely to be an issue? I don't really want to touch the individual workers if I can help it.

Any feedback welcome.
Am I on the right track?
Is this MR likely to be accepted?

Thanks.
Darren.

[boncey]

⬆️ PR raised.
Comments welcome!