openconfig / lemming

An OpenConfig reference device implementation
Apache License 2.0
31 stars 9 forks source link

Correctly populate community route attributes in BGP RIBs. #379

Closed wenovus closed 7 months ago

wenovus commented 7 months ago

Sync fix from upstream GoBGP and add missing unit tests.

Also added sanity community set checks into integration_test/prefix_test to avoid creating another integration test that tests the same logic. Added TODO for race conditions discovered that are triggered by subtests when BGP policy and routes are cleared and re-created. This will block policy logic tests that involve re-advertising the same route under different policies, but will work for tests that are checking a static set of policies, which is the current set of tests I'm looking at to support.

github-actions[bot] commented 7 months ago

Pull Request Test Coverage Report for Build 8346921455

Details


Changes Missing Coverage Covered Lines Changed/Added Lines %
bgp/gobgp.go 11 62 17.74%
<!-- Total: 14 65 21.54% -->
Totals Coverage Status
Change from base Build 8334633740: 0.001%
Covered Lines: 17397
Relevant Lines: 650868

💛 - Coveralls