-
Notifications
You must be signed in to change notification settings - Fork 25
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
crash during shutdown doesn't result in failed test #90
Comments
This seems to be related with #87 . Open this issue again if not. |
Are we still seeing this? I'm clearly seeing crashes cause failed tests (see FRRouting/frr#2427 ) donald |
this problem still exists - this should be fixed for master, 5.0 may still have. I created a test PR to demonstrate the issue (FRRouting/frr#2471) the uncaught crash can be seen in @mwinter-osr I think you said this is something you can easily add ... |
Fixed with Topotest PR 101 |
https://ci1.netdef.org/browse/FRR-FRR-1065/artifact/TOPOI386/MemoryLeaks/memleak_bgp_l3vpn_to_bgp_vrf.test_bgp_l3vpn_to_bgp_vrf.txt
The text was updated successfully, but these errors were encountered: