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

Tagging - Add simple deprecations for purge_tags=False #1185

Conversation

tremble
Copy link
Contributor

@tremble tremble commented May 30, 2022

Depends-On: ansible-collections/amazon.aws#844

SUMMARY

Deprecate the use of purge_tags=False as a default

ISSUE TYPE
  • Feature Pull Request
COMPONENT NAME

plugins/modules/aws_acm.py
plugins/modules/route53_health_check.py
plugins/modules/route53_zone.py
plugins/modules/sqs_queue.py

ADDITIONAL INFORMATION

@tremble tremble force-pushed the tagging/deprecate_purge/simple branch from 15b8949 to 381d3cb Compare May 30, 2022 19:05
@ansibullbot ansibullbot added community_review feature This issue/PR relates to a feature request module module needs_triage owner_pr PR created by owner/maintainer plugins plugin (any type) labels May 30, 2022
@github-actions
Copy link

github-actions bot commented May 30, 2022

Docs Build 📝

Thank you for contribution!✨

This PR has been merged and your docs changes will be incorporated when they are next published.

@softwarefactory-project-zuul

This comment was marked as resolved.

@tremble tremble force-pushed the tagging/deprecate_purge/simple branch from 381d3cb to 3389e12 Compare May 30, 2022 19:44
@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded.

ansible-galaxy-importer FAILURE in 4m 23s (non-voting)
✔️ build-ansible-collection SUCCESS in 4m 46s
✔️ ansible-test-sanity-docker-devel SUCCESS in 9m 43s (non-voting)
✔️ ansible-test-sanity-docker-milestone SUCCESS in 10m 54s
✔️ ansible-test-sanity-docker-stable-2.9 SUCCESS in 14m 30s
✔️ ansible-test-sanity-docker-stable-2.11 SUCCESS in 9m 47s
✔️ ansible-test-sanity-docker-stable-2.12 SUCCESS in 10m 10s
✔️ ansible-test-units-community-aws-python38 SUCCESS in 7m 36s
✔️ ansible-test-splitter SUCCESS in 2m 41s
✔️ integration-community.aws-1 SUCCESS in 11m 57s
⚠️ integration-community.aws-2 SKIPPED
⚠️ integration-community.aws-3 SKIPPED
⚠️ integration-community.aws-4 SKIPPED
⚠️ integration-community.aws-5 SKIPPED
⚠️ integration-community.aws-6 SKIPPED
⚠️ integration-community.aws-7 SKIPPED
⚠️ integration-community.aws-8 SKIPPED
⚠️ integration-community.aws-9 SKIPPED
⚠️ integration-community.aws-10 SKIPPED
⚠️ integration-community.aws-11 SKIPPED
⚠️ integration-community.aws-12 SKIPPED
⚠️ integration-community.aws-13 SKIPPED

@tremble tremble added the mergeit Merge the PR (SoftwareFactory) label May 31, 2022
@softwarefactory-project-zuul

This comment was marked as outdated.

@tremble
Copy link
Contributor Author

tremble commented May 31, 2022

regate

@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded (gate pipeline).

ansible-galaxy-importer FAILURE in 4m 06s (non-voting)
✔️ build-ansible-collection SUCCESS in 5m 46s
✔️ ansible-test-sanity-docker-devel SUCCESS in 10m 36s (non-voting)
✔️ ansible-test-sanity-docker-milestone SUCCESS in 10m 30s
✔️ ansible-test-sanity-docker-stable-2.9 SUCCESS in 11m 42s
✔️ ansible-test-sanity-docker-stable-2.11 SUCCESS in 11m 02s
✔️ ansible-test-sanity-docker-stable-2.12 SUCCESS in 9m 59s
✔️ ansible-test-units-community-aws-python38 SUCCESS in 6m 13s
✔️ ansible-test-splitter SUCCESS in 2m 59s
✔️ integration-community.aws-1 SUCCESS in 14m 28s
⚠️ integration-community.aws-2 SKIPPED
⚠️ integration-community.aws-3 SKIPPED
⚠️ integration-community.aws-4 SKIPPED
⚠️ integration-community.aws-5 SKIPPED
⚠️ integration-community.aws-6 SKIPPED
⚠️ integration-community.aws-7 SKIPPED
⚠️ integration-community.aws-8 SKIPPED
⚠️ integration-community.aws-9 SKIPPED
⚠️ integration-community.aws-10 SKIPPED
⚠️ integration-community.aws-11 SKIPPED
⚠️ integration-community.aws-12 SKIPPED
⚠️ integration-community.aws-13 SKIPPED

@softwarefactory-project-zuul softwarefactory-project-zuul bot merged commit 42688f3 into ansible-collections:main May 31, 2022
softwarefactory-project-zuul bot pushed a commit that referenced this pull request Jun 23, 2022
[stable-3] Fixup short description for aws_acm

SUMMARY
Fixup short description for aws_acm
partial backport from #1185
ISSUE TYPE

Docs Pull Request

COMPONENT NAME
plugins/modules/aws_acm.py
ADDITIONAL INFORMATION
> (as opposed to >-) results in a weird extra new line in the generated docs

Reviewed-by: Alina Buzachis <None>
@tremble tremble deleted the tagging/deprecate_purge/simple branch July 7, 2022 19:26
abikouo pushed a commit to abikouo/community.aws that referenced this pull request Oct 24, 2023
…ctions#1185)

Tagging - Add simple deprecations for purge_tags=False

Depends-On: ansible-collections/amazon.aws#844
SUMMARY
Deprecate the use of purge_tags=False as a default
ISSUE TYPE

Feature Pull Request

COMPONENT NAME
plugins/modules/aws_acm.py
plugins/modules/route53_health_check.py
plugins/modules/route53_zone.py
plugins/modules/sqs_queue.py
ADDITIONAL INFORMATION

Reviewed-by: Markus Bergholz <git@osuv.de>
Reviewed-by: Alina Buzachis <None>

This commit was initially merged in https://github.com/ansible-collections/community.aws
See: ansible-collections@42688f3
abikouo pushed a commit to abikouo/community.aws that referenced this pull request Oct 24, 2023
rds_instance: split up the integration tests

Split the rds_instance target in smaller targets.

Reviewed-by: Alina Buzachis <None>
Reviewed-by: Gonéri Le Bouder <goneri@lebouder.net>
abikouo pushed a commit to abikouo/community.aws that referenced this pull request Oct 24, 2023
rds - add the engine_mode parameter

Depends-On: ansible-collections#1185
Add a new engine_mode parameter to rds_cluster and also adjust the test-suite follow-up to some recent API changes:

With the aurora engine, engine_mode default now on serverless.
We cannot adjust the endpoint_port of a serverless cluster.
aurora clusters are now encrypted by default (storage_encrypted)

The new engine_mode parameter also allows us to re-enable the  Aurora test of the rds_instance integration tests.

Reviewed-by: Gonéri Le Bouder <goneri@lebouder.net>
Reviewed-by: Alina Buzachis <None>
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 mergeit Merge the PR (SoftwareFactory) module module needs_triage owner_pr PR created by owner/maintainer plugins plugin (any type)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants