-
Notifications
You must be signed in to change notification settings - Fork 579
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
[dev.icinga.com #10480] Add InfluxDbWriter feature #3562
Comments
Updated by mfriedrich on 2015-10-28 18:55:48 00:00
I'd like to play with it in my spare time :) |
Updated by mfriedrich on 2015-10-30 23:35:20 00:00 http://stackoverflow.com/questions/33350314/usecases-influxdb-vs-prometheus/ |
Updated by mfriedrich on 2015-11-25 16:04:52 00:00
I'm postponing this until InfluxDB reaches 1.0.0 ... talked to many at OSMC and it seems 0.9.5 again broke the backend. |
Updated by mfriedrich on 2015-12-03 17:01:54 00:00
|
Updated by mfriedrich on 2016-03-31 06:11:24 00:00 https://influxdata.com/blog/update-on-influxdb-clustering-high-availability-and-monetization/ |
Updated by spjmurray on 2016-04-12 14:44:56 00:00 As discussed on IRC I'm happy to pick this up and maintain it. Here's what I imagine the configuration to look like:
The important bits:
InfluxDB also advises not to have more that one piece of data in a tag. Say if we had an MTU check applied to an interface we'd potentially generate mtueth0 as the service name from the check command mtu and the key we applied the check to eth0. In this case the mtu part is redundant as it's already available as $service.check_command$ and all we really care about is this data point is tagged to the interface eth0. This could be possibly surfaced by attaching the key to the service object. Another idea is to make tags programmable:
In this example rather than encoding the template as a fixed string, it can be encoded as a hash. The measurement field is required, tags is optional. Any unresolved macros are omitted from the final set of tags to be sent to InfluxDB. All I can think of right about now. Feel free to highlight any technical snafus I may be making. |
Updated by spjmurray on 2016-04-14 13:31:39 00:00 Working branch: https://github.com/spjmurray/icinga2/tree/feature/influxdb-writer-10480 |
Updated by mfriedrich on 2016-04-19 15:24:52 00:00
You're way too fast for us catching up with a review. Much appreciated though :) I've seen your blog post, leaving as a reference here: http://www.datacentred.co.uk/blog/integrating-icinga2-with-influxdb-and-grafana/ I'll take it for a review though it might take a while. Since it is a new feature it should target 2.5.0 then. Kind regards, |
Updated by spjmurray on 2016-04-19 16:24:53 00:00 shroud has already kicked my ass into shape so most of the major points should have been fixed! Bloody hell, the marketing droids don't hang about... well at least you get a bigger picture of what this is about. |
Updated by spjmurray on 2016-05-03 12:15:03 00:00
Applied in changeset 79c1e88. |
Updated by mfriedrich on 2016-05-03 12:35:40 00:00 You are actually using the http bits from libremote and fixed a bug over there, nice :) The rest looks like you know your development foo, also helped by shroud :) I've partially updated the documentation for addons/features and amended those changes into your patch. I have no objections against the patch though I'm lacking a test setup for now. Once the snapshot packages are built, please invite others to test them before they are hitting the release 2.5.0 then. |
Updated by spjmurray on 2016-05-05 05:26:05 00:00 Thanks Michael. I take it I can start testing the snapshots in staging once 2.4.8 is released? |
Updated by mfriedrich on 2016-05-05 07:03:16 00:00 Snapshot packages are built on each git commit for el7 (vagrant/docker trsts) and via cron for all other distributions once a day. You don't have to wait for the bugfix release 2.4.8, that release will just backport certsin fixes from the master development branch which currently targets 2.5.0 :) TL;DR - start now. |
Updated by spjmurray on 2016-05-09 09:43:57 00:00 Soak test started with 2.4.7 snapshot2016.05.09 1~trusty as of 10:41 09/05/16... |
Updated by mfriedrich on 2016-06-07 19:00:36 00:00
|
Updated by mfriedrich on 2016-06-07 20:01:58 00:00
|
This issue has been migrated from Redmine: https://dev.icinga.com/issues/10480
Created by mfriedrich on 2015-10-28 18:55:17 00:00
Assignee: mfriedrich
Status: Resolved (closed on 2016-05-03 12:15:03 00:00)
Target Version: 2.5.0
Last Update: 2016-05-09 09:43:57 00:00 (in Redmine)
InfluxDB 0.9.x could still use the GraphiteWriter feature although they changed the tree notation from Graphite into a tag based system similar to OpenTSDB. By default the Graphite tree is stored "flat" and does not benefit from the InfluxDB tags and indexes. So there's not really a point in using the Graphite input with InfluxDB at this point.
influxdata/influxdb#2996
influxdata/influxdb#3125
Changesets
2016-05-03 12:12:51 00:00 by spjmurray 79c1e88
Relations:
The text was updated successfully, but these errors were encountered: