-
Notifications
You must be signed in to change notification settings - Fork 9.2k
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 our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
aws_route53_record allow_overwrite
deprecation will break ability to update SOA
#7846
Comments
This broke in #7734 |
…` argument Reference: #7846 Route53 Record `UPSERT` on creation does have some minor valid use cases. We will keep this argument for now while discouraging its usage in the documentation. Output from acceptance testing: ``` --- PASS: TestAccAWSRoute53Record_allowOverwrite (149.81s) ```
Pull request submitted to remove the deprecation: #8274 We will still discourage its usage as it can lead to conflicting management of infrastructure, but that choice will be up to the operator. |
The deprecation has been removed, which will release with version 2.6.0 of the Terraform AWS Provider later today. 👍 |
Thanks @bflad ! Very much appreciated. |
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! |
Community Note
Terraform Version
0.11.12
Affected Resource(s)
2.0.0+
aws_route53_record
Terraform Configuration Files
Our best practice is to set a low
SOA
on service discovery zones. The default SOA of 900 seconds leads to a default negative cache TTL which messes with service discovery when services query for the DNS record before the records have been updated/created. Basically, when this happens DNS lookups will fail for 15 minutes =); we change it to 30 seconds, so negative DNS queries are refreshed more quickly.Since AWS creates this
SOA
record automatically, the only way to accomplish this in a singleterraform apply
is to support theallow_overwrite
flag. Usingterraform import
in this scenario is a poor UX and something which we cannot mitigate; it's a consequence of how AWS creates the zone.Expected Behavior
I understand why this behavior is getting deprecated and it makes sense for non-automatically generated DNS record sets. The
SOA
record, however, is an exception. It's created automatically by AWS.Not sure the best course of action. Terraform should maybe import the
SOA
automatically since it gets generated by AWS, not by terraform. There should be a way to set theSOA
without getting the deprecation warning because this is not a user problem. There's no way we can mitigate this without theallow_overwrite
flag.Actual Behavior
Steps to Reproduce
aws_route53_zone
SOA
via theaws_route53_record
within the sameterraform apply
terraform apply
References
The text was updated successfully, but these errors were encountered: