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 #1309/29ab90eb backport][stable-4] Add support for check mode to SSM Parameter store #1312

Conversation

patchback[bot]
Copy link

@patchback patchback bot commented Jul 5, 2022

This is a backport of PR #1309 as merged into main (29ab90e).

SUMMARY
  • Adds support for check_mode
  • Adds basic waiters for create/delete
  • Fixes bug where module wasn't consistently idempotent
ISSUE TYPE
  • Bugfix Pull Request
  • Feature Pull Request
COMPONENT NAME

plugins/modules/aws_ssm_parameter_store.py

ADDITIONAL INFORMATION

Module was using a deprecated parameter when calling describe_parameters (Filters). This deprecated parameter appears to have some form of caching applied to it and would sometimes return old values. By switching to the ParameterFilters replacement things seem to be more consistent.

Add support for check mode to SSM Parameter store

SUMMARY

Adds support for check_mode
Adds basic waiters for create/delete
Fixes bug where module wasn't consistently idempotent

ISSUE TYPE

Bugfix Pull Request
Feature Pull Request

COMPONENT NAME
plugins/modules/aws_ssm_parameter_store.py
ADDITIONAL INFORMATION
Module was using a deprecated parameter when calling describe_parameters (Filters).  This deprecated parameter appears to have some form of caching applied to it and would sometimes return old values.  By switching to the ParameterFilters replacement things seem to be more consistent.

Reviewed-by: Alina Buzachis <None>
(cherry picked from commit 29ab90e)
@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 Jul 5, 2022
@tremble tremble added the mergeit Merge the PR (SoftwareFactory) label Jul 5, 2022
@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded (gate pipeline).

✔️ ansible-galaxy-importer SUCCESS in 4m 58s (non-voting)
✔️ build-ansible-collection SUCCESS in 6m 57s
✔️ ansible-test-sanity-docker-devel SUCCESS in 10m 17s (non-voting)
✔️ ansible-test-sanity-docker-milestone SUCCESS in 11m 42s
✔️ ansible-test-sanity-docker-stable-2.12 SUCCESS in 13m 11s
✔️ ansible-test-sanity-docker-stable-2.13 SUCCESS in 13m 18s
✔️ ansible-test-units-community-aws-python38 SUCCESS in 7m 28s
✔️ ansible-test-units-community-aws-python39 SUCCESS in 7m 34s
✔️ ansible-test-splitter SUCCESS in 3m 33s
✔️ integration-community.aws-1 SUCCESS in 7m 24s
⚠️ 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 6f21cb9 into stable-4 Jul 5, 2022
@softwarefactory-project-zuul softwarefactory-project-zuul bot deleted the patchback/backports/stable-4/29ab90ebd567bbc8fab1d907304b4b2d79bbe5c8/pr-1309 branch July 5, 2022 13:12
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.

2 participants