diff options
author | Stephen Finucane <sfinucan@redhat.com> | 2022-12-19 16:08:27 +0000 |
---|---|---|
committer | Stephen Finucane <sfinucan@redhat.com> | 2022-12-19 17:02:48 +0000 |
commit | 004b2ab2fbad3655915f226ca2f479b35f7bfc46 (patch) | |
tree | 22998e0df4796a77d4858858773f50ede6e20dbf | |
parent | 29fafabeed24edc958ed8acf87f33d1ba03fb3bc (diff) | |
download | python-openstackclient-004b2ab2fbad3655915f226ca2f479b35f7bfc46.tar.gz |
compute: Switch server restore to SDK
Change-Id: I8df9711b736991c01136988aa06c8540d640f52f
Signed-off-by: Stephen Finucane <sfinucan@redhat.com>
Depends-On: https://review.opendev.org/c/openstack/openstacksdk/+/868108
-rw-r--r-- | openstackclient/compute/v2/server.py | 11 | ||||
-rw-r--r-- | openstackclient/tests/unit/compute/v2/test_server.py | 11 | ||||
-rw-r--r-- | releasenotes/notes/migrate-server-restore-to-sdk-4540f26753031779.yaml | 4 |
3 files changed, 13 insertions, 13 deletions
diff --git a/openstackclient/compute/v2/server.py b/openstackclient/compute/v2/server.py index 8496a13e..dbeaf513 100644 --- a/openstackclient/compute/v2/server.py +++ b/openstackclient/compute/v2/server.py @@ -4050,12 +4050,13 @@ class RestoreServer(command.Command): return parser def take_action(self, parsed_args): - compute_client = self.app.client_manager.compute + compute_client = self.app.client_manager.sdk_connection.compute for server in parsed_args.server: - utils.find_resource( - compute_client.servers, - server - ).restore() + server_id = compute_client.find_server( + server, + ignore_missing=False, + ).id + compute_client.restore_server(server_id) class ResumeServer(command.Command): diff --git a/openstackclient/tests/unit/compute/v2/test_server.py b/openstackclient/tests/unit/compute/v2/test_server.py index f4122162..bf738bfc 100644 --- a/openstackclient/tests/unit/compute/v2/test_server.py +++ b/openstackclient/tests/unit/compute/v2/test_server.py @@ -7499,21 +7499,16 @@ class TestServerRevertMigration(TestServer): class TestServerRestore(TestServer): def setUp(self): - super(TestServerRestore, self).setUp() + super().setUp() # Get the command object to test self.cmd = server.RestoreServer(self.app, None) - # Set methods to be tested. - self.methods = { - 'restore': None, - } - def test_server_restore_one_server(self): - self.run_method_with_servers('restore', 1) + self.run_method_with_sdk_servers('restore_server', 1) def test_server_restore_multi_servers(self): - self.run_method_with_servers('restore', 3) + self.run_method_with_sdk_servers('restore_server', 3) class TestServerResume(TestServer): diff --git a/releasenotes/notes/migrate-server-restore-to-sdk-4540f26753031779.yaml b/releasenotes/notes/migrate-server-restore-to-sdk-4540f26753031779.yaml new file mode 100644 index 00000000..1adc40be --- /dev/null +++ b/releasenotes/notes/migrate-server-restore-to-sdk-4540f26753031779.yaml @@ -0,0 +1,4 @@ +--- +features: + - | + The ``server restore`` command has been migrated to SDK. |