diff options
| author | Viacheslav Hletenko <v.gletenko@vyos.io> | 2023-02-18 11:06:07 +0000 |
|---|---|---|
| committer | Viacheslav Hletenko <v.gletenko@vyos.io> | 2023-02-18 13:50:23 +0000 |
| commit | 87fb9be4cab3a261406c69c723add7467e4ef1fa (patch) | |
| tree | e04eac03d5fe315ffe4e1ced87a6d12323cac751 | |
| parent | b4d25eafdba1150176f71598a770bc694b51054e (diff) | |
| download | vyos-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.py | 10 |
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, ' \ |