Francesco Romani has posted comments on this change.

Change subject: virt: vmchannels: switch to concurrent.thread()
......................................................................


Patch Set 3:

(1 comment)

https://gerrit.ovirt.org/#/c/61460/3/lib/vdsm/virt/vmchannels.py
File lib/vdsm/virt/vmchannels.py:

Line 52:         self._update_lock = threading.Lock()
Line 53:         self._add_channels = {}
Line 54:         self._del_channels = []
Line 55:         self._timeout = None
Line 56:         self._thread = concurrent.thread(self.run)
> name='VM Channels Listener' missing
oops, fixing.
Line 57: 
Line 58:     def start(self):
Line 59:         self._thread.start()
Line 60: 


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I6d1c566c87ffac0cbfb5cc2b22587baa10760cf3
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <[email protected]>
Gerrit-Reviewer: Francesco Romani <[email protected]>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Martin Polednik <[email protected]>
Gerrit-Reviewer: Vinzenz Feenstra <[email protected]>
Gerrit-Reviewer: gerrit-hooks <[email protected]>
Gerrit-HasComments: Yes
_______________________________________________
vdsm-patches mailing list
[email protected]
https://lists.fedorahosted.org/admin/lists/[email protected]

Reply via email to