matthew/sonic-buildimage
1
0
Fork 0
Code Issues Pull Requests Actions 1 Packages Projects Releases Wiki Activity

ASN 0 for BGP monitor (#18322)

Browse Source 
### Why I did it
BGP monitor asn is 0 in production, and we should have test to cover this configuration.

### How I did it
Update sonic-config-engine unit test, BGP monitor uses asn 0.

#### How to verify it
Run sonic-config-engine unit test.
This commit is contained in:
ganglv 2024-03-12 02:46:23 +08:00 committed by GitHub
parent 1ed3644e6a
commit 8a780ca7e9
No account linked to committer's email address
2 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
src/sonic-config-engine/tests/simple-sample-graph.xml
View File

@ -72,7 +72,7 @@
</PeeringSessions> </PeeringSessions>
<Routers xmlns:a="http://schemas.datacontract.org/2004/07/Microsoft.Search.Autopilot.Evolution"> <Routers xmlns:a="http://schemas.datacontract.org/2004/07/Microsoft.Search.Autopilot.Evolution">
<a:BGPRouterDeclaration> <a:BGPRouterDeclaration>
<a:ASN>1</a:ASN> <a:ASN>0</a:ASN>
<a:BgpGroups/> <a:BgpGroups/>
<a:Hostname>BGPMonitor</a:Hostname> <a:Hostname>BGPMonitor</a:Hostname>
<a:Peers> <a:Peers>

2
src/sonic-config-engine/tests/test_cfggen.py
View File

@ -720,7 +720,7 @@ class TestCfgGen(TestCase):
output = self.run_script(argument) output = self.run_script(argument)
self.assertEqual( self.assertEqual(
utils.to_dict(output.strip()), utils.to_dict(output.strip()),
utils.to_dict("{'10.20.30.40': {'rrclient': 0, 'name': 'BGPMonitor', 'local_addr': '10.1.0.32', 'nhopself': 0, 'holdtime': '10', 'asn': '1', 'keepalive': '3'}}") utils.to_dict("{'10.20.30.40': {'rrclient': 0, 'name': 'BGPMonitor', 'local_addr': '10.1.0.32', 'nhopself': 0, 'holdtime': '10', 'asn': '0', 'keepalive': '3'}}")
) )
def test_minigraph_bgp_voq_chassis_peer(self): def test_minigraph_bgp_voq_chassis_peer(self):