Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork4.5k
Fix/ssm put parameter operation#13437
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
base:main
Are you sure you want to change the base?
Fix/ssm put parameter operation#13437
Uh oh!
There was an error while loading.Please reload this page.
Conversation
localstack-bot commentedNov 29, 2025
I have read the CLA Document and I hereby sign the CLA You can retrigger this bot by commentingrecheck in this Pull Request.Posted by theCLA Assistant Lite bot. |
localstack-bot left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
Welcome to LocalStack! Thanks for raising your first Pull Request and landing in your contributions. Our team will reach out with any reviews or feedbacks that we have shortly. We recommend joining ourSlack Community and share your PR on the#community channel to share your contributions with us. Please make sure you are following ourcontributing guidelines and ourCode of Conduct.
jasjaapdua commentedNov 29, 2025
I have read the CLA Document and I hereby sign the CLA |
jasjaapdua commentedNov 29, 2025
The CI checks have failed because of missing labels and I don't have the permission to edit the labels now. Could a maintainer please add the following labels?
This PR is a small bugfix that aligns the SSM |
viren-nadkarni left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
Thanks for raising the PR!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
Could you write a AWS-validated integration test instead of this unit test? You can read more about themhere.
Uh oh!
There was an error while loading.Please reload this page.
Motivation
SSM
put_parameterevents currently always emitoperation: "Create", even when overwriting an existing parameter. AWS emitsoperation: "Update"on overwrite, so downstream listeners see incorrect event detail in LocalStack.Changes
"Update"whenput_parameterreturns a version > 1, otherwise"Create".Tests
Related