Skip to content

added Naemon exporter #745

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

Merged
merged 1 commit into from
May 16, 2017

Conversation

Griesbacher
Copy link
Contributor

This is an exporter for the Naemon Monitoringsystem. It is using the NEB Interface which is basically introduced by Nagios, so it should work with them, too. But there is still a strange bug which is currently preventing this.
Therefore I would only put Naemon in the title, because that is the system it is properly working with.

@brian-brazil
Copy link
Contributor

Based on the docs I'm not quite sure this is an exporter, how does it get its data?

Only counters should end in _total.

@Griesbacher
Copy link
Contributor Author

It's a module, which gets loaded by the core at the start. It uses callbacks and public variables to get its data. Similar to Livestatus or ModGearman, just is one is written in Go and therefore uses CGo.

So the countervector shouldn't use _total? I can change that.

@SuperQ
Copy link
Member

SuperQ commented May 15, 2017

@Griesbacher _total should only be used for values that go up, or are reset to 0 (but not very often). If the value goes up or down, it should not have _total and should be exported as a gauge.

@Griesbacher
Copy link
Contributor Author

Sorry, I've missed the gauge values but they are fixed.

@brian-brazil brian-brazil merged commit 2fae5c2 into prometheus:master May 16, 2017
@brian-brazil
Copy link
Contributor

Thanks!

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