sonic-swss icon indicating copy to clipboard operation
sonic-swss copied to clipboard

BFD Tx, Rx interval support for Vnet Monitored routes.

Open siqbal1986 opened this issue 1 year ago • 4 comments

What I did This change add support for rx_monitor_timer tx_monitor_timer check_directly_connected Why I did it This change is beneficial because it adds support for BFD Tx, Rx interval for Vnet Monitored routes as required by DASH project. How I verified it Added a unit test to verify the functionality. Details if related The Tx and Rx values are added on per-route basis. Therefore the same values are used for each nexthop. if the user needs to se different values for each next hop then they can add the values and nexthops incrementally. e.g.

Add route with nexthop 10.0.0.1(1s,1s) -> BFD(10.0.0.1) with Tx =1s Rx= 1s
Update same route and add another nexthop 10.0.0.2(2s,2s) -> BFD(10.0.0.1) with Tx =1s Rx= 1s |  BFD(10.0.0.2) with Tx =2s Rx= 2s

The user can also update these values by selectively removing and readding a nexthop with different Tx and Rx values.

siqbal1986 avatar Oct 20 '24 20:10 siqbal1986

Commenter does not have sufficient privileges for PR 3335 in repo sonic-net/sonic-swss

azure-pipelines[bot] avatar Oct 21 '24 18:10 azure-pipelines[bot]

/azpw run

siqbal1986 avatar Oct 21 '24 18:10 siqbal1986

/AzurePipelines run

mssonicbld avatar Oct 21 '24 18:10 mssonicbld

Azure Pipelines successfully started running 1 pipeline(s).

azure-pipelines[bot] avatar Oct 21 '24 18:10 azure-pipelines[bot]

/azp run

mssonicbld avatar Jan 11 '25 23:01 mssonicbld

Pull request contains merge conflicts.

azure-pipelines[bot] avatar Jan 11 '25 23:01 azure-pipelines[bot]