Skip to content

Commit e1aba9e

Browse files
committed
Fix for firewall management when multiple servers are specified
1 parent ed50b9d commit e1aba9e

File tree

1 file changed

+12
-9
lines changed

1 file changed

+12
-9
lines changed

manifests/agent.pp

Lines changed: 12 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -471,15 +471,18 @@
471471

472472
# Manage firewall
473473
if $manage_firewall {
474-
firewall { '150 zabbix-agent':
475-
dport => $listenport,
476-
proto => 'tcp',
477-
action => 'accept',
478-
source => $server,
479-
state => [
480-
'NEW',
481-
'RELATED',
482-
'ESTABLISHED'],
474+
$servers = split($server, ',')
475+
$servers.each |$_server| {
476+
firewall { "150 zabbix-agent from ${_server}":
477+
dport => $listenport,
478+
proto => 'tcp',
479+
action => 'accept',
480+
source => $_server,
481+
state => [
482+
'NEW',
483+
'RELATED',
484+
'ESTABLISHED'],
485+
}
483486
}
484487
}
485488
# the agent doesn't work perfectly fine with selinux

0 commit comments

Comments
 (0)