-
Notifications
You must be signed in to change notification settings - Fork 543
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fixed DPB system test cases to bring stability #1795
base: master
Are you sure you want to change the base?
Conversation
cfbc8ce
to
962d4f0
Compare
This pull request introduces 1 alert when merging 962d4f0 into c37cc1c - view on LGTM.com new alerts:
|
This pull request introduces 1 alert when merging 2021663 into c37cc1c - view on LGTM.com new alerts:
|
Signed-off-by: Sangita Maity <samaity@linkedin.com>
@samaity , almost all DPB tests started failing. Can you please take a look asap as this is failing on all PR builds
|
@prsunny, this is strange. It was perfectly fine a few days back. I am looking into this. We didn't make any change in the test file in recent times. I believe, some changes in the building code are failing these test cases as DPB is a highly dispersed topic. will get back soon with my findings. Thanks! |
@samaity , i agree we dint have recent changes related to DPB and this looks strange that many tests started failing. Could you check the logs (syslogs/swss.rec) from the build pipeline and see if there is any issue. You can get the logs from here. Look for log@1/log/test_port_dpb |
/azp run |
Commenter does not have sufficient privileges for PR 1795 in repo Azure/sonic-swss |
Signed-off-by: Sangita Maity samaity@linkedin.com
What I did
Fixed DPB system test cases to bring stability. Used
setup_db
for all dvs database and checked route interface only during Test.Why I did it
to bring stability to test cases.
How I verified it
Details if related