Skip to content

Conversation

@enochtangg
Copy link
Contributor

Small fix for improving logging in the integration test. Instead of dumping all consumer logs (GHA refuses to print all logs), when an error log is seen, capture 10 lines before and after the error and print to stdout.

@enochtangg enochtangg requested a review from a team December 10, 2024 19:35
@john-z-yang
Copy link
Contributor

john-z-yang commented Dec 11, 2024

Also I'm seeing more of

ERROR taskbroker: Task grpc_server failed: GRPC server task failed: tonic::transport::Error(Transport, Os { code: 98, kind: AddrInUse, message: "Address already in use" })

Which really sucks

@markstory
Copy link
Member

I wonder if we're not releasing the ports that grpc is connected to quickly enough?

@enochtangg enochtangg merged commit 9d44086 into main Dec 12, 2024
6 checks passed
@enochtangg enochtangg deleted the fix-logging-integration-test branch December 12, 2024 22:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants