Add data source for aws_ssm_patch_baseline

[jdheyburn]

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

Closes #5116

Release note for CHANGELOG:

Added data source for aws_ssm_patch_baseline

Output from acceptance testing:

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

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

[jdheyburn]

I appreciate you may be busy @bflad and team, the PR has been open for 3 months now. Is there anything I can do to help this get merged?

[aeschright]

Hi @jdheyburn! Thanks for working on this. You'll need to back out the CHANGELOG addition -- that's something the person who merges will take care of so there aren't conflicts.

[jdheyburn]

Apologies - I've reverted the change to the CHANGELOG and updated the imports as it looks like they've changed since the PR was opened.

[jdheyburn]

Hey folks - let me know if there's anything else needed to get this forward, I appreciate the time taken to review this PR.

[bflad]

Hey @jdheyburn 👋 Thank you for submitting this and apologies for the delayed review. Overall this is looking pretty good, just a few items and this should be good to go. Please reach out if you have any questions or do not have time to implement them. 👍

[jdheyburn]

Apologies for several commits - doesn't look like the markdown was in the correct location for it to be picked up by docscheck. My branch is now so far behind it keeps missing all the new checks 😅

Let me know any further suggestions you might have 👍

[bflad]

Thanks for those updates, @jdheyburn, looks good to me 🚀

Output from acceptance testing:

--- PASS: TestAccAWSSsmPatchBaselineDataSource_existingBaseline (8.36s)
--- PASS: TestAccAWSSsmPatchBaselineDataSource_newBaseline (8.76s)

[ghost]

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

For further feature requests or bug reports with this functionality, please create a new GitHub issue following the template for triage. Thanks!

[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!