Skip to content
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

Cleanup lambda_alias #396

Merged
merged 10 commits into from
Feb 19, 2021
Merged

Conversation

tremble
Copy link
Contributor

@tremble tremble commented Feb 7, 2021

SUMMARY
  • use AnsibleAWSModule.client
  • use fail_json_aws
  • Replace custom snake/camel conversion
  • Remove custom "AWSConnection" wrapper and pass AWSRetry wrapped boto3 client instead
  • Fix idempotency when description isn't set.
  • Don't throw an exception when attempting to create a new alias in check mode
  • Add integration tests
ISSUE TYPE
  • Feature Pull Request
COMPONENT NAME

lambda_alias

ADDITIONAL INFORMATION

depends on mattclay/aws-terminator#123

@ansibullbot
Copy link

@ansibullbot ansibullbot added feature This issue/PR relates to a feature request integration tests/integration module module needs_revision This PR fails CI tests or a maintainer has requested a review/revision of the PR needs_triage plugins plugin (any type) tests tests labels Feb 7, 2021
@tremble tremble changed the title Cleanup lambda_alias [Pending Policy] Cleanup lambda_alias Feb 10, 2021
@tremble tremble changed the title [Pending Policy] Cleanup lambda_alias Cleanup lambda_alias Feb 12, 2021
@ansibullbot ansibullbot added community_review and removed needs_revision This PR fails CI tests or a maintainer has requested a review/revision of the PR labels Feb 12, 2021
@tremble
Copy link
Contributor Author

tremble commented Feb 12, 2021

@jillr Not sure why rerunning the tests didn't originally work, I'm guessing IAM was being slow to roll out the new policy. I've rebased and repushed and that seems to have fixed the tests.

@jillr jillr merged commit 38709e4 into ansible-collections:main Feb 19, 2021
@tremble tremble deleted the cleanup/lambda_alias branch June 29, 2021 12:55
alinabuzachis pushed a commit to alinabuzachis/community.aws that referenced this pull request Jul 19, 2021
* Update lambda_alias to use AnsibleAWSModule.client

* Update lambda_alias to use fail_json_aws

* Replace custom snake/camel conversion

* lambda_alias replace use of AWSConnection with passing a standard (wrapped) boto3 connection

* Enable Retries

* Fix idempotency when description isn't set.

* Don't throw an exception when attempting to create a new alias in check mode

* Add revision_id to return docs

* Add integration tests

* add changelog
alinabuzachis pushed a commit to alinabuzachis/community.aws that referenced this pull request Jul 19, 2021
* Update lambda_alias to use AnsibleAWSModule.client

* Update lambda_alias to use fail_json_aws

* Replace custom snake/camel conversion

* lambda_alias replace use of AWSConnection with passing a standard (wrapped) boto3 connection

* Enable Retries

* Fix idempotency when description isn't set.

* Don't throw an exception when attempting to create a new alias in check mode

* Add revision_id to return docs

* Add integration tests

* add changelog
danielcotton pushed a commit to danielcotton/community.aws that referenced this pull request Nov 23, 2021
* Update lambda_alias to use AnsibleAWSModule.client

* Update lambda_alias to use fail_json_aws

* Replace custom snake/camel conversion

* lambda_alias replace use of AWSConnection with passing a standard (wrapped) boto3 connection

* Enable Retries

* Fix idempotency when description isn't set.

* Don't throw an exception when attempting to create a new alias in check mode

* Add revision_id to return docs

* Add integration tests

* add changelog
abikouo pushed a commit to abikouo/community.aws that referenced this pull request Oct 24, 2023
* Update lambda_alias to use AnsibleAWSModule.client

* Update lambda_alias to use fail_json_aws

* Replace custom snake/camel conversion

* lambda_alias replace use of AWSConnection with passing a standard (wrapped) boto3 connection

* Enable Retries

* Fix idempotency when description isn't set.

* Don't throw an exception when attempting to create a new alias in check mode

* Add revision_id to return docs

* Add integration tests

* add changelog

This commit was initially merged in https://github.com/ansible-collections/community.aws
See: ansible-collections@38709e4
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
community_review feature This issue/PR relates to a feature request integration tests/integration module module needs_triage plugins plugin (any type) tests tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants