Change in vdsm[master]: Remove check for incoming migration in _highWrite
Federico Simoncelli
fsimonce at redhat.com
Wed Jul 13 16:31:53 UTC 2011
Federico Simoncelli has posted comments on this change.
Change subject: Remove check for incoming migration in _highWrite
......................................................................
Patch Set 1: (3 inline comments)
....................................................
File vdsm/libvirtvm.py
Line 54: def _highWrite(self):
Removing this check should be safe because we start the VmStatsThread when the migration is finished. See below.
Line 915: self.log.debug(domxml)
If this is the migration destination we are exiting here and we don't execute _domDependentInit (the VmStatsThread starter).
Line 999: hooks.after_vm_migrate_destination(self._dom.XMLDesc(0), self.conf)
The _domDependentInit (the VmStatsThread starter) is executed now when the migration is completed.
--
To view, visit http://gerrit.usersys.redhat.com/706
To unsubscribe, visit http://gerrit.usersys.redhat.com/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I1c27d429b1c610a5a2d6d1c5b342e0564673c315
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: Federico Simoncelli <fsimonce at redhat.com>
More information about the vdsm-patches
mailing list