-
Notifications
You must be signed in to change notification settings - Fork 350
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
new modules: lambda_layer and lambda_layer_info #1095
new modules: lambda_layer and lambda_layer_info #1095
Conversation
Docs Build 📝Thank you for contribution!✨ This PR has been merged and your docs changes will be incorporated when they are next published. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please see https://ansible-collections.github.io/amazon.aws/branch/main/collections/amazon/aws/docsite/dev_guidelines.html#api-throttling-rate-limiting-and-pagination for more information about using AWSRetry.
I'm not a big fan of unit tests which just call the "main" function (execute_module). I don't see a reason why we can't do integration tests here. (unlike some of the IAM/CloudWatch modules where there are security concerns). Tests like this will just be very fragile and need to be re-written with any significant change to the module.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Module code LGTM, if @goneri has time I'd appreciate his review of the unit tests because I know he's working on the plans for that
…tions (#1118) lambda - add support for layers when creating or updating lambda functions Depends-On: #1095 SUMMARY add the ability for module lambda to create/update a lambda function with layers attached ISSUE TYPE Feature Pull Request COMPONENT NAME lambda ADDITIONAL INFORMATION - lambda: name: test-func runtime: python3.7 (...) layers: - layer_version_arn: 'arn:aws:lambda:us-east-1:123456789012:layer:python27-env:7' - layer_name: pycustom-library version: 1 Reviewed-by: Alina Buzachis <None> Reviewed-by: Bikouo Aubin <None>
…tions (ansible-collections#1118) lambda - add support for layers when creating or updating lambda functions Depends-On: ansible-collections#1095 SUMMARY add the ability for module lambda to create/update a lambda function with layers attached ISSUE TYPE Feature Pull Request COMPONENT NAME lambda ADDITIONAL INFORMATION - lambda: name: test-func runtime: python3.7 (...) layers: - layer_version_arn: 'arn:aws:lambda:us-east-1:123456789012:layer:python27-env:7' - layer_name: pycustom-library version: 1 Reviewed-by: Alina Buzachis <None> Reviewed-by: Bikouo Aubin <None>
…1095) new modules: lambda_layer and lambda_layer_info SUMMARY The module lambda_layer allows user to publish or delete lambda layer version, while lambda_layer_info is used to list lambda layers or all versions of a specific lambda layer ISSUE TYPE New Module Pull Request COMPONENT NAME lambda_layer lambda_layer_info ADDITIONAL INFORMATION Closes: ansible-collections#1116 Reviewed-by: Mark Chappell <None> Reviewed-by: Bikouo Aubin <None> Reviewed-by: Jill R <None> Reviewed-by: Alina Buzachis <None> Reviewed-by: Gonéri Le Bouder <goneri@lebouder.net>
…_info (#1095) (#1495) [manual backport stable-5] new modules: lambda_layer and lambda_layer_info (#1095) new modules: lambda_layer and lambda_layer_info SUMMARY The module lambda_layer allows user to publish or delete lambda layer version, while lambda_layer_info is used to list lambda layers or all versions of a specific lambda layer ISSUE TYPE New Module Pull Request COMPONENT NAME lambda_layer lambda_layer_info ADDITIONAL INFORMATION Closes: #1116 Reviewed-by: Mark Chappell Reviewed-by: Bikouo Aubin Reviewed-by: Jill R Reviewed-by: Alina Buzachis Reviewed-by: Gonéri Le Bouder goneri@lebouder.net SUMMARY ISSUE TYPE Bugfix Pull Request Docs Pull Request Feature Pull Request New Module Pull Request COMPONENT NAME ADDITIONAL INFORMATION Reviewed-by: Bikouo Aubin
…tions (#1118) (#1492) [manual backport stable-5] lambda - add support for layers when creating or updating lambda functions (#1118) lambda - add support for layers when creating or updating lambda functions Depends-On: #1095 SUMMARY add the ability for module lambda to create/update a lambda function with layers attached ISSUE TYPE Feature Pull Request COMPONENT NAME lambda ADDITIONAL INFORMATION lambda: name: test-func runtime: python3.7 (...) layers: - layer_version_arn: 'arn:aws:lambda:us-east-1:123456789012:layer:python27-env:7' - layer_name: pycustom-library version: 1 Reviewed-by: Alina Buzachis Reviewed-by: Bikouo Aubin SUMMARY ISSUE TYPE Bugfix Pull Request Docs Pull Request Feature Pull Request New Module Pull Request COMPONENT NAME ADDITIONAL INFORMATION Reviewed-by: Mark Chappell
SUMMARY
The module
lambda_layer
allows user to publish or delete lambda layer version, whilelambda_layer_info
is used to list lambda layers or all versions of a specific lambda layerISSUE TYPE
COMPONENT NAME
lambda_layer
lambda_layer_info
ADDITIONAL INFORMATION
Closes: #1116