]> review.fuel-infra Code Review - puppet-modules/puppetlabs-firewall.git/commitdiff
Fix support for Fedora Rawhide
authorLukas Bezdicka <lbezdick@redhat.com>
Mon, 28 Apr 2014 10:31:56 +0000 (12:31 +0200)
committerLukas Bezdicka <lbezdick@redhat.com>
Tue, 29 Apr 2014 12:50:13 +0000 (14:50 +0200)
On Fedora facter $::operatingsystemrelease can be integer version of Fedora or string Rawhide.

manifests/linux/redhat.pp
spec/unit/classes/firewall_linux_spec.rb

index a4c00b6476bf6f5183066dd21453d7b85d9c992d..f697d211b9f660dfb0164798b68076d1b00f2d3f 100644 (file)
@@ -26,7 +26,7 @@ class firewall::linux::redhat (
     }
   }
 
-  if $::operatingsystem == Fedora and $::operatingsystemrelease >= 15 {
+  if ($::operatingsystem == 'Fedora' and (( $::operatingsystemrelease =~ /^\d+/ and $::operatingsystemrelease >= 15 ) or $::operatingsystemrelease == "Rawhide")) {
     package { 'iptables-services':
       ensure => present,
     }
index 7c76bd0657fc472f90acc8e827e10e7e665c587e..42056c1b1af0528bd97c99a0c75506f5621293e4 100644 (file)
@@ -7,7 +7,7 @@ describe 'firewall::linux', :type => :class do
   context 'RedHat like' do
     %w{RedHat CentOS Fedora}.each do |os|
       context "operatingsystem => #{os}" do
-        releases = (os == 'Fedora' ? [14,15] : [6,7])
+        releases = (os == 'Fedora' ? [14,15,'Rawhide'] : [6,7])
         releases.each do |osrel|
           context "operatingsystemrelease => #{osrel}" do
             let(:facts) { facts_default.merge({ :operatingsystem => os,