-
Notifications
You must be signed in to change notification settings - Fork 9.1k
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_ses_receipt_rule.s3store: aws_ses_receipt_rule.s3store: diffs didn't match during apply #6741
Comments
Just leaving some notes from briefly looking at this. From the debug logs:
Seems to be related to: (the change, which made the trailing period consistent) #6299, (similar downstream behavior change) #6533 and #6535 My two comments here summarize the problem as it now exists: #6535 (comment) For this specific issue, a workaround for the resource "aws_ses_receipt_rule" "s3store" {
# ... other configuration ...
recipients = ["${var.recipient}@${replace(aws_route53_zone.data_zone.name, "/\\.$/", "")}"] The fix will likely be to just always strip the trailing period from the attribute when saving it in the Terraform state. This will likely have to wait until 2.0.0 of the AWS provider since it is a major breaking change. |
Dang... I spent 3 hours my eyes stuck on this trace without seeing the trailing '.'. Thanks a lot, this fixes the issue. |
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
Terraform v0.11.10
Affected Resource(s)
Terraform Configuration Files
$ cat route53.tf
$ cat s3.tf
$ cat ses.tf
Debug Output
https://gist.github.com/iMilnb/3039cb586af94adb097551d13ec707b4
Panic Output
Expected Behavior
Actual Behavior
Steps to Reproduce
terraform apply
Important Factoids
This misbehavior starts with version
1.42
, everything work as intended until v1.41. I read theChangelog
, I have no reference tovpc_id
orvpc_region
in mytf
files.References
Found no reference to a similar issue.
The text was updated successfully, but these errors were encountered: