-
Notifications
You must be signed in to change notification settings - Fork 403
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Implement the list_global_accelerators module #95
base: main
Are you sure you want to change the base?
Implement the list_global_accelerators module #95
Conversation
As a proof of concept for ansible-collections#94
The test
The test
|
Hi @TylerLubeck, thanks for this PR! This module should be named global_accelerators_info.py if it only provides information. It will also need tests. Integration tests will likely rely on the module in #97, to create the accelerator to be gathered by this module, so I think this PR should probably depend on that one. |
One comment in addition to @jillr's it would be good if the module supported describing a specific accelerator. Especially during integration tests this can be helpful for validating that you made the changes you thought you did. Only supporting the listing of all instances of an object can get very time/API-call expensive. (unfortunately boto3 doesn't support server-side filtering) |
@TylerLubeck do you need help to address these issues? Don't hesitate to ping us on IRC (#ansible-aws). |
* Enable all sanity tests. * Update 2.9 sanity ignores. The 2.9 pylint test does not like deprecation by date, so we have to ignore these 'errors'. * Allow 2.11 (and later) as well.
SUMMARY
I took a stab at a proof of concept for part of #94 - this implements the most basic
list_global_accelerators
module to fetch information about active global accelerators for an accountISSUE TYPE
COMPONENT NAME
list_global_accelerators
ADDITIONAL INFORMATION
The global accelerator API doesn't seem to allow for lookup by anything other than ARN, and doesn't allow for using ARNs to filter the pagination, which means that filtering has to be done client side and not server side. It's also highly possible that I'm just missing something in which case I would love to learn more!