From c5f29d683315e1afbe944aca8b3ae819de74d286 Mon Sep 17 00:00:00 2001 From: zhangbailin Date: Fri, 10 Apr 2020 08:39:35 +0800 Subject: [PATCH] FUP: Add volume-update CLI pre V285 tests This commit mainly to fix some comments from [1]. [1]https://review.opendev.org/#/c/712651/19/novaclient/tests/unit/v2/test_shell.py@4001 Blueprint: destroy-instance-with-datavolume Change-Id: Id809f22d0da2cdedf33a2c0df202f3953fd01673 --- doc/source/cli/nova.rst | 4 +++- novaclient/tests/unit/v2/test_shell.py | 12 ++++++++++++ 2 files changed, 15 insertions(+), 1 deletion(-) diff --git a/doc/source/cli/nova.rst b/doc/source/cli/nova.rst index 6db1cd0cb..d661599ed 100644 --- a/doc/source/cli/nova.rst +++ b/doc/source/cli/nova.rst @@ -3966,11 +3966,13 @@ new volume. ``--delete-on-termination`` Specify that the volume should be deleted when the server is destroyed. + It is mutually exclusive with '--no-delete-on-termination'. (Supported by API versions '2.85' - '2.latest') ``--no-delete-on-termination`` Specify that the attached volume should not be deleted when - the server is destroyed. (Supported by API versions '2.85' - '2.latest') + the server is destroyed. It is mutually exclusive with '--delete-on-termination'. + (Supported by API versions '2.85' - '2.latest') .. _nova_bash-completion: diff --git a/novaclient/tests/unit/v2/test_shell.py b/novaclient/tests/unit/v2/test_shell.py index d21252bd3..6aa37f7d4 100644 --- a/novaclient/tests/unit/v2/test_shell.py +++ b/novaclient/tests/unit/v2/test_shell.py @@ -4006,6 +4006,18 @@ class ShellTest(utils.TestCase): self.assert_called('PUT', '/servers/1234/os-volume_attachments/Work', {'volumeAttachment': {'volumeId': 'Work'}}) + def test_volume_update_delete_on_termination_pre_v285(self): + self.assertRaises( + SystemExit, self.run_command, + 'volume-update sample-server --delete-on-termination Work Work', + api_version='2.84') + + def test_volume_update_no_delete_on_termination_pre_v285(self): + self.assertRaises( + SystemExit, self.run_command, + 'volume-update sample-server --no-delete-on-termination Work Work', + api_version='2.84') + def test_volume_update_v285(self): self.run_command('volume-update sample-server --delete-on-termination ' 'Work Work', api_version='2.85')