Skip to content

Commit 77d4dd9

Browse files
authored
Merge pull request #1018 from adamboutcher/fedora34
Fedora 34 and iptables-compat fix; properly utilising iptables param.
2 parents a22ba0e + 486e4b5 commit 77d4dd9

File tree

2 files changed

+14
-1
lines changed

2 files changed

+14
-1
lines changed

manifests/linux.pp

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -31,6 +31,7 @@
3131
$service_name_v6 = $firewall::params::service_name_v6,
3232
$package_name = $firewall::params::package_name,
3333
$ebtables_manage = false,
34+
$iptables_name = $firewall::params::iptables_name,
3435
) inherits ::firewall::params {
3536
$enable = $ensure ? {
3637
'running' => true,
@@ -45,6 +46,7 @@
4546
}
4647

4748
package { 'iptables':
49+
name => $iptables_name,
4850
ensure => $pkg_ensure,
4951
}
5052

manifests/params.pp

Lines changed: 12 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,14 +11,20 @@
1111
$service_name = 'iptables'
1212
$service_name_v6 = 'ip6tables'
1313
$package_name = undef
14+
$iptables_name = 'iptables'
1415
$sysconfig_manage = true
1516
}
1617
'Fedora': {
1718
$service_name = 'iptables'
1819
$service_name_v6 = 'ip6tables'
19-
if versioncmp($::operatingsystemrelease, '15') >= 0 {
20+
if versioncmp($::operatingsystemrelease, '34') >= 0 {
2021
$package_name = 'iptables-services'
22+
$iptables_name = 'iptables-compat'
23+
} elsif versioncmp($::operatingsystemrelease, '15') >= 0 {
24+
$package_name = 'iptables-services'
25+
$iptables_name = 'iptables'
2126
} else {
27+
$iptables_name = 'iptables'
2228
$package_name = undef
2329
}
2430
$sysconfig_manage = true
@@ -28,23 +34,27 @@
2834
$service_name = ['iptables', 'nftables']
2935
$service_name_v6 = 'ip6tables'
3036
$package_name = ['iptables-services', 'nftables']
37+
$iptables_name = 'iptables'
3138
$sysconfig_manage = false
3239
} elsif versioncmp($::operatingsystemrelease, '7.0') >= 0 {
3340
$service_name = 'iptables'
3441
$service_name_v6 = 'ip6tables'
3542
$package_name = 'iptables-services'
43+
$iptables_name = 'iptables'
3644
$sysconfig_manage = true
3745
} else {
3846
$service_name = 'iptables'
3947
$service_name_v6 = 'ip6tables'
4048
$package_name = 'iptables-ipv6'
49+
$iptables_name = 'iptables'
4150
$sysconfig_manage = true
4251
}
4352
}
4453
}
4554
}
4655
'Debian': {
4756
$service_name_v6 = undef
57+
$iptables_name = 'iptables'
4858
case $::operatingsystem {
4959
'Debian': {
5060
if versioncmp($::operatingsystemrelease, 'unstable') >= 0 {
@@ -79,6 +89,7 @@
7989
$package_name = 'net-firewall/iptables'
8090
}
8191
default: {
92+
$iptables_name = 'iptables'
8293
$service_name_v6 = undef
8394
case $::operatingsystem {
8495
'Archlinux': {

0 commit comments

Comments
 (0)