Add Disabled property to aws_route53_health_check

[Glen-Moonpig]

Community Note

• Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
• Please do not leave "+1" or "me too" comments, they generate extra noise for issue followers and do not help prioritize the request
• If you are interested in working on this issue or have submitted a pull request, please leave a comment

Description

CreateHealthCheck supports creating a disabled health check, but this is not implemented in aws_route53_health_check. I want to be able to disable health checks in certain environments I am deploying to, to avoid unnecessary costs.

New or Affected Resource(s)

aws_route53_health_check

Potential Terraform Configuration

resource "aws_route53_health_check" "example" {
  disabled = "true"
}

References

• #0000

[nvmsteve]

This issue appears to have been 'duplicated' multiple times, this suggests a number of people would like this feature. This one has been open for almost a year.
There are 2 pull requests (at least) to add this functionality including #10436 which was closed because it was submitted after #8939, which was closed due to inactivity.

Does anyone know how we can we progress getting this functionality added?

[Glen-Moonpig]

This issue appears to have been 'duplicated' multiple times, this suggests a number of people would like this feature. This one has been open for almost a year.
There are 2 pull requests (at least) to add this functionality including #10436 which was closed because it was submitted after #8939, which was closed due to inactivity.

Does anyone know how we can we progress getting this functionality added?

It would be very useful for us to have this functionality added, but adding such basic functionality to the terraform provider is a real pain and takes considerable time, effort and skill. It would be great if AWS could just generate the code based on their API spec..

[anGie44]

This feature been merged and will release with v3.2.0 of the Terraform AWS Provider, expected out later today. Thanks @Glen-Moonpig for your contributions!

[Glen-Moonpig]

This feature been merged and will release with v3.2.0 of the Terraform AWS Provider, expected out later today. Thanks @Glen-Moonpig for your contributions!

Thanks, glad to see this attribute finally ready to go 😅

[ghost]

This has been released in version 3.2.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!