maxresults in ec2_snapshot_info

[asifiqbal]

SUMMARY

Add maxresults (https://docs.aws.amazon.com/AWSEC2/latest/APIReference/API_DescribeSnapshots.html) to avoid filling up register. I have nearly 80k snapshots

ISSUE TYPE

• an attribute maxresults

COMPONENT NAME

ec2_snapshot_info

ADDITIONAL INFORMATION

I could use maxresults: 100 so the output can fit into my register: results

---
- hosts: localhost
  gather_facts: no

  tasks:
     - amazon.aws.ec2_snapshot_info:
             filters:
                "tag:project": myproject
             maxresults: 100
       register: results
     - debug: var=results
   

[ansibullbot]

Files identified in the description:

• [plugins/modules/ec2_snapshot_info.py](https://github.com/['ansible-collections/amazon.aws', 'ansible-collections/community.aws', 'ansible-collections/community.vmware']/blob/main/plugins/modules/ec2_snapshot_info.py)

If these files are inaccurate, please update the component name section of the description or use the !component bot command.

click here for bot help

[ansibullbot]

@asifiqbal: Greetings! Thanks for taking the time to open this issue. In order for the community to handle your issue effectively, we need a bit more information.

Here are the items we could not find in your description:

• ansible version

Please set the description of this issue with this template:
https://raw.githubusercontent.com/ansible/ansible/devel/.github/ISSUE_TEMPLATE/bug_report.md

click here for bot help

[ansibullbot]

cc @jillr @s-hertel @tremble @wimnat
click here for bot help

[asifiqbal]

ansible 2.9.13
config file = /data/asif/ansible/ansible.cfg
configured module search path = ['/home/acag-asif.iqbal/.ansible/plugins/modules', '/usr/share/ansible/plugins/modules']
ansible python module location = /usr/local/lib/python3.6/site-packages/ansible
executable location = /usr/local/bin/ansible
python version = 3.6.8 (default, Aug 13 2020, 07:46:32) [GCC 4.8.5 20150623 (Red Hat 4.8.5-39)]

[goneri]

This sounds like a good idea. @tremble, @jillr Do we have a standardized name of a key like this?

[tremble]

max_results would be consistent with our normal naming. However, you'd also want to include the next_token parameter.

[abikouo]

Hi @asifiqbal
Feature pull request has been merged.