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 #1105/77cb0974 backport][stable-3] rds_instance - add deletion_protection param #1138

Conversation

patchback[bot]
Copy link

@patchback patchback bot commented May 9, 2022

This is a backport of PR #1105 as merged into main (77cb097).

Depends-On: #1116

SUMMARY

Fixes #922

ISSUE TYPE
  • Feature Pull Request
COMPONENT NAME

rds_instance

rds_instance - add deletion_protection param

Depends-On: #1116
SUMMARY
Fixes #922
ISSUE TYPE

Feature Pull Request

COMPONENT NAME
rds_instance

Reviewed-by: Alina Buzachis <None>
Reviewed-by: Markus Bergholz <[email protected]>
(cherry picked from commit 77cb097)
@ansibullbot
Copy link

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

Build succeeded.

ansible-galaxy-importer FAILURE in 5m 36s (non-voting)
✔️ build-ansible-collection SUCCESS in 4m 22s
✔️ ansible-test-sanity-docker-devel SUCCESS in 9m 41s (non-voting)
✔️ ansible-test-sanity-docker-milestone SUCCESS in 11m 05s
✔️ ansible-test-sanity-docker-stable-2.9 SUCCESS in 11m 13s
✔️ ansible-test-sanity-docker-stable-2.11 SUCCESS in 10m 10s
✔️ ansible-test-sanity-docker-stable-2.12 SUCCESS in 9m 46s
✔️ ansible-test-units-community-aws-python38 SUCCESS in 6m 02s
✔️ ansible-test-splitter SUCCESS in 3m 04s
✔️ integration-community.aws-1 SUCCESS in 44m 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 50s (non-voting)
✔️ build-ansible-collection SUCCESS in 4m 40s
✔️ ansible-test-sanity-docker-devel SUCCESS in 9m 44s (non-voting)
✔️ ansible-test-sanity-docker-milestone SUCCESS in 9m 39s
✔️ ansible-test-sanity-docker-stable-2.9 SUCCESS in 11m 08s
✔️ ansible-test-sanity-docker-stable-2.11 SUCCESS in 11m 53s
✔️ ansible-test-sanity-docker-stable-2.12 SUCCESS in 9m 35s
✔️ ansible-test-units-community-aws-python38 SUCCESS in 6m 04s
✔️ ansible-test-splitter SUCCESS in 3m 09s
✔️ integration-community.aws-1 SUCCESS in 45m 31s
⚠️ 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 105f921 into stable-3 May 9, 2022
@softwarefactory-project-zuul softwarefactory-project-zuul bot deleted the patchback/backports/stable-3/77cb09740beb34940ba678495a0d60768bce2d70/pr-1105 branch May 9, 2022 19:04
abikouo pushed a commit to abikouo/community.aws that referenced this pull request Oct 24, 2023
…ns#1138)

Fix AttributeError when running ec2_metadata_facts

SUMMARY
Prior to 5.0.0 we caught "Exception" within fetch, this was limited to json.JSONDecodeError, however it looks like we also needed to catch AttributeError for non-dict JSON
ISSUE TYPE

Bugfix Pull Request

COMPONENT NAME
ec2_metadata_facts
ADDITIONAL INFORMATION
fixes: ansible-collections#1134
abikouo pushed a commit to abikouo/community.aws that referenced this pull request Oct 24, 2023
Enable ec2_metadata integration tests

SUMMARY
Enable ec2_metadata_facts integration tests
ISSUE TYPE

Feature Pull Request

COMPONENT NAME
ec2_metadata_facts
ADDITIONAL INFORMATION
As shown by ansible-collections#1138 we should try to ensure that the integration tests are enabled

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 new_contributor Help guide this first time contributor plugins plugin (any type)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants