-
Notifications
You must be signed in to change notification settings - Fork 367
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
Add error output to integration test TestGoBGPLifecycle #6532
base: main
Are you sure you want to change the base?
Add error output to integration test TestGoBGPLifecycle #6532
Conversation
0c24a77
to
c123344
Compare
c123344
to
7c40f02
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
The job failed but the newly added error seems not helpful: https://github.com/antrea-io/antrea/actions/runs/10264321176/job/28398120609?pr=6532 |
It should be caused by the |
Maybe I need to refactor the test. I found this: osrg/gobgp#2262. There might be something wrong when starting multiple BGP processes in a host. |
7c40f02
to
53673ee
Compare
The testGoBGPLifecycle gets failed occasionally, and it is not easy to reproduce the failure manually. Adding some error output to which the error occurs may help to resolve the issue. Signed-off-by: Hongliang Liu <[email protected]>
53673ee
to
7c78d7b
Compare
This PR is stale because it has been open 90 days with no activity. Remove stale label or comment, or this will be closed in 90 days. You can add a label "lifecycle/frozen" to skip stale checking. |
For #6501
The testGoBGPLifecycle gets failed occasionally, and it is not easy to reproduce the failure manually. Adding some error output to which the error occurs may help to resolve the issue.