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

[PR #789/872f6e31 backport][stable-3] Update rds_instance_snaphot to use handlers defined in rds.py #1139

Conversation

patchback[bot]
Copy link

@patchback patchback bot commented May 9, 2022

This is a backport of PR #789 as merged into main (872f6e3).

SUMMARY
  • Update module to use handlers defined in rds.py
  • Enable check_mode
  • Add integration tests

Depends-On: ansible-collections/amazon.aws#553
Also requires: mattclay/aws-terminator#185

ISSUE TYPE
  • Feature Pull Request
COMPONENT NAME

rds_instance_snapshot

Update rds_instance_snaphot to use handlers defined in rds.py

SUMMARY

Update module to use handlers defined in rds.py
Enable check_mode
Add integration tests

Depends-On: ansible-collections/amazon.aws#553
Also requires: mattclay/aws-terminator#185

ISSUE TYPE

Feature Pull Request

COMPONENT NAME

rds_instance_snapshot

Reviewed-by: Jill R <None>
Reviewed-by: Alina Buzachis <None>
Reviewed-by: Mark Woolley <mw@marknet15.com>
Reviewed-by: Mark Chappell <None>
Reviewed-by: Markus Bergholz <git@osuv.de>
Reviewed-by: Joseph Torcasso <None>
(cherry picked from commit 872f6e3)
@ansibullbot
Copy link

@ansibullbot ansibullbot added community_review feature This issue/PR relates to a feature request integration tests/integration module module needs_triage new_contributor Help guide this first time contributor plugins plugin (any type) tests tests labels May 9, 2022
@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded.

ansible-galaxy-importer FAILURE in 4m 16s (non-voting)
✔️ build-ansible-collection SUCCESS in 6m 25s
✔️ ansible-test-sanity-docker-devel SUCCESS in 10m 44s (non-voting)
✔️ ansible-test-sanity-docker-milestone SUCCESS in 9m 43s
✔️ ansible-test-sanity-docker-stable-2.9 SUCCESS in 11m 05s
✔️ ansible-test-sanity-docker-stable-2.11 SUCCESS in 10m 21s
✔️ ansible-test-sanity-docker-stable-2.12 SUCCESS in 10m 02s
✔️ ansible-test-units-community-aws-python38 SUCCESS in 7m 16s
✔️ ansible-test-splitter SUCCESS in 2m 46s
✔️ integration-community.aws-1 SUCCESS in 29m 32s
⚠️ 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

@markuman markuman added the mergeit Merge the PR (SoftwareFactory) label May 9, 2022
@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded (gate pipeline).

ansible-galaxy-importer FAILURE in 4m 53s (non-voting)
✔️ build-ansible-collection SUCCESS in 5m 31s
✔️ ansible-test-sanity-docker-devel SUCCESS in 9m 52s (non-voting)
✔️ ansible-test-sanity-docker-milestone SUCCESS in 9m 50s
✔️ ansible-test-sanity-docker-stable-2.9 SUCCESS in 12m 02s
✔️ ansible-test-sanity-docker-stable-2.11 SUCCESS in 9m 55s
✔️ ansible-test-sanity-docker-stable-2.12 SUCCESS in 9m 36s
✔️ ansible-test-units-community-aws-python38 SUCCESS in 6m 08s
✔️ ansible-test-splitter SUCCESS in 2m 47s
✔️ integration-community.aws-1 SUCCESS in 24m 48s
⚠️ 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 0c3dcff into stable-3 May 9, 2022
@softwarefactory-project-zuul softwarefactory-project-zuul bot deleted the patchback/backports/stable-3/872f6e31ee822d09e62f2e24357a68e0387fa4e9/pr-789 branch May 9, 2022 19:04
abikouo added a commit to abikouo/community.aws that referenced this pull request Oct 24, 2023
…ble-collections#1139)

move get_s3_connection, reduce complexity and increase coverage

SUMMARY

get_s3_connection is duplicated into modules s3_object and s3_object_info, the goal of this pull request is to move to a single place, reduce code complexity and increase coverage

ISSUE TYPE


Feature Pull Request

COMPONENT NAME

ADDITIONAL INFORMATION

Reviewed-by: Mark Chappell <None>
Reviewed-by: Gonéri Le Bouder <goneri@lebouder.net>
Reviewed-by: Bikouo Aubin <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 integration tests/integration mergeit Merge the PR (SoftwareFactory) module module new_contributor Help guide this first time contributor plugins plugin (any type) tests tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants