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

website: v0.7 Upgrade Guide #7900

Merged
merged 3 commits into from
Aug 2, 2016
Merged

website: v0.7 Upgrade Guide #7900

merged 3 commits into from
Aug 2, 2016

Conversation

phinze
Copy link
Contributor

@phinze phinze commented Aug 1, 2016

No description provided.

@jbardin
Copy link
Member

jbardin commented Aug 1, 2016

I'm admittedly not the best proofreader, but LGTM

terraform-provisioner-* # provisioner plugins
```

These binaries needed to all be extracted to somewhere on your `$PATH` for Terraform to work.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

"In order for Terraform to work, each of these binaries must have been available on your $PATH, or in the ~/.terraform.d directory."?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ah yes I always forget about good ole ~/.terraform.d!


```
-var 'amis = { us-west-2 = "overriden_value" }'
```
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think we should also mention this is possible via the terraform.tfvars file & -var-files as well.

Some spelling and working fixes.

Added mention of deprecation warning to exisiting data sources as
resources.

Mention `~/.terraform.d` for old builtin plugin location

Added mention of `terraform.tfvars` and `-var-file` in map value
overrides.
@jbardin
Copy link
Member

jbardin commented Aug 2, 2016

Pushed some fixes for all comments so far

@jen20
Copy link
Contributor

jen20 commented Aug 2, 2016

This looks good to me now all the comments have been addressed!

@jbardin jbardin merged commit 9a0e18e into master Aug 2, 2016
@jbardin jbardin deleted the d-upgrade-guide branch August 2, 2016 17:27
@ghost
Copy link

ghost commented Apr 23, 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 23, 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.

4 participants