Fix brackets for 1.0.0-beta3

dgolja / golja-influxdb

Puppet module for InfluxDB

Apache License 2.0

19 stars 73 forks source link

Fix brackets for 1.0.0-beta3 #43

Closed liv3d closed 7 years ago

liv3d commented 7 years ago

1.0.0-beta3 complains about the missing extra backets for:

run.Config.opentsdb
run.Config.collectd

This adds these so Influx can still be started correctly.

Also remove extra white space on a few lines

dgolja commented 7 years ago

tnx for the PR. Would you mind to rebase the change and in the meantime I will try to figure out why tests are failing :)

liv3d commented 7 years ago

Done @n1tr0g, I think 7e263255ade9d76d0659570bf448fac8302bb934 should have fixed the tests failing I believe.

dgolja commented 7 years ago

ok tnx ... I need to do some regression tests on old versions and if double brackets are not compatible I will need to bump the major version of the module.

liv3d commented 7 years ago

Now that Influx 1.0 is out - it's possible that #47 is actually a better fit for this PR.