Add ensure parameter to jenkins::slave

Allows running the slave manually instead of a service, useful in Docker for instance

manifests/slave.pp

@@ -75,6 +75,7 @@
   $disable_ssl_verification = false,
   $labels                   = undef,
   $install_java             = $jenkins::params::install_java,
+  $ensure                   = 'running',
   $enable                   = true
 ) inherits jenkins::params {
 
@@ -192,7 +193,7 @@
   }
 
   service { 'jenkins-slave':
-    ensure     => running,
+    ensure     => $ensure,
     enable     => $enable,
     hasstatus  => true,
     hasrestart => true,

spec/classes/jenkins_slave_spec.rb

@@ -5,7 +5,7 @@
   shared_context 'a jenkins::slave catalog' do
     it { should contain_exec('get_swarm_client') }
     it { should contain_file('/etc/init.d/jenkins-slave') }
-    it { should contain_service('jenkins-slave') }
+    it { should contain_service('jenkins-slave').with(:enable => true, :ensure => 'running') }
     it { should contain_user('jenkins-slave_user').with_uid(nil) }
     # Let the different platform blocks define  `slave_runtime_file` separately below
     it { should contain_file(slave_runtime_file).with_content(/-fsroot \/home\/jenkins-slave/) }
@@ -25,6 +25,11 @@
       let(:params) { {:slave_home => home } }
       it { should contain_file(slave_runtime_file).with_content(/-fsroot #{home}/) }
     end
+
+    describe 'with service disabled' do
+      let(:params) { {:enable => false, :ensure => 'stopped' } }
+      it { should contain_service('jenkins-slave').with(:enable => false, :ensure => 'stopped') }
+    end
   end
 
   describe 'RedHat' do