search

savoirfairelinux / mod-influxdb

Shinken module for exporting data to InfluxDB
GNU Affero General Public License v3.0
13 stars 11 forks source link

Module doesn't work with shinken 2.0.3 (logevent.py is not present) #10

Closed dgilm closed 9 years ago

dgilm commented 9 years ago

Hi,

The module doesn't work with stable shinken (2.0.3) because the module shinken/misc/logevent.py is not present on the system:

https://github.com/savoirfairelinux/mod-influxdb/blob/master/module/module.py#L37 https://github.com/naparuba/shinken/tree/2.0.3/shinken/misc

The problem is that the shinken.io version of the module is also affected. So there is no stable influxdb module suitable if you want to work with the stable version of shinken.

aviau commented 9 years ago

Hello,

thank you for your interest in the module :)

As stated in the README of this repo, I am interested in supporting older versions of shinken. Since you asked, I will provide a patch this week (most likely tomorrow). Until then, the package.json states >2.0.3 and this is coherent.

dgilm commented 9 years ago

Thank you in advance @ReAzem,

In the meantime, I've just downloaded the logevent.py file from shinken git and the module is working pretty fine :)

Looking forward to play with influxdb!

influxdb

aviau commented 9 years ago

@dgilm, Fixed in b7f462e2bce4aebcdf977f01b54ce9442f79dfb7!

I am also releasing 0.7 to shinken.io. I have not tested the fix but its so straightfoward I don't think it will fail. Let me know if you have any issues.

Also, feel free to open other issues if you have other feedback or problems with the module :)

//cc @titilambert: You might no longer need to patch Shinken in @kaji-project.

dgilm commented 9 years ago

Thanks! :-)