search

dgolja / golja-influxdb

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

Configuration of apt / yum repository #30

Open dhoppe opened 8 years ago

dhoppe commented 8 years ago

Please make this optional. The user should be able to configure its own repository.

https://github.com/n1tr0g/golja-influxdb/blob/master/manifests/server/install.pp#L17

dgolja commented 8 years ago

Did you try to use the manage_install parameter to ignore the module installation or you want to have an way that inside this module you can provide custom repo's ?

dhoppe commented 8 years ago

I just need an additional parameter to enable/disable the include of class repo.pp. In some cases the user might be interested in using his local mirror instead of the public mirror of InfluxData.

dannygoulder commented 8 years ago

agreed, it would be great if the module could be configurable to simply install the influxdb package. In my case I use Foreman to manage local repo mirrors (as the influx servers don't have access to the interwebs)

jbiel commented 8 years ago

+1 to make the upstream repo optional. I'd like to install from my own repository instead.

danielpalstra commented 8 years ago

At my current assignment we manage local repo mirrors. I've made pull request #38 to make repo management optional.