search

sensu / catalog

Monitoring as code for Sensu Go. "There's a template for that!"
8 stars 4 forks source link

Add influxdb-metrics integration #169

Closed ccressent closed 2 years ago

ccressent commented 2 years ago

This is still a WIP, especially when it comes to environment variables vs secrets and content of the README.

Closes #85.

ccressent commented 2 years ago

Having a rough time trying to operate influxdb myself. Is this meant to work with the relatively new influxdb 2.0 or is it 1.x only?

The sensu-influxdb-handler plugin uses influxdata/influxdb1-client, so it's for InfluxDB 1.7 and earlier; for 1.8 and 2.x, one is supposed to use influxdata/influxdb-client-go.

There is already an issue open about 1.8/2.x support: https://github.com/sensu/sensu-influxdb-handler/issues/38. We might want to address it if we want this catalog item to be the best it can be at release...

calebhailey commented 2 years ago

Rebased against main

calebhailey commented 2 years ago

Related: https://github.com/sensu/sensu-enterprise-go/pull/2182

calebhailey commented 2 years ago

Rebased against main

calebhailey commented 2 years ago

...and rebased again. Going to get this reviewed and merged now.