diff options
| author | Daniil Baturin <daniil@vyos.io> | 2023-10-08 17:14:08 +0100 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2023-10-08 17:14:08 +0100 |
| commit | 07758d372bbc9b3633e29476afa95bc2861bee15 (patch) | |
| tree | af51c0a604609874fe0a8d367fca92238d7657f5 /src | |
| parent | da4006c2a784ff06cf3af3aad6adee7fef8a5330 (diff) | |
| parent | ab2aeec41a2ef34730b9dc8894b184e0bf00b147 (diff) | |
| download | vyos-1x-07758d372bbc9b3633e29476afa95bc2861bee15.tar.gz vyos-1x-07758d372bbc9b3633e29476afa95bc2861bee15.zip | |
Merge pull request #2347 from c-po/equuleus
pppoe: T5630: allow to specify MRU in addition to already configurable MTU (backport #2335)
Diffstat (limited to 'src')
| -rwxr-xr-x | src/conf_mode/interfaces-pppoe.py | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/src/conf_mode/interfaces-pppoe.py b/src/conf_mode/interfaces-pppoe.py index 6c4c6c95b..49714c558 100755 --- a/src/conf_mode/interfaces-pppoe.py +++ b/src/conf_mode/interfaces-pppoe.py @@ -59,6 +59,11 @@ def verify(pppoe): if {'connect_on_demand', 'vrf'} <= set(pppoe): raise ConfigError('On-demand dialing and VRF can not be used at the same time') + # both MTU and MRU have default values, thus we do not need to check + # if the key exists + if int(pppoe['mru']) > int(pppoe['mtu']): + raise ConfigError('PPPoE MRU needs to be lower then MTU!') + return None def generate(pppoe): |