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

core/diff: Fix attribute mismatch with tags.% #7207

Merged
merged 1 commit into from
Jun 24, 2016
Merged

core/diff: Fix attribute mismatch with tags.% #7207

merged 1 commit into from
Jun 24, 2016

Conversation

catsby
Copy link
Contributor

@catsby catsby commented Jun 16, 2016

This is the start of a patch to fix the acceptance test TestAccAWSInstance_forceNewAndTagsDrift. Right now it's failing like so:

--- FAIL: TestAccAWSInstance_forceNewAndTagsDrift (103.36s)
    testing.go:247: Step 1 error: Error applying: 1 error(s) occurred:

        * aws_instance.foo: diffs didn't match during apply. This is a bug with Terraform and should be reported as a GitHub Issue.

        Please include the following information in your report:

            Terraform Version: 0.7.0
            Resource ID: aws_instance.foo
            Mismatch reason: attribute mismatch: tags.%

(from https://travis-ci.org/hashicorp/terraform/jobs/136731270)

It's a regression test that was introduced in #4749 . With the recent addition of native lists, the state syntax for maps changed(?) from thing.# to thing.%, and so the safety provided in #4749 was lost (sort of).

This is my first foray into core-land so it's not clear to me if the test case is sufficient, and if the additional HasSuffix is enough of we need more of those in other place. I poke with @phinze and we started this at the end of the day, so I wanted to get the PR open to preserve some context.

@jen20
Copy link
Contributor

jen20 commented Jun 24, 2016

Thanks @catsby! This look correct to me. Great catch on your first foray into core land! 👍

@jen20 jen20 merged commit 1f9a2b2 into master Jun 24, 2016
@jen20 jen20 deleted the b-diff-maps branch June 24, 2016 12:03
@jen20 jen20 changed the title [WIP]: core/diff: Fix attribute mismatch with tags.% core/diff: Fix attribute mismatch with tags.% Jun 24, 2016
@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

Successfully merging this pull request may close these issues.

2 participants