michaelklishin / cassandra-chef-cookbook

Chef cookbook for Apache Cassandra, DataStax Enterprise (DSE) and DataStax agent
Apache License 2.0
163 stars 228 forks source link

Only install jamm if setup requested #363

Closed erebe closed 6 years ago

erebe commented 7 years ago

Hello,

Even if we let the default parameter for setup_jamm to false in https://github.com/michaelklishin/cassandra-chef-cookbook/blob/master/attributes/default.rb#L26, the cookbook try to install it as it compute a version here https://github.com/michaelklishin/cassandra-chef-cookbook/blob/master/attributes/default.rb#L112.

This PR aims to correctly use the attribute setup_jamm in order to decide either or not install a special version