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

backend/remote: notify users when uploading something other than cwd #22121

Merged
merged 1 commit into from
Jul 18, 2019

Conversation

mildwonkey
Copy link
Contributor

When a TFC workspace is configured with a working directory (and not with a VCS root), and a user is running terraform init locally from that same directory, TFC uploads the
entire configuration directory, not only the user's cwd. This is not
obvious to the user, so we are adding a descriptive message explaining
what is being uploaded, and why.

No VCS root, WD configured, init from directory other than VCS root:

Screen Shot 2019-07-18 at 10 18 03 AM

When a TFC workspace is configured without a VCS root, and with a
working directory, and a user is running `terraform init` from that same
directory, TFC uploads the entire configuration directory, not only the
user's cwd. This is not obvious to the user, so we are adding a descriptive
message explaining what is being uploaded, and why.
@mildwonkey mildwonkey requested a review from a team July 18, 2019 14:28
@mildwonkey
Copy link
Contributor Author

This PR addresses #21809 - this behavior is as designed, so we should make it clear to the user when we are uploading something other than their local cwd, and why.

@apparentlymart apparentlymart merged commit 190ef53 into master Jul 18, 2019
@apparentlymart apparentlymart deleted the mildwonkey/remote-backend-dir branch July 18, 2019 18:14
@ghost
Copy link

ghost commented Aug 18, 2019

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 Aug 18, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants