Skip to content

Comments

Add netdev collector for darwin#393

Merged
discordianfish merged 1 commit intoprometheus:masterfrom
discordianfish:fish-add-netdev-darwin
Jan 3, 2017
Merged

Add netdev collector for darwin#393
discordianfish merged 1 commit intoprometheus:masterfrom
discordianfish:fish-add-netdev-darwin

Conversation

@discordianfish
Copy link
Member

Same as for openbsd, this is just slightly adjusted from freebsd
variant.
I'd like to merge all those and only have the bare minimum in platform specific files but for now I think adding it as this is fine. Will help with the refactoring later.

Copy link
Member

@grobie grobie left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why is this called CPUTime?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good question. I've copied it over from the freebsd one. I've changed it to use FormatUint directly.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

So this method can be deleted then?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Already did that, looks like the commit is outdated.

Same as for openbsd, this is just slightly adjusted from freebsd
variant.
@discordianfish discordianfish merged commit d2ca252 into prometheus:master Jan 3, 2017
@SuperQ SuperQ mentioned this pull request Jan 15, 2017
tamcore pushed a commit to gitgrave/node_exporter that referenced this pull request Oct 22, 2024
Our company internal Prometheus exporter uses the procfs library and
needs to read `/proc/cmdline`. Since `fs.proc.Path` cannot be accessed
from outside, add `CmdLine` to procfs to read `/proc/cmdline`.

Signed-off-by: Benjamin Drung <benjamin.drung@ionos.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants