Revert "Allow to remove node['grafana']['datasources'] defaults via…

… roles (fixes #27)"

This reverts commit 846d47e.

CHANGELOG.md

@@ -2,11 +2,6 @@
 
 This file is used to list changes made in each version of grafana.
 
-## Dev:
-
-* Allow removing `node['grafana']['datasources']` defaults via roles attributes
-  only by setting values to `nil`. [#27](https://github.com/JonathanTron/chef-grafana/issues/27)
-
 ## 1.5.2:
 
 * Allow configuration of `default_route` via attributes (Miguel Landaeta) [#26](https://github.com/JonathanTron/chef-grafana/pull/26)

README.md

@@ -80,38 +80,6 @@ to know about the multiple databases it should read data from (`elasticsearch`,
 }
 ```
 
-If you're trying to set values via roles, beware of chef's attributes merging.
-To get rid of `graphite` you would have to set the value to `nil` (or `null`
-if using json's role).
-
-As `ruby`:
-
-```ruby
-name "myrole"
-override_attributes({
-  "grafana" => {
-    "datasources" => {
-      "graphite" => nil
-    }
-  }
-})
-```
-
-as `json`:
-
-```json
-{
-  "name": "myrole",
-  "override_attributes": {
-    "grafana": {
-      "datasources": {
-        "graphite": null
-      }
-    }
-  }
-}
-```
-
 #### kibana::nginx
 
 | Attribute                                       | Default                    | Description                       |

libraries/javascript_pp.rb

@@ -4,7 +4,6 @@ def self.pprint(obj, indent=1)
     when Hash
       res = "{\n"
       obj.each do |k,v|
-        next if v.nil?
         res += '  ' * indent
         res += k.to_s + ': '
         res += pprint(v, indent+1)