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