Merge "Fix list-all-project tests for pagination limit"
diff --git a/designate_tempest_plugin/tests/api/v2/test_recordset.py b/designate_tempest_plugin/tests/api/v2/test_recordset.py
index 70e74c6..96179f1 100644
--- a/designate_tempest_plugin/tests/api/v2/test_recordset.py
+++ b/designate_tempest_plugin/tests/api/v2/test_recordset.py
@@ -281,10 +281,14 @@
LOG.info('Re-Fetch Recordsets as Admin tenant for a Primary project '
'using "x-auth-all-projects" HTTP header.')
+ # Note: This is an all-projects list call, so other tests running
+ # in parallel will impact the list result set. Since the default
+ # pagination limit is only 20, we set a param limit of 1000 here.
primary_recordsets_ids = [
item['id'] for item in self.admin_client.list_recordset(
self.zone['id'],
- headers={'x-auth-all-projects': True})[1]['recordsets']]
+ headers={'x-auth-all-projects': True},
+ params={'limit': 1000})[1]['recordsets']]
for recordset_id in [body_pr_1['id'], body_pr_2['id']]:
self.assertIn(
@@ -702,8 +706,11 @@
project_ids_used = [
item['project_id'] for item in self._create_client_recordset(
['primary', 'alt']).values()]
+ # Note: This is an all-projects list call, so other tests running
+ # in parallel will impact the list result set. Since the default
+ # pagination limit is only 20, we set a param limit of 1000 here.
recordsets = self.admin_client.list_owned_recordsets(
- headers={'x-auth-all-projects': True})
+ headers={'x-auth-all-projects': True}, params={'limit': 1000})
LOG.info('Received by API recordsets are {} '.format(recordsets))
project_ids_api = set([item['project_id'] for item in recordsets])
for prj_id in project_ids_used:
diff --git a/designate_tempest_plugin/tests/api/v2/test_transfer_accepts.py b/designate_tempest_plugin/tests/api/v2/test_transfer_accepts.py
index e4f5658..7cfc4a9 100644
--- a/designate_tempest_plugin/tests/api/v2/test_transfer_accepts.py
+++ b/designate_tempest_plugin/tests/api/v2/test_transfer_accepts.py
@@ -196,11 +196,14 @@
# As Admin list all accepted zone transfers, expected:
# each previously transferred zone is listed.
+ # Note: This is an all-projects list call, so other tests running
+ # in parallel will impact the list result set. Since the default
+ # pagination limit is only 20, we set a param limit of 1000 here.
LOG.info('Use Admin client to list all "accepted zone transfers"')
admin_client_accept_ids = [
item['id'] for item in
self.admin_accept_client.list_transfer_accept(
- headers={'x-auth-all-projects': True})]
+ headers={'x-auth-all-projects': True}, params={'limit': 1000})]
for tr_id in transfer_request_ids:
self.assertIn(
tr_id, admin_client_accept_ids,
diff --git a/designate_tempest_plugin/tests/api/v2/test_transfer_request.py b/designate_tempest_plugin/tests/api/v2/test_transfer_request.py
index 666fd79..94a1480 100644
--- a/designate_tempest_plugin/tests/api/v2/test_transfer_request.py
+++ b/designate_tempest_plugin/tests/api/v2/test_transfer_request.py
@@ -228,9 +228,13 @@
LOG.info('List transfer_requests for all projects using Admin tenant '
'and "x-auth-all-projects" HTTP header.')
+ # Note: This is an all-projects list call, so other tests running
+ # in parallel will impact the list result set. Since the default
+ # pagination limit is only 20, we set a param limit of 1000 here.
request_ids = [
item['id'] for item in self.admin_client.list_transfer_requests(
- headers={'x-auth-all-projects': True})[1]['transfer_requests']]
+ headers={'x-auth-all-projects': True},
+ params={'limit': 1000})[1]['transfer_requests']]
for request_id in [primary_transfer_request['id'],
alt_transfer_request['id']]:
diff --git a/designate_tempest_plugin/tests/api/v2/test_zones.py b/designate_tempest_plugin/tests/api/v2/test_zones.py
index 78b8d4a..c2337fe 100644
--- a/designate_tempest_plugin/tests/api/v2/test_zones.py
+++ b/designate_tempest_plugin/tests/api/v2/test_zones.py
@@ -270,8 +270,12 @@
self.admin_client, admin_zone['id'], 'ACTIVE')
LOG.info('As admin user list all projects zones')
+ # Note: This is an all-projects list call, so other tests running
+ # in parallel will impact the list result set. Since the default
+ # pagination limit is only 20, we set a param limit of 1000 here.
body = self.admin_client.list_zones(
- headers={'x-auth-all-projects': True})[1]['zones']
+ headers={'x-auth-all-projects': True},
+ params={'limit': 1000})[1]['zones']
listed_zone_ids = [item['id'] for item in body]
LOG.info('Ensure the fetched response includes all zone '
diff --git a/designate_tempest_plugin/tests/api/v2/test_zones_exports.py b/designate_tempest_plugin/tests/api/v2/test_zones_exports.py
index ef9ba6d..a87e105 100644
--- a/designate_tempest_plugin/tests/api/v2/test_zones_exports.py
+++ b/designate_tempest_plugin/tests/api/v2/test_zones_exports.py
@@ -146,9 +146,13 @@
self.addCleanup(self.alt_client.delete_zone_export, alt_export['id'])
LOG.info('As admin user list zone exports for all projects')
+ # Note: This is an all-projects list call, so other tests running
+ # in parallel will impact the list result set. Since the default
+ # pagination limit is only 20, we set a param limit of 1000 here.
listed_exports_ids = [
item['id'] for item in self.admin_client.list_zone_exports(
- headers={'x-auth-all-projects': True})[1]['exports']]
+ headers={'x-auth-all-projects': True},
+ params={'limit': 1000})[1]['exports']]
LOG.info('Make sure that all previously created zone '
'export IDs are listed')
diff --git a/designate_tempest_plugin/tests/api/v2/test_zones_imports.py b/designate_tempest_plugin/tests/api/v2/test_zones_imports.py
index 1d5f89e..332899d 100644
--- a/designate_tempest_plugin/tests/api/v2/test_zones_imports.py
+++ b/designate_tempest_plugin/tests/api/v2/test_zones_imports.py
@@ -199,8 +199,12 @@
headers={'x-auth-all-projects': True}))
LOG.info('As Admin tenant list import zones for all projects')
+ # Note: This is an all-projects list call, so other tests running
+ # in parallel will impact the list result set. Since the default
+ # pagination limit is only 20, we set a param limit of 1000 here.
body = self.admin_client.list_zone_imports(headers={
- 'x-auth-all-projects': True})[1]['imports']
+ 'x-auth-all-projects': True},
+ params={'limit': 1000})[1]['imports']
LOG.info('Ensure the fetched response includes previously '
'created import ID')