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 #1417/b3bc6893 backport][stable-3] Fix KeyError when Cluster Parameter Group is specified in rds_cluster.py #1432

Conversation

patchback[bot]
Copy link

@patchback patchback bot commented Sep 5, 2022

This is a backport of PR #1417 as merged into main (b3bc689).

SUMMARY

Fix KeyError when comparing state.
Fixes: #1409

ISSUE TYPE
  • Bugfix Pull Request
COMPONENT NAME

rds_cluster.py

….py (#1417)

Fix KeyError when Cluster Parameter Group is specified in rds_cluster.py

SUMMARY
Fix KeyError when comparing state.
Fixes: #1409
ISSUE TYPE

Bugfix Pull Request

COMPONENT NAME
rds_cluster.py

Reviewed-by: Alina Buzachis <None>
Reviewed-by: Markus Bergholz <git@osuv.de>
(cherry picked from commit b3bc689)
@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded.

✔️ ansible-galaxy-importer SUCCESS in 3m 40s
✔️ build-ansible-collection SUCCESS in 5m 19s
ansible-test-sanity-docker-devel FAILURE in 12m 23s (non-voting)
✔️ ansible-test-sanity-docker-milestone SUCCESS in 9m 09s
✔️ ansible-test-sanity-docker-stable-2.12 SUCCESS in 10m 32s
✔️ ansible-test-sanity-docker-stable-2.13 SUCCESS in 11m 38s
✔️ ansible-test-units-community-aws-python38 SUCCESS in 6m 07s
✔️ ansible-test-units-community-aws-python39 SUCCESS in 5m 55s
✔️ ansible-test-splitter SUCCESS in 2m 35s
✔️ integration-community.aws-1 SUCCESS in 11m 40s
⚠️ 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

@ansibullbot
Copy link

@ansibullbot ansibullbot added bug This issue/PR relates to a bug community_review integration tests/integration module module needs_triage new_contributor Help guide this first time contributor plugins plugin (any type) tests tests labels Sep 5, 2022
@tremble
Copy link
Contributor

tremble commented Sep 23, 2022

recheck

@ansibullbot ansibullbot added new_module New module new_plugin New plugin and removed needs_triage labels Sep 23, 2022
@softwarefactory-project-zuul
Copy link
Contributor

@alinabuzachis
Copy link
Contributor

recheck

@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded.

✔️ ansible-galaxy-importer SUCCESS in 4m 27s
✔️ build-ansible-collection SUCCESS in 5m 20s
ansible-test-sanity-docker-devel FAILURE in 9m 05s (non-voting)
✔️ ansible-test-sanity-docker-milestone SUCCESS in 9m 15s
✔️ ansible-test-sanity-docker-stable-2.12 SUCCESS in 11m 12s
✔️ ansible-test-sanity-docker-stable-2.13 SUCCESS in 11m 10s
✔️ ansible-test-units-community-aws-python38 SUCCESS in 7m 06s
✔️ ansible-test-units-community-aws-python39 SUCCESS in 6m 03s
✔️ ansible-test-splitter SUCCESS in 2m 41s
✔️ integration-community.aws-1 SUCCESS in 12m 12s
⚠️ 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
⚠️ integration-community.aws-14 SKIPPED
⚠️ integration-community.aws-15 SKIPPED
⚠️ integration-community.aws-16 SKIPPED
⚠️ integration-community.aws-17 SKIPPED
⚠️ integration-community.aws-18 SKIPPED

@markuman
Copy link
Member

We've forget to merge this. Because the last release of 3.x.x. is already made, I'm going to close this backport pr.

@markuman markuman closed this Oct 10, 2022
@tremble tremble deleted the patchback/backports/stable-3/b3bc68931de04fb2d8558b01c45988710d24cec3/pr-1417 branch February 15, 2023 09:29
abikouo pushed a commit to abikouo/community.aws that referenced this pull request Oct 24, 2023
…-collections#1433)

route53_health_check - Bump version-added on measure_latency

SUMMARY
It didn't get backported yet ansible-collections#1432
ISSUE TYPE

Docs Pull Request

COMPONENT NAME
route53_health_check
ADDITIONAL INFORMATION
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug This issue/PR relates to a bug community_review integration tests/integration module module new_contributor Help guide this first time contributor new_module New module new_plugin New plugin plugins plugin (any type) tests tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants