Add support for influxdb 1.0.0 #47

Merged
merged 1 commit into from Sep 19, 2016

Projects

None yet

3 participants

@dnsmichi
Contributor
dnsmichi commented Sep 8, 2016 edited

Contains all required configuration changes. One breaking change inside which is explicitly mentioned in the README.md (might need some adoption, I'm not sure about your release process :)).

This patch has been properly tested with the icinga2x-influxdb Vagrant box over here: https://github.com/Icinga/icinga-vagrant

ref #46

This was referenced Sep 8, 2016
@n1tr0g

I use semantric versionin so this will be version 4.0.0

@n1tr0g
Owner
n1tr0g commented Sep 19, 2016

thank you again for the PR !! Nice job

@n1tr0g
n1tr0g approved these changes Sep 19, 2016 View changes
@n1tr0g
n1tr0g approved these changes Sep 19, 2016 View changes
@n1tr0g n1tr0g merged commit f6c3fb6 into n1tr0g:master Sep 19, 2016

1 check failed

continuous-integration/travis-ci/pr The Travis CI build failed
Details
@dnsmichi
Contributor

Thank for the great module making my life more easy :) That's the least I can help out :) When I get more time I'd look into managing databases as well, currently solved via shell script after provisioning in our vagrant box.

@dhoppe
Contributor
dhoppe commented Sep 20, 2016

Please publish a new release to get the support of InfluxDB 1.0.0 to the Forge.

@n1tr0g
Owner
n1tr0g commented Sep 20, 2016

I will do it hopefully today, if time permitted. I need to do more testing to make sure I don't release a broken version :(

@dhoppe
Contributor
dhoppe commented Sep 20, 2016

@n1tr0g Let me help you. I created several pull requests regarding Puppet lint, Rubocop and Travis CI testing. #49 #50 #51

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment