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

CONSUL_HTTP_TOKEN doesn't seem to be supported? #78

Closed
wleese opened this issue Feb 11, 2016 · 5 comments
Closed

CONSUL_HTTP_TOKEN doesn't seem to be supported? #78

wleese opened this issue Feb 11, 2016 · 5 comments

Comments

@wleese
Copy link

wleese commented Feb 11, 2016

I can't seem to get containerbuddy to use CONSUL_HTTP_TOKEN. Logging from the agent:

`Feb 11 14:07:12 host consul[18926]: agent: Synced service 'mesos-consul:xxxxxxxx:tst-logredis:31398'

Feb 11 14:08:41 host consul: 2016/02/11 14:08:41 [ERR] http: Request PUT /v1/agent/check/pass/containername-81eb092d0568?note=ok, error: CheckID does not have associated TTL from=172.17.0.3:55587

Feb 11 14:08:41 host consul[18926]: http: Request PUT /v1/agent/check/pass/containername-81eb092d0568?note=ok, error: CheckID does not have associated TTL from=172.17.0.3:55587

Feb 11 14:08:41 host consul: 2016/02/11 14:08:41 [WARN] agent: Service 'containername-81eb092d0568' registration blocked by ACLs

Feb 11 14:08:41 host consul[18926]: agent: Service 'containername-81eb092d0568' registration blocked by ACLs

Feb 11 14:08:41 host consul: 2016/02/11 14:08:41 [WARN] agent: Check 'containername-81eb092d0568' registration blocked by ACLs

Feb 11 14:08:41 host consul[18926]: agent: Check 'containername-81eb092d0568' registration blocked by ACLs

Feb 11 14:08:51 host consul: 2016/02/11 14:08:51 [WARN] agent: Check 'containername-81eb092d0568' registration blocked by ACLs

Feb 11 14:08:51 host consul[18926]: agent: Check 'containername-81eb092d0568' registration blocked by ACLs

Feb 11 14:08:53 host consul: 2016/02/11 14:08:53 [WARN] agent: Check 'containername-649ba536bd72' missed TTL, is now critical

Feb 11 14:08:53 host consul[18926]: agent: Check 'containername-649ba536bd72' missed TTL, is now critical

Feb 11 14:08:53 host consul[18926]: agent: Check 'containername-649ba536bd72' registration blocked by ACLs

Feb 11 14:08:53 host consul: 2016/02/11 14:08:53 [WARN] agent: Check 'containername-649ba536bd72' registration blocked by ACLs`

Am I correct in believing this is simply not supported at the moment? I've tried the 0.1.1 release and building off of master

@tgross
Copy link
Contributor

tgross commented Feb 11, 2016

Am I correct in believing this is simply not supported at the moment? I've tried the 0.1.1 release and building off of master

That looks to be correct, and this should be considered a bug. It looks like we need to pull in CONSUL_HTTP_TOKEN when we set up the Consul config struct, which we do here.

A patch for this would be welcome, otherwise I'll be sure to hit it within the next couple days. Thanks for the bug report, @wleese !

@wleese
Copy link
Author

wleese commented Feb 11, 2016

Golang isn't my thing, but you asked for it ;)

@tgross
Copy link
Contributor

tgross commented Feb 12, 2016

Merged and thanks for the patch, @wleese !

@tgross
Copy link
Contributor

tgross commented Feb 17, 2016

@tgross tgross closed this as completed Feb 17, 2016
@wleese
Copy link
Author

wleese commented Feb 18, 2016

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants