diff options
| author | Christian Poessinger <christian@poessinger.com> | 2021-02-26 23:42:31 +0100 |
|---|---|---|
| committer | Christian Poessinger <christian@poessinger.com> | 2021-02-26 23:42:31 +0100 |
| commit | d55ec755c14b421bfaf4074c2ec83d8d16bd784d (patch) | |
| tree | 60ba502b10ac3d7b1a569d7dd272b21143eeabd4 | |
| parent | 66b399b3f7ab5640c341355f6fd3234df661f85a (diff) | |
| download | vyos-1x-d55ec755c14b421bfaf4074c2ec83d8d16bd784d.tar.gz vyos-1x-d55ec755c14b421bfaf4074c2ec83d8d16bd784d.zip | |
smoketest: policy: adjust to FRR 7.3 CLI syntax
No sequence number support in FRR 7.3 for
- community-list
- extcommunity-list
- large-community-list
| -rwxr-xr-x | smoketest/scripts/cli/test_policy.py | 12 |
1 files changed, 3 insertions, 9 deletions
diff --git a/smoketest/scripts/cli/test_policy.py b/smoketest/scripts/cli/test_policy.py index 8efbab7e5..a144d0b88 100755 --- a/smoketest/scripts/cli/test_policy.py +++ b/smoketest/scripts/cli/test_policy.py @@ -370,9 +370,8 @@ class TestPolicy(unittest.TestCase): if 'rule' not in comm_list_config: continue - seq = '5' for rule, rule_config in comm_list_config['rule'].items(): - tmp = f'bgp community-list {comm_list} seq {seq}' + tmp = f'bgp community-list {comm_list}' if rule_config['action'] == 'permit': tmp += ' permit' else: @@ -381,7 +380,6 @@ class TestPolicy(unittest.TestCase): tmp += ' ' + rule_config['regex'] self.assertIn(tmp, config) - seq = int(seq) + 5 def test_extended_community_list(self): test_data = { @@ -430,14 +428,13 @@ class TestPolicy(unittest.TestCase): if 'rule' not in comm_list_config: continue - seq = '5' for rule, rule_config in comm_list_config['rule'].items(): # if the community is not a number but a name, the expanded # keyword is used expanded = '' if not comm_list.isnumeric(): expanded = ' expanded' - tmp = f'bgp extcommunity-list{expanded} {comm_list} seq {seq}' + tmp = f'bgp extcommunity-list{expanded} {comm_list}' if rule_config['action'] == 'permit': tmp += ' permit' @@ -447,7 +444,6 @@ class TestPolicy(unittest.TestCase): tmp += ' ' + rule_config['regex'] self.assertIn(tmp, config) - seq = int(seq) + 5 def test_large_community_list(self): @@ -497,9 +493,8 @@ class TestPolicy(unittest.TestCase): if 'rule' not in comm_list_config: continue - seq = '5' for rule, rule_config in comm_list_config['rule'].items(): - tmp = f'bgp large-community-list expanded {comm_list} seq {seq}' + tmp = f'bgp large-community-list expanded {comm_list}' if rule_config['action'] == 'permit': tmp += ' permit' @@ -509,7 +504,6 @@ class TestPolicy(unittest.TestCase): tmp += ' ' + rule_config['regex'] self.assertIn(tmp, config) - seq = int(seq) + 5 def test_prefix_list(self): |