Dan Kenigsberg has uploaded a new change for review.
Change subject: Reintroduce afterMigrationStatus
......................................................................
Reintroduce afterMigrationStatus
In
http://gerrit.ovirt.org/19306 I was fooled to believe that we can
drop the unused afterMigrationStatus parameter. However vdsm version
prior to that change (e.g. that of ovirt-3.3.0) expect to delete it.
Change-Id: Ibc55f7fc0d7debd1590011301c38565d27dc137e
Signed-off-by: Dan Kenigsberg <danken(a)redhat.com>
---
M vdsm/vm.py
1 file changed, 3 insertions(+), 0 deletions(-)
git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/59/19859/1
diff --git a/vdsm/vm.py b/vdsm/vm.py
index c375aec..bb4a7ec 100644
--- a/vdsm/vm.py
+++ b/vdsm/vm.py
@@ -285,6 +285,9 @@
new_drives.append(drive)
self._machineParams['drives'] = new_drives
+ # vdsm < 4.13 expect this to exist
+ self._machineParams['afterMigrationStatus'] = ''
+
def run(self):
try:
mstate = ''
--
To view, visit
http://gerrit.ovirt.org/19859
To unsubscribe, visit
http://gerrit.ovirt.org/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: Ibc55f7fc0d7debd1590011301c38565d27dc137e
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Dan Kenigsberg <danken(a)redhat.com>