-
Notifications
You must be signed in to change notification settings - Fork 9.6k
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
ConfigDocument JSON always triggers change with whitespace #6289
Comments
See also #3634. I think the solution for many of these JSON issues will be for Terraform to parse the JSON from either end and compare the resulting data structure instead of just comparing the JSON strings. AWS clearly isn't always storing these documents internally as strings, but is regenerating the JSON on the fly for a describe. |
Seeing this also with Terraform 0.7.1. Thanks for the embedded EOF workaround, it solves the problem for now, though it would be awesome to have newlines or load from a separate JSON file without the newlines being an issue :) |
Still seeing this in v0.8.6, but the EOF workaround is not working for me. |
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 have found a problem that seems similar to this, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further. |
Hi there,
Terraform Version
Affected Resource(s)
Please list the resources as a list, for example:
Terraform Configuration Files
Doesn't trigger change
Triggers change
Expected Behavior
terraform plan
shouldn't expect a change with this config settingActual Behavior
Every time I run
terraform plan
, a change is expected, except when I copy and paste the output of theterraform plan
and use it in the JSON keys in the exact same order without any whitespace.Steps to Reproduce
Please list the steps required to reproduce the issue, for example:
terraform apply
terraform plan
Important Factoids
Nothing special
References
Similar to 6257, but not the same.
The text was updated successfully, but these errors were encountered: