Hm, Perhaps @razzi can take this one? We have to get him permissions on the build server. It looks like we might need a new group for that, in lieu of making in an ops admin.

Change 628913 had a related patch set uploaded (by Razzi; owner: Razzi):
[operations/debs/python-pid@master] Initial debianization of python-pid

https://gerrit.wikimedia.org/r/628913

Change 628913 merged by Ottomata:
[operations/debs/python-pid@master] Initial debianization of python-pid

https://gerrit.wikimedia.org/r/628913

Change 628924 had a related patch set uploaded (by Razzi; owner: Razzi):
[operations/debs/python-pid@debian] Initial debianization of python-pid

https://gerrit.wikimedia.org/r/628924

Change 628924 merged by Ottomata:
[operations/debs/python-pid@debian] Initial debianization of python-pid

https://gerrit.wikimedia.org/r/628924

Change 628929 had a related patch set uploaded (by Razzi; owner: Razzi):
[operations/puppet@production] Add python3-pid debian package to reportupdater

https://gerrit.wikimedia.org/r/628929

Change 628929 merged by Razzi:
[operations/puppet@production] Add python3-pid debian package to reportupdater

https://gerrit.wikimedia.org/r/628929

Can @mforns confirm that reportupdater can use this package as its?

Can @mforns confirm that reportupdater can use this package as its?

It works great! (tested the new code in an-launcher1002)

Now, one behavior changed with the new use of the pid library: When reportupdater is executed and there's already another instance of the same job running, the new execution crashes with the following exception:

Traceback (most recent call last):
  File "/usr/lib/python3/dist-packages/pid/base.py", line 198, in create
    self._flock(self.fh.fileno())
  File "/usr/lib/python3/dist-packages/pid/posix.py", line 22, in _flock
    fcntl.flock(self.fh.fileno(), fcntl.LOCK_EX | fcntl.LOCK_NB)
BlockingIOError: [Errno 11] Resource temporarily unavailable

Before, reportupdater would just log and exit. This situation can happen, if reportupdater is backfilling, and should theoretically not crash.
Will comment on that in the patch.