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

New outputs format in tfstate #43

Merged
merged 2 commits into from
Aug 19, 2016
Merged

New outputs format in tfstate #43

merged 2 commits into from
Aug 19, 2016

Conversation

rsdy
Copy link
Contributor

@rsdy rsdy commented Aug 16, 2016

I'm not sure if this is a 0.7.0 change, but the format of the outputs object in terraform state file change to the following:

"outputs": {
    "datacenter": {
        "sensitive": false,
        "type": "string",
        "value": "mydc"
    }
}

@adammck
Copy link
Owner

adammck commented Aug 16, 2016

Thanks for the patch! I haven't upgraded yet, so didn't notice the change.

I'd like to merge this quickly, but I think it's important that we continue to support the legacy format for those who aren't able to upgrade Terraform immediately. Would you mind adding that here?

@rsdy
Copy link
Contributor Author

rsdy commented Aug 17, 2016

It will take me some time to get back with that patch, but I'll give it a shot.

@rsdy
Copy link
Contributor Author

rsdy commented Aug 19, 2016

I think that should be it. Let me know if there's anything else.

@adammck
Copy link
Owner

adammck commented Aug 19, 2016

Thank you! This looks excellent, and seems to work for me. Merging.

@adammck adammck merged commit df104e9 into adammck:master Aug 19, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants