summaryrefslogtreecommitdiff
path: root/nova/tests/functional/test_servers.py
diff options
context:
space:
mode:
authorStephen Finucane <stephenfin@redhat.com>2020-07-15 14:52:11 +0100
committerStephen Finucane <stephenfin@redhat.com>2020-08-17 10:29:34 +0100
commit0f613248935a71a2057d37e759c3708a8efec121 (patch)
tree5fb42aeb2d340a327420370263472d5d4c5b870d /nova/tests/functional/test_servers.py
parent1c2cccab71def3ccf46e21fe14c0022686481d1a (diff)
downloadnova-0f613248935a71a2057d37e759c3708a8efec121.tar.gz
tests: Add helpers for suspend, resume and reboot of server
We will use this in a later test, but some tests that could have benefited from these are reworked now to validate the approach. Change-Id: I03bcd6753ed776d2ee216dcdca48514e5da8c43e Signed-off-by: Stephen Finucane <stephenfin@redhat.com>
Diffstat (limited to 'nova/tests/functional/test_servers.py')
-rw-r--r--nova/tests/functional/test_servers.py18
1 files changed, 3 insertions, 15 deletions
diff --git a/nova/tests/functional/test_servers.py b/nova/tests/functional/test_servers.py
index d85e148ee7..77d77ec368 100644
--- a/nova/tests/functional/test_servers.py
+++ b/nova/tests/functional/test_servers.py
@@ -344,14 +344,7 @@ class ServersTest(integrated_helpers._IntegratedTestBase):
'SOFT_DELETED')
# Create a second server
- server = self._build_server()
-
- created_server2 = self.api.post_server({'server': server})
- LOG.debug("created_server: %s", created_server2)
- self.assertTrue(created_server2['id'])
-
- # Wait for it to finish being created
- self._wait_for_state_change(created_server2, 'ACTIVE')
+ self._create_server()
# Try to restore the first server, it should fail
ex = self.assertRaises(client.OpenStackApiException,
@@ -8098,16 +8091,11 @@ class AcceleratorServerOpsTest(AcceleratorServerBase):
networks='none', expected_state='ACTIVE')
def test_soft_reboot_ok(self):
- params = {'reboot': {'type': 'SOFT'}}
- self.api.post_server_action(self.server['id'], params)
- self._wait_for_state_change(self.server, 'ACTIVE')
+ self._reboot_server(self.server)
self._check_allocations_usage(self.server)
def test_hard_reboot_ok(self):
- params = {'reboot': {'type': 'HARD'}}
- self.api.post_server_action(self.server['id'], params)
- self._wait_for_state_change(self.server, 'HARD_REBOOT')
- self._wait_for_state_change(self.server, 'ACTIVE')
+ self._reboot_server(self.server, hard=True)
self._check_allocations_usage(self.server)
def test_pause_unpause_ok(self):