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

support collectd threshold states in dashboard CSS #13

Merged
merged 1 commit into from Mar 10, 2013
Merged

support collectd threshold states in dashboard CSS #13

merged 1 commit into from Mar 10, 2013

Conversation

dch
Copy link

@dch dch commented Mar 5, 2013

After this patch, pointing a tuned collectd install including thresholds will highlight threshold warnings and failures directly in the riemann dashboard.

Note this is not a "clean" change, as it includes adjustments to allow clean integration with collectd:

  • Riemann uses ok|warning|critical to colour states in the dashboard.
  • CollectD sends okay|warning|failure notifications via @pyr's write_riemann plugin, along with the original event.

Alternatively this could be patched on collectd side, or rewritten in riemann.config.

@pyr
Copy link
Contributor

pyr commented Mar 5, 2013

@dch, i'll submit a patch on collectd's side as well, thanks for the report

@aphyr
Copy link
Collaborator

aphyr commented Mar 10, 2013

Thanks dch, pyr! :)

aphyr added a commit that referenced this pull request Mar 10, 2013
support collectd threshold states in dashboard CSS
@aphyr aphyr merged commit ae2d48a into riemann:master Mar 10, 2013
@pyr
Copy link
Contributor

pyr commented Mar 11, 2013

just to let you know, there's collectd/collectd#271 waiting to be merged which will emit states the way riemann expects them, sorry about that!

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