Fix test_duplicate_route.py as per new orchagent crash handling changes#19104
Conversation
|
/azp run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
* Following changes in sonic-net/sonic-swss#3587, test_duplicate_route.py needs to be updated accordingly. * Skip the test temporarily until swss submodule update is complete to avoid failures due to a circular dependency. Signed-off-by: Prabhat Aravind <[email protected]>
prabhataravind
force-pushed
the
paravind/duplicate_route_fix
branch
from
9ece769 to
2d637f7
Compare
|
/azp run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
|
Hi @prabhataravind , do you have a task or issue to track the update of the test case? @xixuej what issue are you referring to? I am not aware of any. |
|
Hi @prabhataravind , you skipped test route/test_duplicate_route.py in this PR. When will the test case be fixed? Also, the master branch also has the same issue, probably you should also create a PR to skip in master branch until the test case can pass? |
@xixuej we should be ok to remove the skip now. The failure in master is due to a different change. sonic-mgmt changes corresponding to orchagent crash handling changes were already merged to master via #19102. The failure you are likely seeing is potentially a different one - #20231 |
4 participants
Following changes in Orchagent SAI error handling improvements sonic-swss#3587, test_duplicate_route.py needs to be updated accordingly.
Skip the test temporarily until swss submodule update is complete to avoid failures due to a circular dependency.
Description of PR
Summary:
Fixes # (issue)
Type of change
Back port request
Approach
What is the motivation for this PR?
How did you do it?
How did you verify/test it?
Any platform specific information?
Supported testbed topology if it's a new test case?
Documentation