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

command: Ask for user variables before validation. #3042

Closed
wants to merge 2 commits into from
Closed

command: Ask for user variables before validation. #3042

wants to merge 2 commits into from

Conversation

tpounds
Copy link
Contributor

@tpounds tpounds commented Aug 20, 2015

Fixes an issue where user variables are not prompted for prior to validation. It is an almost identical fix to issue #736 (commit fb3f10e) but for the plan and refresh commands.

Error without fix.

$ terraform refresh
There are warnings and/or errors related to your configuration. Please fix these before continuing.

Errors:

  * 1 error(s) occurred:

* Required variable not set: foo

@tpounds tpounds closed this Aug 20, 2015
@tpounds tpounds reopened this Aug 20, 2015
@tpounds tpounds changed the title Ask for user variables before validation command: Ask for user variables before validation. Aug 21, 2015
@jen20
Copy link
Contributor

jen20 commented Nov 23, 2015

@tpounds Sorry, this PR predates me being a core committer on Terraform and apparently my searches didn't find it. It turns out that all of these fixes have been made via different pull requests between August and now and will be in the next point release. Thanks for the contribution - the tests are appreciated also!

@jen20 jen20 closed this Nov 23, 2015
@tpounds
Copy link
Contributor Author

tpounds commented Nov 23, 2015

No problem thanks for the update!

@ghost
Copy link

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

3 participants