Change in vdsm[master]: BZ#723579 AdvancedStatsThread should expose stop()

danken at redhat.com danken at redhat.com
Wed Aug 3 22:36:27 UTC 2011


Dan Kenigsberg has submitted this change and it was merged.

Change subject: BZ#723579 AdvancedStatsThread should expose stop()
......................................................................


BZ#723579 AdvancedStatsThread should expose stop()

The VmStatsThread wasn't exposing the stop method and therefore
it was never stopped during prepareForShutdown.
Even if it was declared as 'daemon' thread it kept the rest of
the application running satisfying its requests.

Change-Id: If8501b5286394e4e862e4bae965431cd395101fd
---
M vdsm/utils.py
1 file changed, 5 insertions(+), 5 deletions(-)

Approvals:
  Dan Kenigsberg: Looks good to me, approved
  David Naori: Verified


--
To view, visit http://gerrit.usersys.redhat.com/784
To unsubscribe, visit http://gerrit.usersys.redhat.com/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: If8501b5286394e4e862e4bae965431cd395101fd
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Federico Simoncelli <fsimonce at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: David Naori <dnaori at redhat.com>


More information about the vdsm-patches mailing list