summaryrefslogtreecommitdiff
path: root/src/conf_mode/qos.py
diff options
context:
space:
mode:
authorChristian Poessinger <christian@poessinger.com>2023-01-04 17:55:06 +0100
committerChristian Poessinger <christian@poessinger.com>2023-01-04 17:55:48 +0100
commit5867d21077d605f1246459881addffa72ef538ff (patch)
tree1712bea1f5ffc897bce2e4cc3e434491ce24ac0d /src/conf_mode/qos.py
parentae8935ce62e55ad047b51bebef8a1d9124ed1826 (diff)
downloadvyos-1x-5867d21077d605f1246459881addffa72ef538ff.tar.gz
vyos-1x-5867d21077d605f1246459881addffa72ef538ff.zip
qos: T4284: replace qdisc/class instead of always adding it
This makes transitions/updates faster and less error prone
Diffstat (limited to 'src/conf_mode/qos.py')
-rwxr-xr-xsrc/conf_mode/qos.py3
1 files changed, 3 insertions, 0 deletions
diff --git a/src/conf_mode/qos.py b/src/conf_mode/qos.py
index 2eb03237c..1fe3b6aa9 100755
--- a/src/conf_mode/qos.py
+++ b/src/conf_mode/qos.py
@@ -187,6 +187,9 @@ def verify(qos):
if queue_lim < max_tr:
raise ConfigError(f'Policy "{policy}" uses queue-limit "{queue_lim}" < max-threshold "{max_tr}"!')
+ if 'default' in policy_config:
+ if 'bandwidth' not in policy_config['default']:
+ raise ConfigError('Bandwidth not defined for default traffic!')
# we should check interface ingress/egress configuration after verifying that
# the policy name is used only once - this makes the logic easier!