Skip to content

Commit 1a51a22

Browse files
authored
use %r instead of %d to log PID change (#722)
turns out PID is set to None in the beginning, leading to a crash when logging this message.
1 parent 6e82a8f commit 1a51a22

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

elasticapm/base.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -207,7 +207,7 @@ def start_threads(self):
207207
with self._thread_starter_lock:
208208
current_pid = os.getpid()
209209
if self._pid != current_pid:
210-
self.logger.debug("Detected PID change from %d to %d, starting threads", self._pid, current_pid)
210+
self.logger.debug("Detected PID change from %r to %r, starting threads", self._pid, current_pid)
211211
for manager_type, manager in self._thread_managers.items():
212212
self.logger.debug("Starting %s thread", manager_type)
213213
manager.start_thread()

0 commit comments

Comments
 (0)