Ensure target service is running when using lioadm

Change-Id: I279c981364d2099ded9c969e5d868051e0682753 Closes-Bug: rhbz#1109861 (cherry picked from commit 01d23fe)
openstack · Jul 18, 2014 · 037cace · 037cace
1 parent be74bc6
commit 037cace
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 0 deletions.
diff --git a/manifests/backend/iscsi.pp b/manifests/backend/iscsi.pp
@@ -49,6 +49,12 @@
     }
 
     'lioadm': {
+      service { 'target':
+        ensure  => running,
+        enable  => true,
+        require => Package['targetcli'],
+      }
+
       package { 'targetcli':
         ensure => present,
         name   => $::cinder::params::lio_package_name,

diff --git a/spec/classes/cinder_volume_iscsi_spec.rb b/spec/classes/cinder_volume_iscsi_spec.rb
@@ -60,6 +60,11 @@
     end
 
     it { should contain_package('targetcli').with_ensure('present')}
+    it { should contain_service('target').with(
+      :ensure  => 'running',
+      :enable  => 'true',
+      :require => 'Package[targetcli]'
+    ) }
 
   end