summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorChristian Poessinger <christian@poessinger.com>2020-09-26 16:14:16 +0200
committerChristian Poessinger <christian@poessinger.com>2020-09-26 16:14:16 +0200
commitd489867bc375147793da0ad371169d2154102999 (patch)
tree89411530bc88483938bef4c8b4e5e3907c377549
parent1290706dc0dd81565c0566925ed3951f9fd6ddba (diff)
downloadvyos-1x-d489867bc375147793da0ad371169d2154102999.tar.gz
vyos-1x-d489867bc375147793da0ad371169d2154102999.zip
macsec: vxlan: T2653: bugfix in verify() on lower interface MTU size
Introduced in commit 818a75c024e ("ifconfig: T2653: get_mtu() should return int() for easier comparison") where the variable used in the formatted string has not been adjusted.
-rwxr-xr-xsrc/conf_mode/interfaces-macsec.py2
-rwxr-xr-xsrc/conf_mode/interfaces-vxlan.py2
2 files changed, 2 insertions, 2 deletions
diff --git a/src/conf_mode/interfaces-macsec.py b/src/conf_mode/interfaces-macsec.py
index 0a20a121b..613648adb 100755
--- a/src/conf_mode/interfaces-macsec.py
+++ b/src/conf_mode/interfaces-macsec.py
@@ -97,7 +97,7 @@ def verify(macsec):
lower_mtu = Interface(macsec['source_interface']).get_mtu()
if lower_mtu < (int(macsec['mtu']) + 40):
raise ConfigError('MACsec overhead does not fit into underlaying device MTU,\n' \
- f'{underlay_mtu} bytes is too small!')
+ f'{lower_mtu} bytes is too small!')
return None
diff --git a/src/conf_mode/interfaces-vxlan.py b/src/conf_mode/interfaces-vxlan.py
index 002f40aef..04e258fcf 100755
--- a/src/conf_mode/interfaces-vxlan.py
+++ b/src/conf_mode/interfaces-vxlan.py
@@ -76,7 +76,7 @@ def verify(vxlan):
lower_mtu = Interface(vxlan['source_interface']).get_mtu()
if lower_mtu < (int(vxlan['mtu']) + 50):
raise ConfigError('VXLAN has a 50 byte overhead, underlaying device ' \
- f'MTU is to small ({underlay_mtu} bytes)')
+ f'MTU is to small ({lower_mtu} bytes)')
verify_mtu_ipv6(vxlan)
verify_address(vxlan)