Arch Linux is not a Red Hat distro

Previously, Arch Linux was incorrectly assumed to be part of the RedHat
osfamily. It actually has its own osfamily: Archlinux. However, this was
added in Facter 1.7.0, and previous versions use an osfamily of Linux,
so we just check the value of operatingsystem.

diff --git a/manifests/params.pp b/manifests/params.pp
index 9d0f64d1ab628d6285d50f52e9bcd780306817de..1e926e7a45ec73e44137be701aeced85060e67cd 100644 (file)
--- a/manifests/params.pp
+++ b/manifests/params.pp
@@ -6,10 +6,6 @@ class firewall::params {
           $service_name = 'iptables'
           $package_name = undef
         }
-        'Archlinux': {
-          $service_name = ['iptables','ip6tables']
-          $package_name = undef
-        }
         'Fedora': {
           if versioncmp($::operatingsystemrelease, '15') >= 0 {
             $package_name = 'iptables-services'
@@ -61,8 +57,16 @@ class firewall::params {
       $package_name = 'net-firewall/iptables'
     }
     default: {
-      $package_name = undef
-      $service_name = 'iptables'
+      case $::operatingsystem {
+        'Archlinux': {
+          $service_name = ['iptables','ip6tables']
+          $package_name = undef
+        }
+        default: {
+          $service_name = 'iptables'
+          $package_name = undef
+        }
+      }
     }
   }
 }

diff --git a/spec/unit/classes/firewall_linux_archlinux_spec.rb b/spec/unit/classes/firewall_linux_archlinux_spec.rb
index cf5b19b0f9867f5ff314ddb754cc7814955fbebd..36dc9251ac696c9bae0813c664caafcbccd26e7a 100644 (file)
--- a/spec/unit/classes/firewall_linux_archlinux_spec.rb
+++ b/spec/unit/classes/firewall_linux_archlinux_spec.rb
@@ -3,7 +3,7 @@ require 'spec_helper'
 describe 'firewall::linux::archlinux', :type => :class do
   let(:facts) do
     {
-      :osfamily        => 'RedHat',
+      :osfamily        => 'Archlinux',
       :operatingsystem => 'Archlinux'
     }
   end