Francesco Romani has uploaded a new change for review.

Change subject: migration: deprecate != 'online' migrations
......................................................................

migration: deprecate != 'online' migrations

Since long time the only supported method for migration
is actually 'online'. In the API schema, the methods
are actually an enumeration with a single value ('online').

This patch adds an explicit deprecation reminder.

Change-Id: I40a9c9ab26d0bb4dd9e3a2ff449b318fa0e96f49
Signed-off-by: Francesco Romani <[email protected]>
---
M vdsm/virt/migration.py
1 file changed, 2 insertions(+), 0 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/55/29555/1

diff --git a/vdsm/virt/migration.py b/vdsm/virt/migration.py
index 528adf0..52e4892 100644
--- a/vdsm/virt/migration.py
+++ b/vdsm/virt/migration.py
@@ -62,6 +62,8 @@
         self._vm = vm
         self._dst = dst
         self._mode = mode
+        if method != METHOD_ONLINE:
+            self.log.warning('migration method %s is deprecated', method)
         self._method = method
         self._dstparams = dstparams
         self._machineParams = {}


-- 
To view, visit http://gerrit.ovirt.org/29555
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I40a9c9ab26d0bb4dd9e3a2ff449b318fa0e96f49
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <[email protected]>
_______________________________________________
vdsm-patches mailing list
[email protected]
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Reply via email to