Merge pull request #341 from superseb/superseb/configure_client_port

Make client port configurable, issue #335

lib/puppet/provider/sensu_client_config/json.rb

@@ -77,6 +77,14 @@ def bind=(value)
     conf['client']['bind'] = value
   end
 
+  def port
+    conf['client']['port']
+  end
+
+  def port=(value)
+    conf['client']['port'] = value
+  end
+
   def subscriptions
     conf['client']['subscriptions'] || []
   end

lib/puppet/type/sensu_client_config.rb

@@ -51,6 +51,11 @@ def insync?(is)
     defaultto '127.0.0.1'
   end
 
+  newproperty(:port) do
+    desc "The port that client will bind to"
+    defaultto '3030'
+  end
+
   newparam(:base_path) do
     desc "The base path to the client config file"
     defaultto '/etc/sensu/conf.d/'

manifests/client/config.pp

@@ -26,6 +26,7 @@
     client_name   => $sensu::client_name,
     address       => $sensu::client_address,
     bind          => $sensu::client_bind,
+    port          => $sensu::client_port,
     subscriptions => $sensu::subscriptions,
     safe_mode     => $sensu::safe_mode,
     custom        => $sensu::client_custom,

manifests/init.pp

@@ -255,6 +255,7 @@
   $api_password                = undef,
   $subscriptions               = [],
   $client_bind                 = '127.0.0.1',
+  $client_port                 = '3030',
   $client_address              = $::ipaddress,
   $client_name                 = $::fqdn,
   $client_custom               = {},

spec/classes/sensu_client_spec.rb

@@ -13,6 +13,8 @@
           :ensure        => 'present',
           :client_name   => 'host.domain.com',
           :address       => '2.3.4.5',
+          :bind          => '127.0.0.1',
+          :port          => '3030',
           :subscriptions => [],
           :ensure        => 'present',
           :custom        => {}
@@ -33,6 +35,8 @@
           :ensure        => 'present',
           :client_name   => 'myclient',
           :address       => '1.2.3.4',
+          :bind          => '127.0.0.1',
+          :port          => '3030',
           :subscriptions => ['all'],
           :ensure        => 'present',
           :safe_mode     => true,