diff options
author | Christian Breunig <christian@breunig.cc> | 2024-01-22 15:01:08 +0100 |
---|---|---|
committer | GitHub <noreply@github.com> | 2024-01-22 15:01:08 +0100 |
commit | c1bc037af3c0b1aa5287fd5cc9bf21a7253372af (patch) | |
tree | e6b3668caa8cfd52c510e7b947e85d9fb82ab564 /src/conf_mode | |
parent | fd4838266592448ea1be9ed693e297300ea7b1d2 (diff) | |
parent | 8362b19bbfac3cc41d3fe5b85e43a0d9374b98a8 (diff) | |
download | vyos-1x-c1bc037af3c0b1aa5287fd5cc9bf21a7253372af.tar.gz vyos-1x-c1bc037af3c0b1aa5287fd5cc9bf21a7253372af.zip |
Merge pull request #2872 from vyos/mergify/bp/sagitta/pr-2852
T5958: QoS add basic implementation of policy shaper-hfsc (backport #2852)
Diffstat (limited to 'src/conf_mode')
-rwxr-xr-x | src/conf_mode/qos.py | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/conf_mode/qos.py b/src/conf_mode/qos.py index ad4121a49..40d7a6c16 100755 --- a/src/conf_mode/qos.py +++ b/src/conf_mode/qos.py @@ -149,7 +149,7 @@ def verify(qos): if 'class' in policy_config: for cls, cls_config in policy_config['class'].items(): # bandwidth is not mandatory for priority-queue - that is why this is on the exception list - if 'bandwidth' not in cls_config and policy_type not in ['priority_queue', 'round_robin']: + if 'bandwidth' not in cls_config and policy_type not in ['priority_queue', 'round_robin', 'shaper_hfsc']: raise ConfigError(f'Bandwidth must be defined for policy "{policy}" class "{cls}"!') if 'match' in cls_config: for match, match_config in cls_config['match'].items(): @@ -173,7 +173,7 @@ def verify(qos): if 'default' not in policy_config: raise ConfigError(f'Policy {policy} misses "default" class!') if 'default' in policy_config: - if 'bandwidth' not in policy_config['default'] and policy_type not in ['priority_queue', 'round_robin']: + if 'bandwidth' not in policy_config['default'] and policy_type not in ['priority_queue', 'round_robin', 'shaper_hfsc']: raise ConfigError('Bandwidth not defined for default traffic!') # we should check interface ingress/egress configuration after verifying that |