From 960cace189d7ace2bea0968646b1348b415e0363 Mon Sep 17 00:00:00 2001 From: Jonathan Voss Date: Thu, 13 Jul 2023 14:15:47 -0500 Subject: policy: T5357: only delete migrated BGP community rules --- src/migration-scripts/policy/3-to-4 | 8 +++++--- 1 file 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: -- cgit v1.2.3