ok


On Mon, Jan 6, 2014 at 3:09 PM, Steven Cary <notifications@github.com>wrote:

> In attributes/default.rb:
>
> > @@ -12,9 +12,9 @@
> >  default[:exhibitor][:install_dir] = "/opt/exhibitor"
> >
> >
> > -default[:exhibitor][:snapshot_dir] = "/tmp/zookeeper_snapshots"
> > -default[:exhibitor][:transaction_dir] = "/tmp/zookeeper_transactions"
> > -default[:exhibitor][:log_index_dir] = "/tmp/zookeeper_log_indexes"
> > +default[:exhibitor][:snapshot_dir] = "/var/lib/zookeeper_snapshots"
> > +default[:exhibitor][:transaction_dir] = "/var/lib/zookeeper_transactions"
> > +default[:exhibitor][:log_index_dir] = "/var/lib/zookeeper_log_indexes"
>
> hi gina, I think we should override these in the exhibitor role instead of
> modifying the default attributes:
> https://github.com/index/chef-repo/blob/master/roles/exhibitor_instance.rb
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/index/chef-zookeeper/pull/3/files#r8681081>
> .
>