Cong Hou
Cong Hou
> @congh-nvidia can you please handle the conflicts so i can merge ? Hi @liat-grozovik, the conflict is fixed, waiting for the checkers to pass.
PR checker failed in test_duplicate_route.py, failure is caused by PR https://github.com/sonic-net/sonic-mgmt/pull/12460, I have left message for the author. Will retrigger the checker after the issue is fixed.
Hi @liat-grozovik, the checkers passed, could you please help merged? Thanks.
@vaibhavhd Could you help check this issue? Thanks.
Hi @developfast @yxieca, this PR is causing failure in test test_duplicate_route.py which is in the PR checker: https://elastictest.org/scheduler/testplan/661f33b8baaf1d86b899274b?testcase=route%2Ftest_duplicate_route.py%7C%7C%7C2&type=console ``` ==================================== ERRORS ==================================== _______ ERROR at setup of test_duplicate_routes[4-Loopback-vlab-01-None] _______ duthosts...
Hi, I double checked, looks like the test route/test_duplicate_route.py was not running in the checkers of this PR, but it was running in PR https://github.com/sonic-net/sonic-mgmt/pull/12374/
Hi @vaibhavhd currently I don't have time to fix this and also I don't quite understand why the position of logic for starting the sniffer and sender was moved in...