Francesco Romani has posted comments on this change.
Change subject: keep pauseCode when migrating paused VM
......................................................................
Patch Set 3:
(1 comment)
http://gerrit.ovirt.org/#/c/27801/3/vdsm/virt/vm.py
File vdsm/virt/vm.py:
Line 1952: self._ongoingCreations.release()
Line 1953: self.log.debug("_ongoingCreations released")
Line 1954:
Line 1955: migrating = 'migrationDest' in self.conf or \
Line 1956: 'restoreState' in self.conf
+1
That is exactly what I was thinking about, and that's
why I asked about the context of this patch. If we want to keep the pauseCode across a
migration, chances are to always put the status to UP can be troublesome, to say the
least.
I'm kinda brainstorming now, but I think your suggestion to set the status as the one
of the source VM could be a sensible move.
Line 1957: if migrating and self.lastStatus != vmstatus.DOWN:
Line 1958: self._waitForIncomingMigrationFinish()
Line 1959:
Line 1960: self.lastStatus = vmstatus.UP
--
To view, visit
http://gerrit.ovirt.org/27801
To unsubscribe, visit
http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Iead0697bbebba3f261040221b04cd3745d8ef036
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Arik Hadas <ahadas(a)redhat.com>
Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek(a)redhat.com>
Gerrit-Reviewer: Vinzenz Feenstra <vfeenstr(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes