Skip to content
This repository has been archived by the owner on Aug 25, 2021. It is now read-only.

Cloud Auto-joining not working with gce #16

Closed
sc7565 opened this issue Oct 3, 2018 · 3 comments
Closed

Cloud Auto-joining not working with gce #16

sc7565 opened this issue Oct 3, 2018 · 3 comments
Labels
bug Something isn't working

Comments

@sc7565
Copy link

sc7565 commented Oct 3, 2018

Setting up the Auto-join values in values.yaml results in error

join:
- "provider=gce project_name=test-0 tag_value=consul-server"

Error -

==> config: Unknown extra arguments: [project_name=test-0 tag_value=consul-server -domain=consul]

Is this valid error or something is wrong with my configuration

@mitchellh mitchellh added bug Something isn't working good first issue labels Oct 3, 2018
@jailson-silva
Copy link

I got the same error, buts was in AWS.

join:
- "provider=aws tag_key=consul-staging tag_value=auto-join"

Error logs:
==> config: Unknown extra arguments: [tag_key=consul-staging tag_value=auto-join -domain=consul]

helm version
Client: &version.Version{SemVer:"v2.11.0", GitCommit:"2e55dbe1fdb5fdb96b75ff144a339489417b146b", GitTreeState:"clean"}
Server: &version.Version{SemVer:"v2.11.0", GitCommit:"2e55dbe1fdb5fdb96b75ff144a339489417b146b", GitTreeState:"clean"}

Someone can help us?

@adilyse
Copy link
Contributor

adilyse commented Oct 17, 2018

This change made for another issue was potentially causing problems here as well:

Additionally, try one of these formats for the flags instead:
join: ["provider=gce", "project_name=test-0", "tag_value=consul-server"]

or

join:
  - "provider=gce"
  - "project_name=test-0"
  - "tag_value=consul-server"

I'll see if I can make the documentation clearer about the desired format in this case.

@sc7565 @jailson-silva If these changes don't solve your problem, feel free to reopen this issue!

@adilyse
Copy link
Contributor

adilyse commented Nov 12, 2018

Following up on this issue: it turns out that there were missing quotes in the template file that were causing problems. The original syntax is correct, and should work now that this PR is merged. Sorry for the confusion!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

4 participants