afonso/core-extra
1
0
Fork 0
Code Issues Pull requests Projects Releases Wiki Activity

create ebtables chains as needed

Browse source 
* otherwise every switch gets a chain, causing problems with simultaneous running Python scripts
This commit is contained in:
Jeff Ahrenholz 2019-10-31 13:00:46 -07:00
parent 19300c7564
commit 85c926ff47
1 changed files with 15 additions and 16 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

17
daemon/core/nodes/network.py
View file

@ -181,7 +181,7 @@ class EbtablesQueue:
def ebchange(self, wlan):
"""
Flag a change to the given WLAN"s _linked dict, so the ebtables
Flag a change to the given WLAN's _linked dict, so the ebtables
chain will be rebuilt at the next interval.
:return: nothing
@ -197,8 +197,13 @@ class EbtablesQueue:
:return: nothing
"""
with wlan._linked_lock:
if wlan.has_ebtables_chain:
# flush the chain
self.cmds.append(f"-F {wlan.brname}")
else:
wlan.has_ebtables_chain = True
self.cmds.extend([f"-N {wlan.brname} -P {wlan.policy}",
f"-A FORWARD --logical-in {wlan.brname} -j {wlan.brname}"])
# rebuild the chain
for netif1, v in wlan._linked.items():
for netif2, linked in v.items():
@ -297,14 +302,7 @@ class CoreNetwork(CoreNetworkBase):
:raises CoreCommandError: when there is a command exception
"""
self.net_client.create_bridge(self.brname)
# create a new ebtables chain for this bridge
cmds = [
f"{EBTABLES_BIN} -N {self.brname} -P {self.policy}",
f"{EBTABLES_BIN} -A FORWARD --logical-in {self.brname} -j {self.brname}",
]
ebtablescmds(self.host_cmd, cmds)
self.has_ebtables_chain = False
self.up = True
def shutdown(self):
@ -320,6 +318,7 @@ class CoreNetwork(CoreNetworkBase):
try:
self.net_client.delete_bridge(self.brname)
if self.has_ebtables_chain:
cmds = [
f"{EBTABLES_BIN} -D FORWARD --logical-in {self.brname} -j {self.brname}",
f"{EBTABLES_BIN} -X {self.brname}",