Support setting datacenter when using consul remote state

[bodiug]

Small change to allow setting datacenter when using consul remote state:

terraform remote config -backend=consul -backend-config=path=some/path -backend-config=access_token=some-token -backend-config=datacenter=dc1

This fixes #1614.

[stack72]

Hi @guidob

The tests pass as expected here. Please could you add some documentation for this as part of the PR?

% make testacc TEST=./builtin/providers/consul TESTARGS='-run='                                                                                                                                         1 ↵
==> Checking that code complies with gofmt requirements...
/Users/stacko/Code/go/bin/stringer
go generate $(go list ./... | grep -v /terraform/vendor/)
2016/08/11 20:39:44 Generated command/internal_plugin_list.go
TF_ACC=1 go test ./builtin/providers/consul -v -run= -timeout 120m
=== RUN   TestAccDataConsulKeys_basic
--- PASS: TestAccDataConsulKeys_basic (0.03s)
=== RUN   TestAccConsulAgentService_basic
--- PASS: TestAccConsulAgentService_basic (0.02s)
=== RUN   TestAccConsulCatalogEntry_basic
--- PASS: TestAccConsulCatalogEntry_basic (0.03s)
=== RUN   TestAccConsulKeyPrefix_basic
--- PASS: TestAccConsulKeyPrefix_basic (0.09s)
=== RUN   TestConsulKeysMigrateState
--- PASS: TestConsulKeysMigrateState (0.00s)
=== RUN   TestConsulKeysMigrateState_empty
--- PASS: TestConsulKeysMigrateState_empty (0.00s)
=== RUN   TestAccConsulKeys_basic
--- PASS: TestAccConsulKeys_basic (0.07s)
=== RUN   TestAccConsulNode_basic
--- PASS: TestAccConsulNode_basic (0.02s)
=== RUN   TestAccConsulService_basic
--- PASS: TestAccConsulService_basic (0.03s)
=== RUN   TestResourceProvider
--- PASS: TestResourceProvider (0.00s)
=== RUN   TestResourceProvider_impl
--- PASS: TestResourceProvider_impl (0.00s)
=== RUN   TestResourceProvider_Configure
--- PASS: TestResourceProvider_Configure (0.00s)
PASS
ok      github.com/hashicorp/terraform/builtin/providers/consul 0.304s

Thanks

Paul

[bodiug]

@stack72 ,

Good point, first time for me. Added a small piece.

[stack72]

@guidob thanks for getting back to me so fast on this :) This LGTM so will merge when the build goes green!

Thanks for the work on this

[ghost]

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.