Merge pull request #183 from DataDog/jaime/fixdogstatsd

[puppet] dogstatsd should be on by default.

Author: truthbk

manifests/init.pp

@@ -104,7 +104,7 @@
 #       String. Default: empty
 #   $use_dogstatsd
 #       Enables the dogstatsd server
-#       Boolean. Default: false
+#       Boolean. Default: true
 #   $dogstatsd_port
 #       Specifies the port to be used by dogstatsd. Must have use_dogstatsd set
 #       String. Default: empty
@@ -216,7 +216,7 @@
   $pup_port = '',
   $pup_interface = '',
   $pup_url = '',
-  $use_dogstatsd = false,
+  $use_dogstatsd = true,
   $dogstatsd_target = '',
   $dogstatsd_interval = '',
   $dogstatsd_normalize = true,

spec/classes/datadog_agent_spec.rb

@@ -122,7 +122,7 @@
                     'content' => /^# bind_host: localhost\n/,
                     )}
                     it { should contain_file('/etc/dd-agent/datadog.conf').with(
-                    'content' => /^use_dogstatsd: no\n/,
+                    'content' => /^use_dogstatsd: yes\n/,
                     )}
                     it { should contain_file('/etc/dd-agent/datadog.conf').with(
                     'content' => /^dogstatsd_port: 8125\n/,
@@ -341,10 +341,10 @@
                 )}
             end
 
-            context 'with use_dogstatsd set to yes' do
-                let(:params) {{:use_dogstatsd => true}}
+            context 'with use_dogstatsd set to no' do
+                let(:params) {{:use_dogstatsd => false}}
                 it { should contain_file('/etc/dd-agent/datadog.conf').with(
-                    'content' => /^use_dogstatsd: yes\n/,
+                    'content' => /^use_dogstatsd: no\n/,
                 )}
             end
             context 'with dogstatsd_port set to 8126' do