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.

Sign up for GitHub

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

Jump to bottom

[PR #1627/3cf69cd4 backport][stable-4] Fix KeyError when Description is not present in ssm_parameter #1630

Merged
merged 1 commit into from
Dec 22, 2022
Merged

[PR #1627/3cf69cd4 backport][stable-4] Fix KeyError when Description is not present in ssm_parameter #1630

merged 1 commit into from
Dec 22, 2022

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

@rwha @patchback
Fix KeyError when Description is not present in ssm_parameter (#1627)
c8c602a 
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)
@patchback patchback bot mentioned this pull request Dec 22, 2022
Merged
@ansibullbot
Copy link

cc @jillr @markuman @mikedlr @nathanwebsterdotme @ozbillwang @s-hertel @tremble
click here for bot help

@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 new_module New module new_plugin New plugin plugins plugin (any type) tests tests labels Dec 22, 2022
@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded.

✔️ ansible-galaxy-importer SUCCESS in 4m 12s
✔️ build-ansible-collection SUCCESS in 5m 24s
❌ ansible-test-sanity-docker-devel FAILURE in 15m 41s (non-voting)
❌ ansible-test-sanity-docker-milestone FAILURE in 10m 13s (non-voting)
✔️ ansible-test-sanity-docker-stable-2.12 SUCCESS in 11m 29s
✔️ ansible-test-sanity-docker-stable-2.13 SUCCESS in 10m 37s
✔️ ansible-test-sanity-docker-stable-2.14 SUCCESS in 12m 59s
✔️ ansible-test-units-amazon-aws-python36 SUCCESS in 6m 22s
✔️ ansible-test-units-amazon-aws-python38 SUCCESS in 6m 16s
✔️ ansible-test-units-amazon-aws-python39 SUCCESS in 6m 28s
✔️ ansible-test-changelog SUCCESS in 2m 25s
✔️ ansible-test-splitter SUCCESS in 2m 28s
✔️ integration-community.aws-1 SUCCESS in 6m 11s
⚠️ 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 4m 26s
✔️ build-ansible-collection SUCCESS in 5m 07s
❌ ansible-test-sanity-docker-devel FAILURE in 12m 42s (non-voting)
❌ ansible-test-sanity-docker-milestone FAILURE in 10m 43s (non-voting)
✔️ ansible-test-sanity-docker-stable-2.12 SUCCESS in 10m 34s
✔️ ansible-test-sanity-docker-stable-2.13 SUCCESS in 10m 37s
✔️ ansible-test-sanity-docker-stable-2.14 SUCCESS in 10m 07s
✔️ ansible-test-units-amazon-aws-python36 SUCCESS in 7m 03s
✔️ ansible-test-units-amazon-aws-python38 SUCCESS in 5m 57s
✔️ ansible-test-units-amazon-aws-python39 SUCCESS in 6m 30s
✔️ ansible-test-changelog SUCCESS in 2m 45s
✔️ ansible-test-splitter SUCCESS in 2m 41s
✔️ integration-community.aws-1 SUCCESS in 5m 41s
⚠️ 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 0b3c51d into stable-4 Dec 22, 2022
@softwarefactory-project-zuul softwarefactory-project-zuul bot deleted the patchback/backports/stable-4/3cf69cd4a59be18cd5aae5143d3cdc3c1846601d/pr-1627 branch December 22, 2022 15:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@tremble tremble tremble approved these changes

Assignees
No one assigned
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 new_module New module new_plugin New plugin plugins plugin (any type) tests tests
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@ansibullbot @tremble @rwha