[PR #1384/569fff4c backport][stable-4] route53_info - fix max_items when not paginating

changelogs/fragments/1384-route53_info-fix-max-items.yml

@@ -0,0 +1,2 @@
+bugfixes:
+- route53_info - fix ``max_items`` parameter when used with non-paginated commands (https://github.com/ansible-collections/community.aws/issues/1383).

plugins/modules/route53_info.py

@@ -519,11 +519,8 @@ def reusable_delegation_set_details():
     params = dict()
 
     if not module.params.get('delegation_set_id'):
-        # Set PaginationConfig with max_items
         if module.params.get('max_items'):
-            params['PaginationConfig'] = dict(
-                MaxItems=module.params.get('max_items')
-            )
+            params['MaxItems'] = str(module.params.get('max_items'))
 
         if module.params.get('next_marker'):
             params['Marker'] = module.params.get('next_marker')
@@ -581,11 +578,8 @@ def list_hosted_zones_by_name():
     if module.params.get('dns_name'):
         params['DNSName'] = module.params.get('dns_name')
 
-    # Set PaginationConfig with max_items
     if module.params.get('max_items'):
-        params['PaginationConfig'] = dict(
-            MaxItems=module.params.get('max_items')
-        )
+        params['MaxItems'] = str(module.params.get('max_items'))
 
     return client.list_hosted_zones_by_name(**params)
 

tests/integration/targets/route53/tasks/main.yml

@@ -79,6 +79,22 @@
         - hosted_zones.HostedZone.ResourceRecordSetCount == 2
         - hosted_zones.HostedZone.Config.PrivateZone
 
+# Needs CI permissions updated
+#  # Ensure that we can use the non-paginated list_by_name method with max_items
+#  - name: Get zone 1 details only
+#    route53_info:
+#      query: hosted_zone
+#      hosted_zone_method: list_by_name
+#      dns_name: '{{ zone_one }}'
+#      max_items: 1
+#    register: list_by_name_result
+#
+#  - name: Assert that we found exactly one zone when querying by name
+#    assert:
+#      that:
+#        - list_by_name_result.HostedZones | length == 1
+#        - list_by_name_result.HostedZones[0].Name == '{{ zone_one }}'
+
   - name: 'Create A record using zone fqdn'
     route53:
       state: present