Skip to content
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

0.7.0-rc2 template_file: diffs didn't match during apply. #7244

Closed
hngkr opened this issue Jun 20, 2016 · 8 comments
Closed

0.7.0-rc2 template_file: diffs didn't match during apply. #7244

hngkr opened this issue Jun 20, 2016 · 8 comments

Comments

@hngkr
Copy link

hngkr commented Jun 20, 2016

Terraform Version

Terraform v0.7.0-rc2 (46a0709)

Affected Resource(s)

  • template_file

Terraform Configuration Files

provider "aws" {
  region = "eu-west-1"
}

resource "aws_api_gateway_rest_api" "terraform_template_rest_api" {
  name = "TerraformTemplateTest"
  description = "This is for testing a Terraform Template bug"
}

resource "template_file" "dummy" {
  template = "${file("${path.module}/dummy.tpl")}"
  vars {
    rest_api = "${aws_api_gateway_rest_api.terraform_template_rest_api.id}"
  }
}

Output

https://gist.github.com/hngkr/f98472c788ac883086425170b65b3f0d

Expected Behavior

Should have created the template_file resource

Actual Behavior

Failed with: diffs didn't match during apply
Mismatch reason: extra attributes: vars.rest_api

Steps to Reproduce

  1. terraform apply

Important Factoids

Running it a second time works.
Same construction has worked fine in 0.7.0-rc1 and earlier.

@jbardin
Copy link
Member

jbardin commented Jun 20, 2016

This looks similar to what was solved in #7205, except with maps instead of sets.

I'd like to figure out what changes between rc1 and rc2 to trigger it though

@hngkr
Copy link
Author

hngkr commented Jun 20, 2016

At this point of the evening, I'm certainly not clever enough to look thru' what #7205 does, but I just tried with master - and it's complaining about the same thing as above.

@jbardin
Copy link
Member

jbardin commented Jun 20, 2016

This regression is a side effect of #7082.

@glasser
Copy link
Contributor

glasser commented Jun 23, 2016

Running into this one too.

jbardin added a commit that referenced this issue Jun 24, 2016
jbardin added a commit that referenced this issue Jun 27, 2016
core: don't compare map diffs for computed values
@jbardin jbardin closed this as completed Jun 27, 2016
@colemickens
Copy link

Was this auto-closed on accident?

@jbardin
Copy link
Member

jbardin commented Jun 28, 2016

Hi @colemickens,

Nope, but the links between the issues are confusing, both because I mentioned the this issue in the code, and didn't change the PR message to include the "fixes" notation. Sorry about that.

This was closed by #7249

@hngkr
Copy link
Author

hngkr commented Jun 28, 2016

I'll just add that I just tried to rerun the script from the reported issue above. I used Terraform from the master branch, and I'm no longer getting the error message. Everything works beautifully. Thanks a lot!

@ghost
Copy link

ghost commented Apr 24, 2020

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.

@ghost ghost locked and limited conversation to collaborators Apr 24, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

4 participants