Skip to content

Conversation

@rockholla
Copy link
Contributor

…ameter resource that is not ignoring value changes

Description

For context, I added an issue as well: #10

Adding in support for passing through the overwrite attribute of the aws_ssm_parameter resource: https://registry.terraform.io/providers/hashicorp/aws/6.17.0/docs/resources/ssm_parameter#overwrite-14

Motivation and Context

The overwrite attribute on the aws_ssm_parameter seems to remain as a non-deprecated attribute, so ensuring that this piece of the SSM parameter functionality can be utilized via this module.

Breaking Changes

Assuming there aren't any actual versions of the AWS provider/AWS APIs where this was indeed removed from support from AWS apis, then I think it wouldn't be breaking?

How Has This Been Tested?

I can go through some testing scenarios/add test cases if this is even the right move. Figured I'd start here w/ just proposing it basically and I will move forward w/ that if this looks like the right move.

  • I have updated at least one of the examples/* to demonstrate and validate my change(s)
  • I have tested and validated these changes using one or more of the provided examples/* projects
  • I have executed pre-commit run -a on my pull request

…ameter resource that is not ignoring value changes
@rockholla rockholla changed the title add in overwrite variable and pass through to the call to the ssm par… feat: add in overwrite variable and pass through to the call to the ssm par… Oct 21, 2025
@rockholla rockholla changed the title feat: add in overwrite variable and pass through to the call to the ssm par… feat: Add in overwrite variable and pass through to the call to the ssm par… Oct 21, 2025
@antonbabenko
Copy link
Member

It was deprecated in AWS provider version 5, so we can't support it.

resource/aws_ssm_parameter

The overwrite attribute has been deprecated. Existing parameters should be explicitly imported rather than relying on the "import on create" behavior previously enabled by setting overwrite = true. In a future major version the overwrite attribute will be removed and attempting to create a parameter that already exists will fail.

@antonbabenko antonbabenko merged commit e1dd7a2 into terraform-aws-modules:master Oct 21, 2025
6 checks passed
antonbabenko pushed a commit that referenced this pull request Oct 21, 2025
## [1.2.0](v1.1.3...v1.2.0) (2025-10-21)

### Features

* Add in overwrite variable and pass through to the call to the ssm par… ([#11](#11)) ([e1dd7a2](e1dd7a2))
@antonbabenko
Copy link
Member

This PR is included in version 1.2.0 🎉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants