From 8cb0a28607e7f8b9eb4bb71a95b39230d43a969c Mon Sep 17 00:00:00 2001
From: Stephen Finucane <sfinucan@redhat.com>
Date: Tue, 2 Nov 2021 09:59:27 +0000
Subject: compute: Don't warn if disk overcommit params unset

Due to a small logic error, we were emitting a warning about a
deprecated option when the user tried to live migrate an instance using
microversion 2.25 even though the user hadn't actually set that option.
Correct this.

Change-Id: Ib61e817bd4ced9b5533e7c7f9d8f0b45fe81c211
Signed-off-by: Stephen Finucane <sfinucan@redhat.com>
Story: 2009657
Task: 43836
---
 openstackclient/tests/unit/compute/v2/test_server.py | 20 ++++++++++----------
 1 file changed, 10 insertions(+), 10 deletions(-)

(limited to 'openstackclient/tests')

diff --git a/openstackclient/tests/unit/compute/v2/test_server.py b/openstackclient/tests/unit/compute/v2/test_server.py
index 3d8c17fd..285b2b41 100644
--- a/openstackclient/tests/unit/compute/v2/test_server.py
+++ b/openstackclient/tests/unit/compute/v2/test_server.py
@@ -4812,7 +4812,7 @@ class TestServerMigrate(TestServer):
         verifylist = [
             ('live_migration', False),
             ('block_migration', None),
-            ('disk_overcommit', False),
+            ('disk_overcommit', None),
             ('wait', False),
         ]
         parsed_args = self.check_parser(self.cmd, arglist, verifylist)
@@ -4834,7 +4834,7 @@ class TestServerMigrate(TestServer):
             ('live_migration', False),
             ('host', 'fakehost'),
             ('block_migration', None),
-            ('disk_overcommit', False),
+            ('disk_overcommit', None),
             ('wait', False),
         ]
         parsed_args = self.check_parser(self.cmd, arglist, verifylist)
@@ -4856,7 +4856,7 @@ class TestServerMigrate(TestServer):
         verifylist = [
             ('live_migration', False),
             ('block_migration', True),
-            ('disk_overcommit', False),
+            ('disk_overcommit', None),
             ('wait', False),
         ]
         parsed_args = self.check_parser(self.cmd, arglist, verifylist)
@@ -4897,7 +4897,7 @@ class TestServerMigrate(TestServer):
             ('live_migration', False),
             ('host', 'fakehost'),
             ('block_migration', None),
-            ('disk_overcommit', False),
+            ('disk_overcommit', None),
             ('wait', False),
         ]
         parsed_args = self.check_parser(self.cmd, arglist, verifylist)
@@ -4923,7 +4923,7 @@ class TestServerMigrate(TestServer):
             ('live_migration', True),
             ('host', None),
             ('block_migration', None),
-            ('disk_overcommit', False),
+            ('disk_overcommit', None),
             ('wait', False),
         ]
         parsed_args = self.check_parser(self.cmd, arglist, verifylist)
@@ -4947,7 +4947,7 @@ class TestServerMigrate(TestServer):
             ('live_migration', True),
             ('host', 'fakehost'),
             ('block_migration', None),
-            ('disk_overcommit', False),
+            ('disk_overcommit', None),
             ('wait', False),
         ]
         parsed_args = self.check_parser(self.cmd, arglist, verifylist)
@@ -4975,7 +4975,7 @@ class TestServerMigrate(TestServer):
             ('live_migration', True),
             ('host', 'fakehost'),
             ('block_migration', None),
-            ('disk_overcommit', False),
+            ('disk_overcommit', None),
             ('wait', False),
         ]
         parsed_args = self.check_parser(self.cmd, arglist, verifylist)
@@ -5002,7 +5002,7 @@ class TestServerMigrate(TestServer):
         verifylist = [
             ('live_migration', True),
             ('block_migration', True),
-            ('disk_overcommit', False),
+            ('disk_overcommit', None),
             ('wait', False),
         ]
         parsed_args = self.check_parser(self.cmd, arglist, verifylist)
@@ -5088,7 +5088,7 @@ class TestServerMigrate(TestServer):
         verifylist = [
             ('live_migration', False),
             ('block_migration', None),
-            ('disk_overcommit', False),
+            ('disk_overcommit', None),
             ('wait', True),
         ]
         parsed_args = self.check_parser(self.cmd, arglist, verifylist)
@@ -5108,7 +5108,7 @@ class TestServerMigrate(TestServer):
         verifylist = [
             ('live_migration', False),
             ('block_migration', None),
-            ('disk_overcommit', False),
+            ('disk_overcommit', None),
             ('wait', True),
         ]
         parsed_args = self.check_parser(self.cmd, arglist, verifylist)
-- 
cgit v1.2.1