version-2 directory has an unnecessary init check

[minorOffense]

https://github.com/deric/puppet-zookeeper/blob/master/manifests/config.pp#L81 Uses a flag to check if the data directory needs to be initialized. This is unnecessary because the "creates" flag on the exec will automatically know if it needs to run based on the presence of that file.

[deric]

I guess the motivation was that in some cases calling zookeeper-server-initialize might not be considered as dangerous.