]> review.fuel-infra Code Review - puppet-modules/puppetlabs-firewall.git/commitdiff
Excluding SLES from CLUSTERIP tests.
authorJonathan Tripathy <jt@puppetlabs.com>
Thu, 18 Jun 2015 10:05:38 +0000 (11:05 +0100)
committerJonathan Tripathy <jt@puppetlabs.com>
Thu, 18 Jun 2015 10:05:38 +0000 (11:05 +0100)
spec/acceptance/firewall_clusterip_spec.rb

index d8f0ad644c64a067465ad7203ccdb98e33aaca5d..bdd601745687381ae2306f728aa15896565195c5 100644 (file)
@@ -7,35 +7,37 @@ describe 'firewall type', :unless => UNSUPPORTED_PLATFORMS.include?(fact('osfami
     shell('ip6tables --flush; ip6tables -t nat --flush; ip6tables -t mangle --flush')
   end
 
-  # 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
+  # SLES doesn't have the CLUSTERIP module
+  if default['platform'] !~ /sles/
+    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
-        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/)
+        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
       end
     end