diff options
author | khramshinr <khramshinr@gmail.com> | 2024-04-16 15:49:19 +0800 |
---|---|---|
committer | khramshinr <khramshinr@gmail.com> | 2024-04-16 15:49:19 +0800 |
commit | da40bd2b2a826986de128354ea1bfc041ada0016 (patch) | |
tree | 29ccba696202cd9fe4bfcfad1b88e67a99bbfc97 /src/conf_mode | |
parent | a6ccf358c7148781be438c4a2f89468ebfe5d48f (diff) | |
download | vyos-1x-da40bd2b2a826986de128354ea1bfc041ada0016.tar.gz vyos-1x-da40bd2b2a826986de128354ea1bfc041ada0016.zip |
qos: T4248: Allow to remove the only rule from the qos class
Diffstat (limited to 'src/conf_mode')
-rwxr-xr-x | src/conf_mode/qos.py | 23 |
1 files changed, 23 insertions, 0 deletions
diff --git a/src/conf_mode/qos.py b/src/conf_mode/qos.py index 3dfb4bab8..ccfc8f6b8 100755 --- a/src/conf_mode/qos.py +++ b/src/conf_mode/qos.py @@ -70,6 +70,22 @@ def get_shaper(qos, interface_config, direction): return (map_vyops_tc[shaper_type], shaper_config) + +def _clean_conf_dict(conf): + """ + Delete empty nodes from config e.g. + match ADDRESS30 { + ip { + source {} + } + } + """ + if isinstance(conf, dict): + return {node: _clean_conf_dict(val) for node, val in conf.items() if val != {} and _clean_conf_dict(val) != {}} + else: + return conf + + def get_config(config=None): if config: conf = config @@ -120,6 +136,13 @@ def get_config(config=None): if 'queue_limit' not in qos['policy'][policy][p_name]['precedence'][precedence]: qos['policy'][policy][p_name]['precedence'][precedence]['queue_limit'] = \ str(int(4 * max_thr)) + # cleanup empty match config + if 'class' in p_config: + for cls, cls_config in p_config['class'].items(): + if 'match' in cls_config: + cls_config['match'] = _clean_conf_dict(cls_config['match']) + if cls_config['match'] == {}: + del cls_config['match'] return qos |