Skip to content
This repository has been archived by the owner on Mar 14, 2022. It is now read-only.

adding open sockets #19

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

adding open sockets #19

wants to merge 4 commits into from

Conversation

rgarcia
Copy link

@rgarcia rgarcia commented May 5, 2017

We're trying to extend this plugin to report on the number of open sockets that a container has open, but running into problems getting it to work.

I've verified that the metric is calculated correctly by setting Verbose to true and seeing it show up along side the other metrics computed by the plugin, e.g. logs like:

[2017-05-05 04:11:45] docker : Value parameters to be emitted:
container : 8b364b4/ecs-agent
dimensions : {}
point_type : custom.open_sockets
value : [0]
type_instance : None 
time : <module 'time' (built-in)>

However, I don't see this metric in SignalFX, and if I turn on the CSV plugin to write additionally write all metrics to file, I don't see the custom metric appear there.

I've submitted this PR mostly for guidance, but if you think it's generally useful I'm happy to clean it up for a merge.

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

Successfully merging this pull request may close these issues.

2 participants