summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorViacheslav Hletenko <v.gletenko@vyos.io>2023-02-18 11:06:07 +0000
committerViacheslav Hletenko <v.gletenko@vyos.io>2023-02-18 13:50:23 +0000
commit87fb9be4cab3a261406c69c723add7467e4ef1fa (patch)
treee04eac03d5fe315ffe4e1ced87a6d12323cac751
parentb4d25eafdba1150176f71598a770bc694b51054e (diff)
downloadvyos-1x-87fb9be4cab3a261406c69c723add7467e4ef1fa.tar.gz
vyos-1x-87fb9be4cab3a261406c69c723add7467e4ef1fa.zip
T5011: Set default values for min_mtu max_mtu
Some interface drivers don't support/provide min_mtu and max_mtu values For example VyOS in docker container with 'veth' driver on some platforms As a workarund add default values for min/max MTU for calculations and pass function "verify_mtu(config)"
-rw-r--r--python/vyos/configverify.py10
1 files changed, 8 insertions, 2 deletions
diff --git a/python/vyos/configverify.py b/python/vyos/configverify.py
index fcc8cc733..8fddd91d0 100644
--- a/python/vyos/configverify.py
+++ b/python/vyos/configverify.py
@@ -36,8 +36,14 @@ def verify_mtu(config):
mtu = int(config['mtu'])
tmp = Interface(config['ifname'])
- min_mtu = tmp.get_min_mtu()
- max_mtu = tmp.get_max_mtu()
+ # Not all interfaces support min/max MTU
+ # https://vyos.dev/T5011
+ try:
+ min_mtu = tmp.get_min_mtu()
+ max_mtu = tmp.get_max_mtu()
+ except: # Fallback to defaults
+ min_mtu = 68
+ max_mtu = 9000
if mtu < min_mtu:
raise ConfigError(f'Interface MTU too low, ' \