search

sensu / sensu-influxdb-handler

Sensu Go InfluxDB Metrics Handler
https://sensu.io
MIT License
14 stars 16 forks source link

Event annotations #37

Closed nixwiz closed 4 years ago

nixwiz commented 4 years ago

Closes #36

Provide annotation point for 1.x parity.

Example of annotation with this Grafana Annotation definition: select description from sensu_event where $timeFilter ORDER BY time ASC

Annotation
calebhailey commented 4 years ago

That was fast!!

jspaleta commented 4 years ago

lgtm

jspaleta commented 4 years ago

did a local build and a test with system-profile-linux check.

Seems like its working. There's some corner cases that need to be massaged would definitely like to be able to optionally disable annotation

example: the system-profile-linux asset is a basically just a big ole metrics check. I'd like to run the influxdb handler without annotations as a metrics_handler for that check but run influxdb handler with annotations as a status handler + incident filter to catch/annotate check failures

sensu-discourse commented 4 years ago

This pull request has been mentioned on Sensu Community. There might be relevant details there:

https://discourse.sensu.io/t/pushing-check-results-to-influxdb/1793/14