]> review.fuel-infra Code Review - puppet-modules/puppetlabs-firewall.git/commitdiff
(IAC-1771) Update code for AlmaLinux
authorDavid Swan <david.swan@puppet.com>
Tue, 19 Oct 2021 09:20:20 +0000 (10:20 +0100)
committerDavid Swan <david.swan@puppet.com>
Tue, 19 Oct 2021 09:32:45 +0000 (10:32 +0100)
lib/puppet/util/firewall.rb
manifests/linux.pp
spec/unit/classes/firewall_linux_redhat_spec.rb

index 06c64c05189f98dd470915fc0ca48578279306ea..ffa83f352315c53b55fb2b513b09e21c1cacec09 100644 (file)
@@ -172,7 +172,8 @@ module Puppet::Util::Firewall
     # Basic normalisation for older Facter
     os_key = Facter.value(:osfamily)
     os_key ||= case Facter.value(:operatingsystem)
-               when 'RedHat', 'CentOS', 'Fedora', 'Scientific', 'SL', 'SLC', 'Ascendos', 'CloudLinux', 'PSBM', 'OracleLinux', 'OVS', 'OEL', 'Amazon', 'XenServer', 'VirtuozzoLinux', 'Rocky'
+               when 'RedHat', 'CentOS', 'Fedora', 'Scientific', 'SL', 'SLC', 'Ascendos', 'CloudLinux',
+                    'PSBM', 'OracleLinux', 'OVS', 'OEL', 'Amazon', 'XenServer', 'VirtuozzoLinux', 'Rocky', 'AlmaLinux'
                  'RedHat'
                when 'Debian', 'Ubuntu'
                  'Debian'
@@ -198,7 +199,7 @@ module Puppet::Util::Firewall
     end
 
     # RHEL 7 and newer also use systemd to persist iptable rules
-    if os_key == 'RedHat' && ['RedHat', 'CentOS', 'Scientific', 'SL', 'SLC', 'Ascendos', 'CloudLinux', 'PSBM', 'OracleLinux', 'OVS', 'OEL', 'XenServer', 'VirtuozzoLinux', 'Rocky']
+    if os_key == 'RedHat' && ['RedHat', 'CentOS', 'Scientific', 'SL', 'SLC', 'Ascendos', 'CloudLinux', 'PSBM', 'OracleLinux', 'OVS', 'OEL', 'XenServer', 'VirtuozzoLinux', 'Rocky', 'AlmaLinux']
        .include?(Facter.value(:operatingsystem)) && Facter.value(:operatingsystemrelease).to_i >= 7
       os_key = 'Fedora'
     end
index a81788b8bd8b8d0a0a8d4fb8682395ef9d7fb545..c782a2c951f86e5d2dae2a81987bdd1df3cf2adc 100644 (file)
@@ -59,7 +59,7 @@ class firewall::linux (
   case $::operatingsystem {
     'RedHat', 'CentOS', 'Fedora', 'Scientific', 'SL', 'SLC', 'Ascendos',
     'CloudLinux', 'PSBM', 'OracleLinux', 'OVS', 'OEL', 'Amazon', 'XenServer',
-    'VirtuozzoLinux', 'Rocky': {
+    'VirtuozzoLinux', 'Rocky', 'AlmaLinux': {
       class { "${title}::redhat":
         ensure          => $ensure,
         ensure_v6       => $_ensure_v6,
index 5055acb5258485851f185e69daa04826c5ad1770..d274ac64fec278100d65c49d502c27f212e5fe82 100644 (file)
@@ -34,7 +34,7 @@ RSpec.shared_examples 'ensures iptables service' do
 end
 
 describe 'firewall::linux::redhat', type: :class do
-  ['RedHat', 'CentOS', 'Fedora'].each do |os|
+  ['RedHat', 'CentOS', 'Fedora', 'AlmaLinux'].each do |os|
     oldreleases = ((os == 'Fedora') ? ['14'] : ['6.5'])
     newreleases = ((os == 'Fedora') ? ['15', 'Rawhide'] : ['7.0.1406'])
     nftablesreleases = ((os == 'Fedora') ? [] : ['8.0'])