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.

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

tests: OSPFv3 hello tests investigation, make assert output unique #16415

Merged
merged 1 commit into from
Jul 18, 2024

Conversation

nabahr
Copy link
Contributor

@nabahr nabahr commented Jul 17, 2024

These failing tests are hard to track down. Added numbering to each assert to easily tell where the test fails.

@frrbot frrbot bot added the tests Topotests, make check, etc label Jul 17, 2024
These failing tests are hard to track down. Added numbering to each assert
to easily tell where the test fails.

Signed-off-by: Nathan Bahr <[email protected]>
@nabahr nabahr force-pushed the ospfv3_hello_tests branch from ae2a154 to ddc1c2e Compare July 17, 2024 22:03
@Jafaral
Copy link
Member

Jafaral commented Jul 18, 2024

ci:rerun

@Jafaral
Copy link
Member

Jafaral commented Jul 18, 2024

ci:rerun attempt to trigger the test failure under investigation

Copy link
Member

@riw777 riw777 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good

@riw777 riw777 merged commit f0ae1e8 into FRRouting:master Jul 18, 2024
11 checks passed
@nabahr nabahr deleted the ospfv3_hello_tests branch July 23, 2024 21:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
master size/M tests Topotests, make check, etc
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants