Merge "Compute API Server Actions Test"
diff --git a/patrole_tempest_plugin/tests/api/compute/test_server_actions_rbac.py b/patrole_tempest_plugin/tests/api/compute/test_server_actions_rbac.py
index c4b44e7..4b86210 100644
--- a/patrole_tempest_plugin/tests/api/compute/test_server_actions_rbac.py
+++ b/patrole_tempest_plugin/tests/api/compute/test_server_actions_rbac.py
@@ -13,9 +13,12 @@
# License for the specific language governing permissions and limitations
# under the License.
+import testtools
+
from tempest.common import waiters
from tempest import config
from tempest.lib import decorators
+from tempest.lib import exceptions as lib_exc
from patrole_tempest_plugin import rbac_rule_validation
from patrole_tempest_plugin.tests.api.compute import rbac_base
@@ -25,10 +28,6 @@
class ServerActionsRbacTest(rbac_base.BaseV2ComputeRbacTest):
- def tearDown(self):
- self.rbac_utils.switch_role(self, switchToRbacRole=False)
- super(ServerActionsRbacTest, self).tearDown()
-
@classmethod
def setup_clients(cls):
super(ServerActionsRbacTest, cls).setup_clients()
@@ -37,13 +36,6 @@
@classmethod
def skip_checks(cls):
super(ServerActionsRbacTest, cls).skip_checks()
- if not CONF.compute_feature_enabled.api_extensions:
- raise cls.skipException(
- '%s skipped as no compute extensions enabled' % cls.__name__)
- if not CONF.compute_feature_enabled.interface_attach:
- raise cls.skipException(
- '%s skipped as interface attachment is not available'
- % cls.__name__)
@classmethod
def resource_setup(cls):
@@ -51,6 +43,30 @@
super(ServerActionsRbacTest, cls).resource_setup()
cls.server_id = cls.create_test_server(wait_until='ACTIVE',
validatable=True)['id']
+ cls.flavor_ref = CONF.compute.flavor_ref
+ cls.flavor_ref_alt = CONF.compute.flavor_ref_alt
+ cls.image_ref = CONF.compute.image_ref
+
+ def setUp(self):
+ super(ServerActionsRbacTest, self).setUp()
+ try:
+ waiters.wait_for_server_status(self.client,
+ self.server_id, 'ACTIVE')
+ except lib_exc.NotFound:
+ # if the server was found to be deleted by a previous test,
+ # a new one is built
+ server = self.create_test_server(
+ validatable=True,
+ wait_until='ACTIVE')
+ self.__class__.server_id = server['id']
+ except Exception:
+ # Rebuilding the server in case something happened during a test
+ self.__class__.server_id = self.rebuild_server(
+ self.server_id, validatable=True)
+
+ def tearDown(self):
+ self.rbac_utils.switch_role(self, switchToRbacRole=False)
+ super(ServerActionsRbacTest, self).tearDown()
def _test_start_server(self):
self.client.start_server(self.server_id)
@@ -62,6 +78,21 @@
waiters.wait_for_server_status(self.client, self.server_id,
'SHUTOFF')
+ def _test_resize_server(self, flavor):
+ self.client.resize_server(self.server_id, flavor)
+ waiters.wait_for_server_status(self.client, self.server_id,
+ 'VERIFY_RESIZE')
+
+ def _test_revert_resize_server(self):
+ self.client.revert_resize_server(self.server_id)
+ waiters.wait_for_server_status(self.client, self.server_id,
+ 'ACTIVE')
+
+ def _test_confirm_resize_server(self):
+ self.client.confirm_resize_server(self.server_id)
+ waiters.wait_for_server_status(self.client, self.server_id,
+ 'ACTIVE')
+
@rbac_rule_validation.action(
service="nova",
rule="os_compute_api:servers:stop")
@@ -78,3 +109,59 @@
self._test_stop_server()
self.rbac_utils.switch_role(self, switchToRbacRole=True)
self._test_start_server()
+
+ @rbac_rule_validation.action(
+ service="nova",
+ rule="os_compute_api:servers:resize")
+ @decorators.idempotent_id('0546fbdd-2d8f-4ce8-ac00-f1e2129d0765')
+ @testtools.skipUnless(CONF.compute_feature_enabled.resize,
+ 'Resize is not available.')
+ def test_resize_server(self):
+ self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self._test_resize_server(self.flavor_ref_alt)
+
+ @rbac_rule_validation.action(
+ service="nova",
+ rule="os_compute_api:servers:revert_resize")
+ @decorators.idempotent_id('d41b64b8-a72d-414a-a4c5-94e1eb5e5a96')
+ @testtools.skipUnless(CONF.compute_feature_enabled.resize,
+ 'Resize is not available.')
+ def test_revert_resize_server(self):
+ self._test_resize_server(self.flavor_ref_alt)
+ self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self._test_revert_resize_server()
+
+ @rbac_rule_validation.action(
+ service="nova",
+ rule="os_compute_api:servers:confirm_resize")
+ @decorators.idempotent_id('f51620cb-dfcb-4e5d-b421-2e0edaa1316e')
+ @testtools.skipUnless(CONF.compute_feature_enabled.resize,
+ 'Resize is not available.')
+ def test_confirm_resize_server(self):
+ self._test_resize_server(self.flavor_ref_alt)
+ self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.addCleanup(
+ lambda: (self._test_resize_server(self.flavor_ref),
+ self._test_confirm_resize_server())
+ )
+ self._test_confirm_resize_server()
+
+ @rbac_rule_validation.action(
+ service="nova",
+ rule="os_compute_api:servers:rebuild")
+ @decorators.idempotent_id('54b1a30b-c96c-472c-9c83-ccaf6ec7e20b')
+ def test_rebuild_server(self):
+ self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.client.rebuild_server(self.server_id, self.image_ref)
+ waiters.wait_for_server_status(self.client, self.server_id,
+ 'ACTIVE')
+
+ @rbac_rule_validation.action(
+ service="nova",
+ rule="os_compute_api:servers:reboot")
+ @decorators.idempotent_id('19f27856-56e1-44f8-8615-7257f6b85cbb')
+ def test_reboot_server(self):
+ self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.client.reboot_server(self.server_id, type='HARD')
+ waiters.wait_for_server_status(self.client, self.server_id,
+ 'ACTIVE')