Skip to content
This repository has been archived by the owner on Nov 30, 2018. It is now read-only.

Upgrading to Terraform 0.10.8 #18

Merged
merged 5 commits into from Nov 3, 2017
Merged

Upgrading to Terraform 0.10.8 #18

merged 5 commits into from Nov 3, 2017

Conversation

tombuildsstuff
Copy link
Member

Turns out master's still pointing to Terraform 0.10.0

$ govendor fetch github.com/hashicorp/terraform/...@v0.10.8

in order to fix the build - I then compared this branch's version and ran:

$ govendor fetch github.com/hashicorp/consul/api@610f3c86a089817b5bd5729a3b8c2db33a9ae2b0
$ govendor fetch github.com/hashicorp/go-getter@a686900cb3753aa644dc4812be91ceaf9fdd3b98
$ govendor fetch github.com/coreos/etcd/clientv3@v3.2.7
govendor fetch github.com/hashicorp/go-plugin@a5174f84d7f8ff00fb07ab4ef1f380d32eee0e63

This should finally fix hashicorp/terraform-provider-azurerm#351

```
$ govendor fetch github.com/hashicorp/terraform/...@v0.10.8
```

in order to fix the build - I then compared this branch's version and ran:

```
$ govendor fetch github.com/hashicorp/consul/api@610f3c86a089817b5bd5729a3b8c2db33a9ae2b0
$ govendor fetch github.com/hashicorp/go-getter@a686900cb3753aa644dc4812be91ceaf9fdd3b98
$ govendor fetch github.com/coreos/etcd/clientv3@v3.2.7
govendor fetch github.com/hashicorp/go-plugin@a5174f84d7f8ff00fb07ab4ef1f380d32eee0e63
```
```
govendor fetch google.golang.org/grpc@7657092a1303cc5a6fa3fee988d57c665683a4da
```
Copy link
Member

@apparentlymart apparentlymart left a comment

Choose a reason for hiding this comment

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

LGTM! Thanks for fixing this up.

I'm also working on hashicorp/terraform#16543 to merge this provider back into the main terraform binary, so we can avoid problems of this nature in future. Having this fix in place will be helpful to those using Terraform 0.10 though, since it will always depend on the separated version of this provider.

(Once we have a release with it integrated, I'll make a change to the README in this repo to clarify that this codebase would then be in maintenance mode only, with continued development going on in the Terraform Core repository.)

@tombuildsstuff tombuildsstuff merged commit 7ffa4fb into master Nov 3, 2017
@tombuildsstuff tombuildsstuff deleted the vendor-funtime branch November 3, 2017 15:19
tombuildsstuff added a commit that referenced this pull request Nov 3, 2017
@phamann
Copy link

phamann commented Nov 8, 2017

@tombuildsstuff @apparentlymart It seems that this PR and the corresponding 1.0.2 release has broken clients still using Terraform 0.10.0 as this plugin as a provider. I get the following error:

provider.terraform: dial unix /var/folders/gm/g6yqvzvx1tg6890p349w52xm0000gn/T/plugin648040195|netrpc: connect: no such file or directory

Upgrading my local Terraform to 0.10.8 solved the problem.

@apparentlymart
Copy link
Member

Hmm... thanks for reporting that, @phamann.

That certainly wasn't intended 😖 ... I think this is due to a change to the underlying plugin library we use which now supports several different plugin protocols, but seems to be confusing old versions of Terraform by including the protocol announcement (|netrpc) in its startup message.

Unfortunately I don't think there's much we can do here since we can't re-issue the old 0.10 releases to have the new client... we'll watch out for this as we upgrade the go-plugin version used by other providers.

Sorry for the unexpected breakage!

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.

data.terraform_remote_state does not recognize backend = "azurerm"
3 participants