Skip to content
Snippets Groups Projects
Commit 9911e9e6 authored by Sybren A. Stüvel's avatar Sybren A. Stüvel
Browse files

Use pid=%d instead of PID %r for consistency with other PID logging

parent e7d12ccf
No related branches found
No related tags found
No related merge requests found
...@@ -445,12 +445,12 @@ class AbstractSubprocessCommand(AbstractCommand): ...@@ -445,12 +445,12 @@ class AbstractSubprocessCommand(AbstractCommand):
return 'Empty PID file %s, refusing to create new subprocess just to be sure' % pidfile return 'Empty PID file %s, refusing to create new subprocess just to be sure' % pidfile
pid = int(pid_str) pid = int(pid_str)
self._log.warning('Found PID file %s with PID %r', pidfile, pid) self._log.warning('Found PID file %s with pid=%d', pidfile, pid)
try: try:
proc = psutil.Process(pid) proc = psutil.Process(pid)
except psutil.NoSuchProcess: except psutil.NoSuchProcess:
self._log.warning('Deleting pidfile %s for stale PID %r', pidfile, pid) self._log.warning('Deleting pidfile %s for stale pid=%d', pidfile, pid)
pidfile.unlink() pidfile.unlink()
return None return None
return 'Subprocess from %s is still running: %s' % (pidfile, proc) return 'Subprocess from %s is still running: %s' % (pidfile, proc)
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment