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

Show content emitted by kernel supervisor when it fails to connect #5983

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

jmcphers
Copy link
Collaborator

This change is intended to help troubleshoot #5337, which continues to show up occasionally in CI but for which no one has found a reliable local repro.

It shows any output emitted by the supervisor alongside the connection timeout message, for easy debugging.

@jmcphers jmcphers requested a review from midleman January 14, 2025 03:01
Copy link

E2E Tests 🚀  ?
This PR will run tests tagged with: @critical

// user can see it without clicking over to the logs.
if (fs.existsSync(outFile)) {
// Note that we don't need to append this content
// to the lgos since the output file is already
Copy link
Collaborator

Choose a reason for hiding this comment

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

nit, lgos -> logs

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.

2 participants