featureprofiles icon indicating copy to clipboard operation
featureprofiles copied to clipboard

RT-1.51: BGP multipath ECMP and RT-1.52 bgp_multipath_wecmp_test.go : Added deviation

Open cprabha opened this issue 1 year ago • 2 comments

Hi,

bgp_multipath_ecmp_test.go 1. I have updated with deviation for max paths. 2. Current script is with one source and one destination. I have updated the destination with all advertised routes and added multiple source to verify load balance.

bgp_multipath_wecmp_test.go

  1. bgp4PeerRoute.ExtendedCommunities().Add() is missing in configure OTG, it will lead to panic failure.
  2. In traffic flow creation, port1 used as rx port, fixed this to start setting rx from port2.

Thanks Prabha

cprabha avatar Jan 13 '24 02:01 cprabha

Pull Request Test Coverage Report for Build 9378621909

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 55.5%

Totals Coverage Status
Change from base Build 9378378586: 0.0%
Covered Lines: 1983
Relevant Lines: 3573

💛 - Coveralls

coveralls avatar Jan 19 '24 18:01 coveralls

Can we fix the Readme. Have run the tests Rt-1.51 and 1.52 and it passes. So once we have the readme passed we can merge it.

Sure, will update README.

cprabha avatar May 15 '24 19:05 cprabha