dgolja / golja-influxdb

Puppet module for InfluxDB
Apache License 2.0
19 stars 70 forks source link

support InfluxDB 1.0 #46

Closed dvdjaco closed 7 years ago

dvdjaco commented 8 years ago

InfluxDB 1.0 was officially released today. Looking at the release notes the only relevant changes seem to be in the config file, any plans about it?

dnsmichi commented 8 years ago

Using the default class, the influxd daemon cannot be started.

Sep  8 19:00:55 centos-7 influxd: [run] 2016/09/08 19:00:55 Using configuration at: /etc/influxdb/influxdb.conf
Sep  8 19:00:55 centos-7 influxd: run: parse config: toml: cannot load TOML value of type map[string]interface {} into a Go slice

Applying the config changes from the changelog, it will work.

These are the slice errors:

I'll prepare a PR, I'm using your great module inside the icinga2x-influxdb Vagrant box :)

dnsmichi commented 8 years ago

PR #47 - for tests:

git clone https://github.com/Icinga/icinga-vagrant && cd icinga-vagrant/icinga2x-influxdb && vagrant up

dgolja commented 8 years ago

i will review the changes in the next days and release the new version fully supporting v1.0

codec commented 8 years ago

You might also be interested in https://github.com/codec/golja-influxdb/commit/78f718b0e21e9f6111269c76a6f1ef4dc9ee8ef8 (I'm not sending a PR for this because I've branched off of @dnsmichi's changes)

Edit: Never mind. This option is no longer valid in Influx 1.0. :cry:

KlavsKlavsen commented 8 years ago

looking forward to module supporting 1.0 - so I can start with the first influxdb GA in my new install :)

mrh666 commented 8 years ago

+1 can't wait to try v.1.0! :)

AndreCAndersen commented 7 years ago

Did the 1.0 version get released?

dgolja commented 7 years ago

the module is officially released.