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 #1627/3cf69cd4 backport][stable-5] Fix KeyError when Description is not present in ssm_parameter #1631

Conversation

patchback[bot]
Copy link

@patchback patchback bot commented Dec 22, 2022

This is a backport of PR #1627 as merged into main (3cf69cd).

SUMMARY

Fixes #1471

ISSUE TYPE
  • Bugfix Pull Request
COMPONENT NAME

ssm_parameter

ADDITIONAL INFORMATION

Fix KeyError when Description is not present in ssm_parameter

SUMMARY
Fixes #1471
ISSUE TYPE

Bugfix Pull Request

COMPONENT NAME
ssm_parameter
ADDITIONAL INFORMATION

Reviewed-by: Markus Bergholz <[email protected]>
Reviewed-by: Mark Chappell <None>
(cherry picked from commit 3cf69cd)
@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 Dec 22, 2022
@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded.

✔️ ansible-galaxy-importer SUCCESS in 4m 03s
✔️ build-ansible-collection SUCCESS in 5m 47s
✔️ ansible-test-sanity-docker-devel SUCCESS in 10m 36s (non-voting)
✔️ ansible-test-sanity-docker-milestone SUCCESS in 9m 35s (non-voting)
✔️ ansible-test-sanity-docker-stable-2.12 SUCCESS in 9m 06s
✔️ ansible-test-sanity-docker-stable-2.13 SUCCESS in 9m 15s
✔️ ansible-test-sanity-docker-stable-2.14 SUCCESS in 9m 37s
✔️ ansible-test-units-amazon-aws-python36 SUCCESS in 8m 06s
✔️ ansible-test-units-amazon-aws-python38 SUCCESS in 6m 20s
✔️ ansible-test-units-amazon-aws-python39 SUCCESS in 6m 00s
✔️ ansible-test-changelog SUCCESS in 2m 27s
✔️ ansible-test-splitter SUCCESS in 2m 42s
✔️ integration-community.aws-1 SUCCESS in 5m 42s
⚠️ 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
⚠️ integration-community.aws-19 SKIPPED
⚠️ integration-community.aws-20 SKIPPED
⚠️ integration-community.aws-21 SKIPPED
⚠️ integration-community.aws-22 SKIPPED

@tremble tremble added the mergeit Merge the PR (SoftwareFactory) label Dec 22, 2022
@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded (gate pipeline).

✔️ ansible-galaxy-importer SUCCESS in 3m 33s
✔️ build-ansible-collection SUCCESS in 5m 57s
✔️ ansible-test-sanity-docker-devel SUCCESS in 11m 41s (non-voting)
✔️ ansible-test-sanity-docker-milestone SUCCESS in 9m 37s (non-voting)
✔️ ansible-test-sanity-docker-stable-2.12 SUCCESS in 10m 27s
✔️ ansible-test-sanity-docker-stable-2.13 SUCCESS in 9m 35s
✔️ ansible-test-sanity-docker-stable-2.14 SUCCESS in 9m 48s
✔️ ansible-test-units-amazon-aws-python36 SUCCESS in 6m 31s
✔️ ansible-test-units-amazon-aws-python38 SUCCESS in 5m 44s
✔️ ansible-test-units-amazon-aws-python39 SUCCESS in 5m 53s
✔️ ansible-test-changelog SUCCESS in 2m 23s
✔️ ansible-test-splitter SUCCESS in 2m 40s
✔️ integration-community.aws-1 SUCCESS in 6m 04s
⚠️ 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
⚠️ integration-community.aws-19 SKIPPED
⚠️ integration-community.aws-20 SKIPPED
⚠️ integration-community.aws-21 SKIPPED
⚠️ integration-community.aws-22 SKIPPED

@softwarefactory-project-zuul softwarefactory-project-zuul bot merged commit 4528449 into stable-5 Dec 22, 2022
@softwarefactory-project-zuul softwarefactory-project-zuul bot deleted the patchback/backports/stable-5/3cf69cd4a59be18cd5aae5143d3cdc3c1846601d/pr-1627 branch December 22, 2022 15:23
abikouo pushed a commit to abikouo/community.aws that referenced this pull request Oct 24, 2023
…ctions#1631)

route53_health_check - add support for CALCULATED type

SUMMARY

Fixes ansible-collections#1442

ISSUE TYPE


Feature Pull Request

COMPONENT NAME

plugins/modules/route53_health_check.py
ADDITIONAL INFORMATION

Reviewed-by: Helen Bailey <[email protected]>
Reviewed-by: Taeho Park
Reviewed-by: Alina Buzachis
Reviewed-by: Mike Graves <[email protected]>
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 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.

3 participants