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.

Sign up for GitHub

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

Jump to bottom

Docker process mapper #25

Closed
peterbourgon opened this issue May 12, 2015 · 3 comments · Fixed by #62
Closed

Docker process mapper #25

peterbourgon opened this issue May 12, 2015 · 3 comments · Fixed by #62
Assignees
@tomwilkie
Milestone
endmay

Comments

@peterbourgon
Copy link
Contributor

In the probe, extract metadata from the Docker runtime, and attach it to nodes as appropriate.
@tomwilkie
Copy link
Contributor

I'm guessing we want to attach docker container id and name to nodes at a minimum. Note the mapping will be many-to-one (nodes to containers).

@peterbourgon
Copy link
Contributor Author

Full ack.

@tomwilkie
Copy link
Contributor

We're already using this docker client library in weave - https://github.com/fsouza/go-dockerclient

And launch-scope already bind mounts the docker socket into the container for you. Let me know if you need anything else.

@peterbourgon peterbourgon self-assigned this May 18, 2015
@peterbourgon peterbourgon added this to the endmay milestone May 18, 2015
rade added a commit that referenced this issue May 19, 2015
@rade
Merge pull request #25 from dpw/24_iptables_w
550aa30 
Check that 'iptables -w' is supported before using it

Fixes #24
@tomwilkie tomwilkie assigned tomwilkie and unassigned peterbourgon May 19, 2015
@tomwilkie tomwilkie mentioned this issue May 19, 2015
Merged
jml added a commit that referenced this issue Jul 14, 2016
@jml
Squashed 'tools/' changes from 0620e58..ca6ebfb
d3074bf 
ca6ebfb Merge pull request #25 from weaveworks/fix-brokenness
bfb1747 Test directories need ./ prefixes, obviously.
5b9b314 Merge pull request #24 from weaveworks/find-files
8786427 Remove spurious debugging code from test
8b7ec6e Speed up test by using git ls-files
cf53dc1 Exclude vendor from shell linting
b2ab380 Fix field name
c86fd3d Add notification config for wcloud
f643920 Merge pull request #23 from weaveworks/only-lint-git-files
47a0152 Only lint git files
50d47f9 Merge pull request #22 from weaveworks/shell-lint

git-subtree-dir: tools
git-subtree-split: ca6ebfb
jml added a commit that referenced this issue Jul 15, 2016
@jml
Squashed 'tools/' changes from 0620e58..e9e7e6b
3881934 
e9e7e6b Merge pull request #26 from weaveworks/this-time-for-sure
df494d6 Remove dependencies
c045d16 Properly exclude vendor from lint
2cfcf08 Add blacklist to wcloud client
ca6ebfb Merge pull request #25 from weaveworks/fix-brokenness
bfb1747 Test directories need ./ prefixes, obviously.
5b9b314 Merge pull request #24 from weaveworks/find-files
8786427 Remove spurious debugging code from test
8b7ec6e Speed up test by using git ls-files
cf53dc1 Exclude vendor from shell linting
b2ab380 Fix field name
c86fd3d Add notification config for wcloud
f643920 Merge pull request #23 from weaveworks/only-lint-git-files
47a0152 Only lint git files
50d47f9 Merge pull request #22 from weaveworks/shell-lint

git-subtree-dir: tools
git-subtree-split: e9e7e6b
2opremio pushed a commit that referenced this issue Aug 26, 2016
@jml
Merge pull request #25 from weaveworks/fix-brokenness
ca6ebfb 
Test directories need ./ prefixes, obviously.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@tomwilkie tomwilkie

Labels
None yet
Projects
None yet
Milestone
endmay
Development

Successfully merging a pull request may close this issue.

A docker process mapper tomwilkie/scope
2 participants
@peterbourgon @tomwilkie