Nir Soffer has uploaded a new change for review.

Change subject: monitor: Rename storage domain monitor threads
......................................................................

monitor: Rename storage domain monitor threads

These threads used the default names (Thread-N), making it hard to
locate a domain monitor thread for certain domain. Now the threads are
named "monitor/short-uuid". short-uuuid is the first 8 characters of the
storage domain uuid since system thread names are limited to 15
characters.

Change-Id: I86480d2205be602cbffe6c37acd63a67a4aa2c96
Signed-off-by: Nir Soffer <nsof...@redhat.com>
---
M vdsm/storage/monitor.py
1 file changed, 2 insertions(+), 1 deletion(-)


  git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/24/63524/1

diff --git a/vdsm/storage/monitor.py b/vdsm/storage/monitor.py
index 607f8af..4ffa7e5 100644
--- a/vdsm/storage/monitor.py
+++ b/vdsm/storage/monitor.py
@@ -252,7 +252,8 @@
 class MonitorThread(object):
 
     def __init__(self, sdUUID, hostId, interval, changeEvent, checker):
-        self.thread = concurrent.thread(self._run, logger=log.name)
+        self.thread = concurrent.thread(self._run, logger=log.name,
+                                        name="monitor/" + sdUUID[:7])
         self.stopEvent = threading.Event()
         self.domain = None
         self.sdUUID = sdUUID


-- 
To view, visit https://gerrit.ovirt.org/63524
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I86480d2205be602cbffe6c37acd63a67a4aa2c96
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Nir Soffer <nsof...@redhat.com>
_______________________________________________
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/admin/lists/vdsm-patches@lists.fedorahosted.org

Reply via email to