Added ComplianceSeverity Argument

[marcotesch]

Signed-off-by: Marco Tesch mtesch@tecracer.de

Community Note

• Please vote on this pull request by adding a 👍 reaction to the original pull request comment to help the community and maintainers prioritize this request
• Please do not leave "+1" comments, they generate extra noise for pull request followers and do not help prioritize the request

Changes proposed in this pull request:

• Added compliance_severity argument to resource aws_ssm_association

[bflad]

Hi @marcotesch 👋 Thanks for contributing this, its looking pretty good so far. Can you please add a new acceptance test in aws/resource_aws_ssm_association_test.go to cover this new functionality?

[marcotesch]

Hi @bflad ,

I tried to understand the acceptance test layout as you use it, hope I have done everything right 🙈 .
I added the new Acceptance Test for the functionality. Here is also the output of that test run:

make testacc TEST=./aws TESTARGS='-run=TestAccAWSSSMAssociation_withComplianceSeverity'
==> Checking that code complies with gofmt requirements...
TF_ACC=1 go test ./aws -v -parallel 20 -run=TestAccAWSSSMAssociation_withComplianceSeverity -timeout 120m
=== RUN   TestAccAWSSSMAssociation_withComplianceSeverity
=== PAUSE TestAccAWSSSMAssociation_withComplianceSeverity
=== CONT  TestAccAWSSSMAssociation_withComplianceSeverity
--- PASS: TestAccAWSSSMAssociation_withComplianceSeverity (19.02s)
PASS
ok      github.com/terraform-providers/terraform-provider-aws/aws       19.066s

[marcotesch]

Critical Severity added as well.

[bflad]

Looks great, thanks @marcotesch! 🚀

--- PASS: TestAccAWSSSMAssociation_withDocumentVersion (19.27s)
--- PASS: TestAccAWSSSMAssociation_withComplianceSeverity (28.60s)
--- PASS: TestAccAWSSSMAssociation_withScheduleExpression (34.14s)
--- PASS: TestAccAWSSSMAssociation_withParameters (36.16s)
--- PASS: TestAccAWSSSMAssociation_withAssociationName (36.17s)
--- PASS: TestAccAWSSSMAssociation_withAssociationNameAndScheduleExpression (36.17s)
--- PASS: TestAccAWSSSMAssociation_withTargets (39.26s)
--- PASS: TestAccAWSSSMAssociation_withOutputLocation (67.86s)
--- PASS: TestAccAWSSSMAssociation_basic (149.51s)

[bflad]

This has been released in version 2.3.0 of the Terraform AWS provider. Please see the Terraform documentation on provider versioning or reach out if you need any assistance upgrading.

[ghost]

I'm going to lock this issue because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues.

If you feel this issue should be reopened, we encourage creating a new issue linking back to this one for added context. Thanks!