Merge pull request #545 from jonnytpuppet/clusterip

Comment out spec test for el7 due to unknown interface name

Author: DavidS

spec/acceptance/firewall_clusterip_spec.rb

@@ -7,41 +7,37 @@
     shell('ip6tables --flush; ip6tables -t nat --flush; ip6tables -t mangle --flush')
   end
 
-    # Have to comment out el-7 as the interface name is inconsistent
-    if default['platform'] !~ /el-7/
-      describe 'clusterip' do
-        context 'cluster ipv4 test' do
-          it 'applies' do
-            pp = <<-EOS
-              class { '::firewall': }
-              firewall {
-                '830 - clusterip test':
-                  chain                 => 'FORWARD',
-                  jump                  => 'CLUSTERIP',
-                  destination           => '1.1.1.1',
-                  iniface               => 'eth0',
-                  clusterip_new         => true,
-                  clusterip_hashmode    => "sourceip",
-                  clusterip_clustermac  => "01:00:5E:00:00:00",
-                  clusterip_total_nodes => "2",
-                  clusterip_local_node  => "1",
-                  clusterip_hash_init   => "1337",
-              }
-            EOS
+  # Have to comment out el-7 as the interface name is inconsistent
+  describe 'clusterip' do
+    context 'cluster ipv4 test' do
+      it 'applies' do
+        pending("MODULES-2124 should be resolved for clusterip RHEL7 support") if default['platform'] =~ /el-7/
+        pp = <<-EOS
+          class { '::firewall': }
+          firewall {
+            '830 - clusterip test':
+              chain                 => 'FORWARD',
+              jump                  => 'CLUSTERIP',
+              destination           => '1.1.1.1',
+              iniface               => 'eth0',
+              clusterip_new         => true,
+              clusterip_hashmode    => "sourceip",
+              clusterip_clustermac  => "01:00:5E:00:00:00",
+              clusterip_total_nodes => "2",
+              clusterip_local_node  => "1",
+              clusterip_hash_init   => "1337",
+          }
+        EOS
 
-            apply_manifest(pp, :catch_failures => true)
-          end
+        apply_manifest(pp, :catch_failures => true)
+      end
 
-          it 'should contain the rule' do
-            shell('iptables-save') do |r|
-              expect(r.stdout).to match(/-A FORWARD -d (1.1.1.1\/32|1.1.1.1) -i eth0 -p tcp -m comment --comment "830 - clusterip test" -j CLUSTERIP --new --hashmode sourceip --clustermac 01:00:5E:00:00:00 --total-nodes 2 --local-node 1 --hash-init 1337/)
-            end
-          end
+      it 'should contain the rule' do
+        pending("MODULES-2124 should be resolved for clusterip RHEL7 support") if default['platform'] =~ /el-7/
+        shell('iptables-save') do |r|
+          expect(r.stdout).to match(/-A FORWARD -d (1.1.1.1\/32|1.1.1.1) -i eth0 -p tcp -m comment --comment "830 - clusterip test" -j CLUSTERIP --new --hashmode sourceip --clustermac 01:00:5E:00:00:00 --total-nodes 2 --local-node 1 --hash-init 1337/)
         end
       end
-    else
-      pending("MODULES-2124 should be resolved for clusterip RHEL7 support") do
-        true.should be(true)
-      end
     end
+  end
 end