Add @test.attr(type='slow') to slow tests to reduce test run time.
Currently, there are a few tests, predominantly compute and volume
tests, that run very slowly. These include tests like
test_revert_resize_server in compute which takes almost 4 minutes
to run as well as test_volume_backup_export which takes almost
30 seconds to run [0].
This patch adds @test.attr(type='slow') decorator above these
slow tests (tests whose run time is >= 30 seconds or very
close to 30 seconds).
This means that the regex in post_test_hook can be modified
to be '(?!.*\[.*\bslow\b.*\])(^patrole_tempest_plugin\.tests\.api)',
similar to how Tempest does it already [1].
[0] http://logs.openstack.org/38/443238/2/check/gate-tempest-dsvm-patrole-admin-ubuntu-xenial-nv/222bfda/console.html
[1] https://github.com/openstack/tempest/blob/master/tox.ini
Change-Id: Ib7bf72a9a6cef79d9d54b0b9dea53338f708ba30
Partially-Implements: blueprint decrease-tempest-gate-runtime
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 4b86210..ab75709 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
@@ -19,6 +19,7 @@
from tempest import config
from tempest.lib import decorators
from tempest.lib import exceptions as lib_exc
+from tempest import test
from patrole_tempest_plugin import rbac_rule_validation
from patrole_tempest_plugin.tests.api.compute import rbac_base
@@ -101,6 +102,7 @@
self.rbac_utils.switch_role(self, switchToRbacRole=True)
self._test_stop_server()
+ @test.attr(type='slow')
@rbac_rule_validation.action(
service="nova",
rule="os_compute_api:servers:start")
@@ -110,6 +112,7 @@
self.rbac_utils.switch_role(self, switchToRbacRole=True)
self._test_start_server()
+ @test.attr(type='slow')
@rbac_rule_validation.action(
service="nova",
rule="os_compute_api:servers:resize")
@@ -120,6 +123,7 @@
self.rbac_utils.switch_role(self, switchToRbacRole=True)
self._test_resize_server(self.flavor_ref_alt)
+ @test.attr(type='slow')
@rbac_rule_validation.action(
service="nova",
rule="os_compute_api:servers:revert_resize")
@@ -131,6 +135,7 @@
self.rbac_utils.switch_role(self, switchToRbacRole=True)
self._test_revert_resize_server()
+ @test.attr(type='slow')
@rbac_rule_validation.action(
service="nova",
rule="os_compute_api:servers:confirm_resize")
diff --git a/patrole_tempest_plugin/tests/api/compute/test_server_volume_attachments_rbac.py b/patrole_tempest_plugin/tests/api/compute/test_server_volume_attachments_rbac.py
index 4e85fbe..21137a7 100644
--- a/patrole_tempest_plugin/tests/api/compute/test_server_volume_attachments_rbac.py
+++ b/patrole_tempest_plugin/tests/api/compute/test_server_volume_attachments_rbac.py
@@ -17,6 +17,7 @@
from tempest import config
from tempest.lib.common.utils import test_utils
from tempest.lib import decorators
+from tempest import test
from patrole_tempest_plugin import rbac_rule_validation
from patrole_tempest_plugin.tests.api.compute import rbac_base
@@ -76,6 +77,7 @@
self.client.show_volume_attachment(
self.server['id'], attachment['id'])
+ @test.attr(type='slow')
@rbac_rule_validation.action(
service="nova",
rule="os_compute_api:os-volumes-attachments:update")
diff --git a/patrole_tempest_plugin/tests/api/volume/admin/test_volumes_backup_admin_rbac.py b/patrole_tempest_plugin/tests/api/volume/admin/test_volumes_backup_admin_rbac.py
index 468cb61..b801b67 100644
--- a/patrole_tempest_plugin/tests/api/volume/admin/test_volumes_backup_admin_rbac.py
+++ b/patrole_tempest_plugin/tests/api/volume/admin/test_volumes_backup_admin_rbac.py
@@ -15,6 +15,7 @@
from tempest import config
from tempest.lib import decorators
+from tempest import test
from patrole_tempest_plugin import rbac_rule_validation
from patrole_tempest_plugin.tests.api.volume import rbac_base
@@ -39,6 +40,7 @@
super(VolumesBackupsAdminRbacTest, cls).resource_setup()
cls.volume = cls.create_volume()
+ @test.attr(type='slow')
@rbac_rule_validation.action(service="cinder",
rule="backup:backup-export")
@decorators.idempotent_id('e984ec8d-e8eb-485c-98bc-f1856020303c')
@@ -50,6 +52,7 @@
self.backups_client.export_backup(
backup['id'])['backup-record']
+ @test.attr(type='slow')
@rbac_rule_validation.action(service="cinder",
rule="backup:backup-import")
@decorators.idempotent_id('1e70f039-4556-44cc-9cc1-edf2b7ed648b')