Weighted ecmp HLD #738
Weighted ecmp HLD #738
Conversation
|
|
caizhenghui-juniper
changed the title
|
|
||
| #### Unit Test cases | ||
|
|
||
| #### System Test cases |
There was a problem hiding this comment.
please describe system test cases.
There was a problem hiding this comment.
Will add:
set up ECMP route with weight through config_db static_route table(e.g. two routes in NHG with weight of 2,8).
Send traffic with enough tuple variations and observe traffic load unevenly distributed on the two routes in the NHG as (20% and 80%).
| * @flags CREATE_AND_SET | ||
| * @default 1 | ||
| */ | ||
| SAI_NEXT_HOP_GROUP_MEMBER_ATTR_WEIGHT, |
There was a problem hiding this comment.
what is the sai does not support this attribute, what would be the system behavior?
There was a problem hiding this comment.
When vendor's SAI don't support, it will be ECMP vs WECMP.
There was a problem hiding this comment.
Check when SAI is supported and handling it accordingly.
yxieca
force-pushed
the
master
branch
2 times, most recently
from
8498931
to
8837dc2
Compare
|
sonic-net/sonic-swss#1752 code PR. |
|
@caizhenghui-juniper can you please help to sign the EasyCLA? Thanks. |
|
@zhangyanzhao code PR is merged. Any reason not to merge HLD? |
No description provided.