Nir Soffer has submitted this change and it was merged. Change subject: virt: vmchannels: switch to concurrent.thread() ......................................................................
virt: vmchannels: switch to concurrent.thread() Let's use concurrent.thread for vmchannels.Listener, in order to get rid of the inheritance from threading.Thread - which is frowned upon, and in order to have all the goodies that concurrent.thread() provides (e.g. system thread names). Change-Id: I6d1c566c87ffac0cbfb5cc2b22587baa10760cf3 Signed-off-by: Francesco Romani <[email protected]> Reviewed-on: https://gerrit.ovirt.org/61460 Reviewed-by: Nir Soffer <[email protected]> Continuous-Integration: Jenkins CI --- M lib/vdsm/virt/vmchannels.py 1 file changed, 8 insertions(+), 3 deletions(-) Approvals: Nir Soffer: Looks good to me, approved Jenkins CI: Passed CI tests Francesco Romani: Verified -- To view, visit https://gerrit.ovirt.org/61460 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I6d1c566c87ffac0cbfb5cc2b22587baa10760cf3 Gerrit-PatchSet: 8 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: Nir Soffer <[email protected]> Gerrit-Reviewer: Vinzenz Feenstra <[email protected]> Gerrit-Reviewer: gerrit-hooks <[email protected]> _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/admin/lists/[email protected]
