WIP: Check only one runner

[davidwaroquiers]

This PR adds info to the auxiliary collection when the runner is started. The idea would be to warn the user that there may be another runner running already, in which case bad things could happen.

Currently just inserts the info when starting the runner and removes it when stopping it.

Still to be done:

• Deal with the locked document (if two people try to start the runner at the exact same time, guess it would not be very common but still)
• Deal with the specific goal of this PR: what do we do when this happens
• Deal with when the document with the runner info in the auxiliary collection is not present

[codecov-commenter]

Codecov Report

Attention: Patch coverage is 76.98745% with 110 lines in your changes missing coverage. Please review.

Project coverage is 73.65%. Comparing base (d2d5bc7) to head (8fc6704).

Files with missing lines	Patch %	Lines
src/jobflow_remote/jobs/daemon.py	80.00%	22 Missing and 16 partials ⚠️
src/jobflow_remote/jobs/jobcontroller.py	66.21%	24 Missing and 1 partial ⚠️
src/jobflow_remote/cli/runner.py	57.14%	9 Missing and 3 partials ⚠️
src/jobflow_remote/jobs/upgrade.py	87.91%	7 Missing and 4 partials ⚠️
src/jobflow_remote/cli/admin.py	80.43%	5 Missing and 4 partials ⚠️
src/jobflow_remote/cli/utils.py	69.56%	2 Missing and 5 partials ⚠️
src/jobflow_remote/utils/db.py	50.00%	7 Missing ⚠️
src/jobflow_remote/cli/formatting.py	91.66%	1 Missing ⚠️

Additional details and impacted files

@@             Coverage Diff             @@
##           develop     #150      +/-   ##
===========================================
+ Coverage    72.28%   73.65%   +1.37%     
===========================================
  Files           45       46       +1     
  Lines         5671     6070     +399     
  Branches       890      953      +63     
===========================================
+ Hits          4099     4471     +372     
+ Misses        1260     1252       -8     
- Partials       312      347      +35     

Files with missing lines	Coverage Δ
src/jobflow_remote/jobs/runner.py	78.85% <ø> (ø)
src/jobflow_remote/cli/formatting.py	85.82% <91.66%> (-0.01%)	⬇️
src/jobflow_remote/cli/utils.py	63.75% <69.56%> (+0.18%)	⬆️
src/jobflow_remote/utils/db.py	77.27% <50.00%> (-1.86%)	⬇️
src/jobflow_remote/cli/admin.py	87.87% <80.43%> (-3.99%)	⬇️
src/jobflow_remote/jobs/upgrade.py	87.91% <87.91%> (ø)
src/jobflow_remote/cli/runner.py	64.90% <57.14%> (+35.30%)	⬆️
src/jobflow_remote/jobs/jobcontroller.py	78.35% <66.21%> (-0.88%)	⬇️
src/jobflow_remote/jobs/daemon.py	67.89% <80.00%> (+11.76%)	⬆️

[davidwaroquiers]

Hi @gpetretto

Still a few things to be done but I think before going on, it would be better to have the expert's eye on this ;)

[gpetretto]

I have added some more comments to the latest changes

[davidwaroquiers]

I think it's all good. I did not test myself the functionality though. I would say it might be something to mention in the next release that if people encounter "weird things" when starting/stopping, they should open an issue. What do you think ?
I can't approve my own PR even if it has more become yours ;-)

[davidwaroquiers]

Added last few comments or suggestions.

[davidwaroquiers]

All good! Can't approve as it was initially my PR. Bit approved 😀