Skip to content
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.

Sign up for GitHub

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

Jump to bottom

ospf_suppress_fa -> test failure every time for me. #15261

Closed
donaldsharp opened this issue Jan 30, 2024 · 2 comments · Fixed by #15378
Closed

ospf_suppress_fa -> test failure every time for me. #15261

donaldsharp opened this issue Jan 30, 2024 · 2 comments · Fixed by #15378
Labels
triage Needs further investigation
Milestone
10.0

Comments

@donaldsharp
Copy link
Member

donaldsharp commented Jan 30, 2024 •
edited
Loading 

FAILED test_ospf_suppress_fa.py::test_converge_protocols - AssertionError: r1 has not converged
FAILED test_ospf_suppress_fa.py::test_ospf_set_suppress_fa - AssertionError: Unable to get expected initial states
============================================================================= 2 failed in 65.66s (0:01:05) ===================

Git bisect leads me to this commit:

commit 390081329874c4fd8b4a15ffa2a090275b5fc8ac (HEAD, refs/bisect/bad)
Author: Igor Ryzhov <[email protected]>
Date:   Wed Jan 24 23:04:23 2024 +0200

    zebra: convert to mgmtd
    
    Signed-off-by: Igor Ryzhov <[email protected]>
@donaldsharp donaldsharp added the triage Needs further investigation label Jan 30, 2024
@donaldsharp
Copy link
Member Author

test_ospf_suppress_fa.tar.gz

This is the log file

@Jafaral Jafaral added this to the 10.0 milestone Jan 30, 2024
@donaldsharp
Copy link
Member Author

The bgp_evpn_mh topotest fails on the same commit.

@idryzhov idryzhov mentioned this issue Feb 15, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
triage Needs further investigation
Projects
None yet
Milestone
10.0
Development

Successfully merging a pull request may close this issue.

tests: fix ospf_suppress_fa timing issue idryzhov/frr
2 participants
@donaldsharp @Jafaral