Skip to content
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

support InfluxDB 1.0 #46

Closed
dvdjaco opened this issue Sep 8, 2016 · 8 comments
Closed

support InfluxDB 1.0 #46

dvdjaco opened this issue Sep 8, 2016 · 8 comments

Comments

@dvdjaco
Copy link

dvdjaco commented Sep 8, 2016

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
Copy link
Contributor

dnsmichi commented Sep 8, 2016

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.

  • data-logging-enabled -> trace-logging-enabled
  • [cluster] -> [coordinator]

These are the slice errors:

  • [collectd] -> [[collectd]]
  • [opentsdb] -> [[opentsdb]]

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

@dnsmichi
Copy link
Contributor

dnsmichi commented Sep 8, 2016

PR #47 - for tests:

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

@dgolja
Copy link
Owner

dgolja commented Sep 9, 2016

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

@codec
Copy link

codec commented Sep 12, 2016

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. 😢

@KlavsKlavsen
Copy link

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

@mrh666
Copy link

mrh666 commented Sep 15, 2016

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

@andre-c-andersen
Copy link

Did the 1.0 version get released?

@dgolja
Copy link
Owner

dgolja commented Oct 23, 2016

the module is officially released.

@dgolja dgolja closed this as completed Oct 23, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

7 participants