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

weavecloud-agent w/o token attempts to talk to local app #2299

Closed
rade opened this issue Mar 4, 2017 · 3 comments
Closed

weavecloud-agent w/o token attempts to talk to local app #2299

rade opened this issue Mar 4, 2017 · 3 comments
Assignees
Labels
bug Broken end user or developer functionality; not working as the developers intended it
Milestone

Comments

@rade
Copy link
Member

rade commented Mar 4, 2017

$ docker run --privileged -d --name=weavecloud-agent --net=host --pid=host -v /var/run/docker.sock:/var/run/docker.sock weaveworks/cloud-agent
a6e15b98221ceb697d619933091d215d0af7ca84dc3f336261b49e83c967433d
$ docker logs weavecloud-agent 
time="2017-03-04T11:11:07Z" level=info msg="publishing to: 127.0.0.1:4040" 
<probe> INFO: 2017/03/04 11:11:07.526733 command line args: --logtostderr=true --mode=probe --probe-only=true --probe.docker=true
...
<probe> ERRO: 2017/03/04 11:11:08.528537 Error fetching app details: Get http://127.0.0.1:4040/api: dial tcp 127.0.0.1:4040: getsockopt: connection refused
<repeats every 10 seconds>

That only happens when launching the weavecloud-agent without a service token, which is not documented. Still, it's surprising behaviour.

@rade rade added the bug Broken end user or developer functionality; not working as the developers intended it label Mar 4, 2017
@rade rade self-assigned this Mar 4, 2017
@2opremio
Copy link
Contributor

2opremio commented Mar 4, 2017 via email

@rade
Copy link
Member Author

rade commented Mar 4, 2017

No, it's because of incorrect flag parsing. I've submitted a PR to fix that.

@2opremio
Copy link
Contributor

2opremio commented Mar 4, 2017 via email

@rade rade closed this as completed in #2300 Mar 5, 2017
rade added a commit that referenced this issue Mar 5, 2017
parse --probe-only as intended

Fixes #2299.
@rade rade added this to the 1.3 milestone Mar 5, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Broken end user or developer functionality; not working as the developers intended it
Projects
None yet
Development

No branches or pull requests

2 participants