Remove the require function from config.pp

[PierreR]

Do you really need this require/include ?

puppet-zookeeper/manifests/config.pp

Line 7 in 800fbe2

require ::zookeeper::install

I guess this is because you initialize the datastore at the end of the file ? Still the order is already managed in init.pp and I am wondering (knowing nothing about zookeeper per se) if this initialization really belongs in a config.pp file.

Anyhow this is annoying for use, because we use a testing tool that prohibits the usage of the require function. When we really need it we prefer chaining Class with ->.

Thanks for your support !

[deric]

I think the require statement is redundant in this case, as we already have chaining -> in here