Skip to content

fix(sandbox): route console logs to stderr#949

Merged
johntmyers merged 1 commit intomainfrom
902-sandbox-stderr-logging/jm
Apr 24, 2026
Merged

fix(sandbox): route console logs to stderr#949
johntmyers merged 1 commit intomainfrom
902-sandbox-stderr-logging/jm

Conversation

@johntmyers
Copy link
Copy Markdown
Collaborator

Summary

Route openshell-sandbox console shorthand logging to stderr so payload-producing commands keep stdout clean.

Related Issue

Closes #902

Changes

  • switched both console shorthand tracing sinks in openshell-sandbox from stdout to stderr
  • updated the /var/log fallback warning to describe stderr-only console logging
  • added a regression test covering startup logs on stderr with clean stdout
  • updated the sandbox custom-container architecture note to match the new behavior

Testing

  • mise run pre-commit passes
  • Unit tests added/updated
  • E2E tests added/updated (if applicable)

Notes:

  • Ran RUSTC_WRAPPER= cargo test -p openshell-sandbox --test stdout_logging successfully.
  • Ran mise run pre-commit, but the workspace test:rust task fails in this environment because openshell-prover cannot link z3 (ld: library 'z3' not found).

Checklist

  • Follows Conventional Commits
  • Commits are signed off (DCO)
  • Architecture docs updated (if applicable)

Signed-off-by: John Myers <johntmyers@users.noreply.github.com>
@johntmyers johntmyers self-assigned this Apr 23, 2026
@johntmyers johntmyers requested a review from a team as a code owner April 23, 2026 22:52
@copy-pr-bot
Copy link
Copy Markdown

copy-pr-bot Bot commented Apr 23, 2026

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

@johntmyers johntmyers merged commit 7f8e210 into main Apr 24, 2026
10 checks passed
@johntmyers johntmyers deleted the 902-sandbox-stderr-logging/jm branch April 24, 2026 14:49
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.

openshell-sandbox tracing to stdout on main/dev corrupts stdout payloads when Landlock is unavailable

3 participants