Merge d0a1caa into f9f2e7d

spec/acceptance/class_plugin_ovs_stats_spec.rb

@@ -0,0 +1,43 @@
+require 'spec_helper_acceptance'
+
+describe 'collectd::plugin::ovs_stats class' do
+  context 'basic parameters' do
+    # Using puppet_apply as a helper
+    it 'works idempotently with no errors' do
+      pp = <<-EOS
+      class{'collectd':
+        utils => true,
+      }
+      class{ 'collectd::plugin::ovs_stats':
+        port => 6639,
+      }
+      # Add one write plugin to keep logs quiet
+      class{'collectd::plugin::csv':}
+      # Create a socket to query
+      class{'collectd::plugin::unixsock':
+        socketfile  => '/var/run/collectd-sock',
+        socketgroup => 'root',
+      }
+
+      EOS
+      # Run 3 times since the collectd_version
+      # fact is impossible until collectd is
+      # installed.
+      apply_manifest(pp, catch_failures: false)
+      apply_manifest(pp, catch_failures: true)
+      apply_manifest(pp, catch_changes: true)
+      # Wait to get some data
+      shell('sleep 10')
+    end
+
+    describe service('collectd') do
+      it { is_expected.to be_running }
+    end
+
+    # it is expected to unload the ovs_stats plugin, since there
+    # is no ovs running inside the docker container.
+    # At the same time, collectd should continue to run.
+    describe command('collectdctl -s /var/run/collectd-sock listval') do
+      its(:exit_status) { is_expected.to eq 0 }
+  end
+end

spec/classes/collectd_plugin_ovs_stats_spec.rb

@@ -26,7 +26,7 @@
 
         it 'will create config which will contain port configuration' do
           is_expected.to contain_file('ovs_stats.load').with(
-            content: %r{Port 666}
+            content: %r{Port "666"}
           )
         end
 

templates/plugin/ovs_stats.conf.erb

@@ -1,6 +1,6 @@
 <Plugin "ovs_stats">
 <%- if @port %>
-  Port <%= @port %>
+  Port "<%= @port %>"
 <%- end -%>
 <%- if @address %>
   Address "<%= @address %>"