Allow to cancel a job or request an abort

[medihack]

Closes #511

Successful PR Checklist:

• [x] Tests
  • [ ] (not applicable?)
• [x] Documentation
  • [ ] (not applicable?)

PR label(s):

• [x] https://github.com/procrastinate-org/procrastinate/labels/PR%20type%3A%20breaking%20%F0%9F%92%A5
• [x] https://github.com/procrastinate-org/procrastinate/labels/PR%20type%3A%20feature%20%E2%AD%90%EF%B8%8F
• [ ] https://github.com/procrastinate-org/procrastinate/labels/PR%20type%3A%20bugfix%20%F0%9F%95%B5%EF%B8%8F
• [ ] https://github.com/procrastinate-org/procrastinate/labels/PR%20type%3A%20miscellaneous%20%F0%9F%91%BE
• [ ] https://github.com/procrastinate-org/procrastinate/labels/PR%20type%3A%20dependencies%20%F0%9F%A4%96
• [ ] https://github.com/procrastinate-org/procrastinate/labels/PR%20type%3A%20documentation%20%F0%9F%93%9A

The only thing it breaks is when using the shell that cancel will leave the job in cancelled status (before the change it was left with failed). But I see it more as a fix than a breaking change.

The design decisions should be clear from the documentation page.

There are maybe parts to improve (maybe the acceptance tests?). Providing and testing sync and async APIs is quite hard, and sometimes also quite ugly 🙄. And in the end, it was more additional code than I thought.

[github-actions[bot]]

Coverage report

Click to see where and how coverage changed

File	Statements	Missing	Coverage	Coverage (new stmts)	Lines missing
procrastinate
exceptions.py
job_context.py
jobs.py
manager.py
shell.py
testing.py
worker.py
Project Total

This report was generated by python-coverage-comment-action

[ewjoachim]

Congratulations :)

[medihack]

Congratulations :)

Thank you for the super nice review process and the welcoming manner. I'll integrate Procrastinate in our applications in the upcoming days, but I will soon be back to take care of #1083. I also open another issue to track and discuss a possible listen-notify abort feature on top of this one.