Refactor iscsi initiator-name reset into separate profile

This currently assumes nova-compute and iscsid run in the same context which
isn't true for a containerized deployment

Change-Id: I91f1ce7625c351745dbadd84b565d55598ea5b59

2 files changed, 42 insertions, 9 deletions

diff --git a/spec/classes/tripleo_profile_base_iscsid_spec.rb b/spec/classes/tripleo_profile_base_iscsid_spec.rb
new file mode 100644
index 0000000..04f3c2c
--- /dev/null
+++ b/spec/classes/tripleo_profile_base_iscsid_spec.rb
@@ -0,0 +1,42 @@
+#
+# Copyright (C) 2017 Red Hat, Inc.
+#
+# Licensed under the Apache License, Version 2.0 (the "License"); you may
+# not use this file except in compliance with the License. You may obtain
+# a copy of the License at
+#
+#      http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an "AS IS" BASIS, WITHOUT
+# WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the
+# License for the specific language governing permissions and limitations
+# under the License.
+#
+
+require 'spec_helper'
+
+describe 'tripleo::profile::base::iscsid' do
+  shared_examples_for 'tripleo::profile::base::iscsid' do
+    context 'default params' do
+      let(:params) { { :step => 2, } }
+
+      it {
+        is_expected.to contain_package('iscsi-initiator-utils')
+        is_expected.to contain_exec('reset-iscsi-initiator-name')
+        is_expected.to contain_file('/etc/iscsi/.initiator_reset')
+      }
+    end
+  end
+
+
+  on_supported_os.each do |os, facts|
+    context "on #{os}" do
+      let(:facts) do
+        facts.merge({ :hostname => 'node.example.com' })
+      end
+
+      it_behaves_like 'tripleo::profile::base::iscsid'
+    end
+  end
+end
diff --git a/spec/classes/tripleo_profile_base_nova_compute_spec.rb b/spec/classes/tripleo_profile_base_nova_compute_spec.rb
index 545a1fa..e491ed9 100644
--- a/spec/classes/tripleo_profile_base_nova_compute_spec.rb
+++ b/spec/classes/tripleo_profile_base_nova_compute_spec.rb
@@ -27,9 +27,6 @@ describe 'tripleo::profile::base::nova::compute' do
         is_expected.to_not contain_class('tripleo::profile::base::nova')
         is_expected.to_not contain_class('nova::compute')
         is_expected.to_not contain_class('nova::network::neutron')
-        is_expected.to_not contain_package('iscsi-initiator-utils')
-        is_expected.to_not contain_exec('reset-iscsi-initiator-name')
-        is_expected.to_not contain_file('/etc/iscsi/.initiator_reset')
       }
     end
 
@@ -52,9 +49,6 @@ eos
           is_expected.to contain_class('tripleo::profile::base::nova')
           is_expected.to contain_class('nova::compute')
           is_expected.to contain_class('nova::network::neutron')
-          is_expected.to contain_package('iscsi-initiator-utils')
-          is_expected.to contain_exec('reset-iscsi-initiator-name')
-          is_expected.to contain_file('/etc/iscsi/.initiator_reset')
           is_expected.to_not contain_package('nfs-utils')
         }
       end
@@ -68,9 +62,6 @@ eos
           is_expected.to contain_class('tripleo::profile::base::nova')
           is_expected.to contain_class('nova::compute')
           is_expected.to contain_class('nova::network::neutron')
-          is_expected.to contain_package('iscsi-initiator-utils')
-          is_expected.to contain_exec('reset-iscsi-initiator-name')
-          is_expected.to contain_file('/etc/iscsi/.initiator_reset')
           is_expected.to contain_package('nfs-utils')
         }
       end