From 9911e9e6c22e2ebbd3f775df9706c1f2f8f39f54 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Sybren=20A=2E=20St=C3=BCvel?= <sybren@stuvel.eu>
Date: Thu, 15 Nov 2018 14:44:15 +0100
Subject: [PATCH] Use pid=%d instead of PID %r for consistency with other PID
logging
---
flamenco_worker/commands.py | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/flamenco_worker/commands.py b/flamenco_worker/commands.py
index 31d201ce..4bf208bd 100644
--- a/flamenco_worker/commands.py
+++ b/flamenco_worker/commands.py
@@ -445,12 +445,12 @@ class AbstractSubprocessCommand(AbstractCommand):
return 'Empty PID file %s, refusing to create new subprocess just to be sure' % pidfile
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:
proc = psutil.Process(pid)
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()
return None
return 'Subprocess from %s is still running: %s' % (pidfile, proc)
--
GitLab