sonic-mgmt
sonic-mgmt copied to clipboard
Published
20 hours ago •
sonic-net
sonic-net
[action] [PR:12346] [test_po_cleanup] Do a safe config reload
Description of PR
Summary: Fixes # (issue)
Type of change
- [x] Bug fix
- [ ] Testbed and Framework(new/improvement)
- [ ] Test case(new/improvement)
Back port request
- [ ] 201911
- [ ] 202012
- [ ] 202205
- [ ] 202305
- [ ] 202311
Approach
What is the motivation for this PR?
For test_po_cleanup and test_po_cleanup_after_reload, do a safe reload and wait for BGP to come up. This makes sure that the next test case that runs gets the testbed in a good, stable state.
This is particularly needed on slow-CPU devices where a proper config reload will take some time to run.
How did you do it?
How did you verify/test it?
Verified that two runs of pc/test_po_cleanup.py on MSN2700 passed. This doesn't necessarily guarantee it won't ever fail because of this error, though, since it seems to be somewhat flaky.
![azure-pipelines[bot] avatar](https://avatars.githubusercontent.com/in/9426?v=4 "Published by a pub.dev verified publisher"){kind=small}