search

hectcastro / chef-graphite

A Chef cookbook to install Graphite.
Apache License 2.0
29 stars 28 forks source link

Adding support for Apache 2.4 #29

Closed defunctknightsurfer closed 9 years ago

defunctknightsurfer commented 9 years ago

This change should allow for Apache 2.4 support of the graphite cookbook. I have tested the cookbook in test kitchen, and tested the change in a working environment.

hectcastro commented 9 years ago

This pull request is failing because of an issue with the cookbook linter (Foodcritic):

FC001: Use strings in preference to symbols to access node attributes: /home/travis/build/hectcastro/chef-graphite/tmp/foodcritic/cookbook/templates/default/graphite.conf.erb:18

For more detail, please see the definition for FC001.

hectcastro commented 9 years ago

Sorry for not noticing the Foodcritic fix commit. Thanks, @pkringle.

:+1: