saltstack/salt
1
0
Fork 0
mirror of https://github.com/saltstack/salt.git synced 2025-04-17 10:10:20 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 2

Add permanent argument in firewalld.

Browse source 
As described in #30240 there are some inconsistency between ports and services.
This commit is contained in:
abednarik 2016-01-11 16:03:04 -03:00
parent 54b9641330
commit e3d4bf51da
1 changed files with 14 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

18
salt/modules/firewalld.py
View file

@ -441,7 +441,7 @@ def remove_masquerade(zone):
return __firewall_cmd('--zone={0} --remove-masquerade'.format(zone))
def add_port(zone, port):
def add_port(zone, port, permanent=True):
'''
Allow specific ports in a zone.
@ -456,10 +456,15 @@ def add_port(zone, port):
if not get_masquerade(zone):
add_masquerade(zone)
return __firewall_cmd('--zone={0} --add-port={1}'.format(zone, port))
cmd = '--zone={0} --add-port={1}'.format(zone, port)
if permanent:
cmd += ' --permanent'
return __firewall_cmd(cmd)
def remove_port(zone, port):
def remove_port(zone, port, permanent=True):
'''
Remove a specific port from a zone.
@ -471,7 +476,12 @@ def remove_port(zone, port):
salt '*' firewalld.remove_port internal 443/tcp
'''
return __firewall_cmd('--zone={0} --remove-port={1}'.format(zone, port))
cmd = '--zone={0} --remove-port={1}'.format(zone, port)
if permanent:
cmd += ' --permanent'
return __firewall_cmd(cmd)
def list_ports(zone):