Change in mom[master]: guest manager, monitor: remove implicit start()

[alitke at redhat.com]

Adam Litke has submitted this change and it was merged.

Change subject: guest manager, monitor: remove implicit start()
......................................................................


guest manager, monitor: remove implicit start()

For classes which inherit from threading.Thread(),
implicit start() in __init__ is generally considered a bad idea.

This patch makes the start() explicit for better clarity.
Moreover, future patches want to control the actual starting
of threads, so this patch paves the road for them.

Change-Id: Ia06bc0d9db0f53ae4c50aa7d25956ec626d8a1cc
Signed-off-by: Francesco Romani <fromani at redhat.com>
Reviewed-on: https://gerrit.ovirt.org/31589
Reviewed-by: Adam Litke <alitke at redhat.com>
Reviewed-by: Jenkins CI
---
M mom/GuestManager.py
M mom/GuestMonitor.py
M mom/__init__.py
3 files changed, 2 insertions(+), 2 deletions(-)

Approvals:
  Adam Litke: Looks good to me, approved
  Jenkins CI: Looks good to me, but someone else must approve
  Francesco Romani: Verified



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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ia06bc0d9db0f53ae4c50aa7d25956ec626d8a1cc
Gerrit-PatchSet: 12
Gerrit-Project: mom
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani at redhat.com>
Gerrit-Reviewer: Adam Litke <alitke at redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: automation at ovirt.org