Skip to content

Conversation

@addaleax
Copy link
Collaborator

In particular, enabling mongodb-runner:verbose debug logs lets you read all server logs.

This should help debug the server setup errors we've been seeing in mongosh CI.

Description

Open Questions

Checklist

In particular, enabling `mongodb-runner:verbose` debug logs
lets you read all server logs.

This should help debug the server setup errors we've been seeing in
mongosh CI.
Promise.allSettled([once(stdout, 'end'), once(stderr, 'end')]).then(
() => outStream.end(),
() => {
/* ignore error */
Copy link
Collaborator

Choose a reason for hiding this comment

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

is this still not ignoring an error?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

.allSettled() never rejects, hence the change in description here – the behavior is similar (but now we always wait for both stdout and stderr to be closed successfully instead of one of them potentially closing because of an error)

@addaleax addaleax merged commit bc923b2 into main Oct 21, 2025
33 checks passed
@addaleax addaleax deleted the mongodb-runner-verbose-logs branch October 21, 2025 09:26
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.

3 participants