diff options
| author | Jonathan Voss <jvoss@onvox.net> | 2023-07-13 14:15:47 -0500 |
|---|---|---|
| committer | Jonathan Voss <jvoss@onvox.net> | 2023-07-13 16:26:19 -0500 |
| commit | 960cace189d7ace2bea0968646b1348b415e0363 (patch) | |
| tree | a3e2f787741b1a4e78e7b166a43a55d6de6da3ad | |
| parent | ac60fe7d1840b8768542ee4b3f28f46544c290f2 (diff) | |
| download | vyos-1x-960cace189d7ace2bea0968646b1348b415e0363.tar.gz vyos-1x-960cace189d7ace2bea0968646b1348b415e0363.zip | |
policy: T5357: only delete migrated BGP community rules
| -rwxr-xr-x | src/migration-scripts/policy/3-to-4 | 8 |
1 files changed, 5 insertions, 3 deletions
diff --git a/src/migration-scripts/policy/3-to-4 b/src/migration-scripts/policy/3-to-4 index bae30cffc..49e0b4c38 100755 --- a/src/migration-scripts/policy/3-to-4 +++ b/src/migration-scripts/policy/3-to-4 @@ -51,7 +51,7 @@ def community_migrate(config: ConfigTree, rule: list[str]) -> bool: :rtype: bool """ community_list = list((config.return_value(rule)).split(" ")) - config.delete(rule) + if 'none' in community_list: config.set(rule + ['none']) return False @@ -61,8 +61,10 @@ def community_migrate(config: ConfigTree, rule: list[str]) -> bool: community_action = 'add' community_list.remove('additive') for community in community_list: - config.set(rule + [community_action], value=community, - replace=False) + if len(community): + config.set(rule + [community_action], value=community, + replace=False) + config.delete(rule) if community_action == 'replace': return False else: |