Merge pull request #17 from Lavaburn/bugfix_open_source

Bugfix: $is_pe fact can be a string.
evenup · Jul 15, 2015 · a8d7e18 · a8d7e18
2 parents 654a35a + e87856f
commit a8d7e18
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 1 deletion.
diff --git a/.fixtures.yml b/.fixtures.yml
@@ -1,4 +1,8 @@
 fixtures:
+  forge_modules:
+    stdlib:
+      repo: "puppetlabs/stdlib"
+      ref: "4.1.0"  
   symlinks:
     graphite_reporter: "#{source_dir}"
 
diff --git a/README.md b/README.md
@@ -11,6 +11,7 @@ Requirements
 
 * `Puppet`
 * A [Graphite](http://graphite.wikidot.com/) server
+* puppetlabs/stdlib (>= 4.0.0) module
 
 Installation & Usage
 --------------------

diff --git a/manifests/params.pp b/manifests/params.pp
@@ -10,7 +10,7 @@
   $graphite_suffix = 'puppet'
   $graphite_reverse_hostname = true
 
-  if $::is_pe {
+  if str2bool($::is_pe) {
     $config_file  = '/etc/puppetlabs/puppet/graphite.yaml'
     $config_owner = 'pe_puppet'
     $config_group = 'pe_puppet'