Change in vdsm[master]: lib: executor: log pool status on discard
fromani at redhat.com
fromani at redhat.com
Tue Nov 10 09:25:16 UTC 2015
Francesco Romani has uploaded a new change for review.
Change subject: lib: executor: log pool status on discard
......................................................................
lib: executor: log pool status on discard
Log the pool size and content when a worker is discarded,
to make troubleshooting and debug easier.
Change-Id: I949181968a97a7bcec703bc36ef9e1a0f30c6858
Signed-off-by: Francesco Romani <fromani at redhat.com>
---
M lib/vdsm/executor.py
1 file changed, 9 insertions(+), 2 deletions(-)
git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/33/48333/1
diff --git a/lib/vdsm/executor.py b/lib/vdsm/executor.py
index 1ce14bc..82c8d8c 100644
--- a/lib/vdsm/executor.py
+++ b/lib/vdsm/executor.py
@@ -127,6 +127,11 @@
with self._lock:
if self._running:
self._add_worker()
+ # this is a debug aid, it is not that important to be precise;
+ # intentionally done outside the lock
+ self._log.info('pool[%i]: (%s)',
+ len(self._workers),
+ ' '.join(str(w) for w in self._workers))
def _worker_stopped(self, worker):
"""
@@ -222,10 +227,12 @@
if self._discarded:
raise AssertionError("Attempt to discard worker twice")
self._discarded = True
- self._log.debug("Worker %s discarded while doing '%s'",
- self.name, self._task.name)
+ self._log.debug("Worker %s discarded", str(self))
self._executor._worker_discarded(self)
+ def __str__(self):
+ return '%s on %s' % (self.name, self._task.name)
+
class TaskQueue(object):
"""
--
To view, visit https://gerrit.ovirt.org/48333
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I949181968a97a7bcec703bc36ef9e1a0f30c6858
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani at redhat.com>
More information about the vdsm-patches
mailing list