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(a)redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com>
Gerrit-Reviewer: Federico Simoncelli <fsimonce(a)redhat.com>