chore(sync): rebase fork onto upstream/main (2026-05-27)#6
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Rebase fork delta (11 commits, cred-inject + bind-mounts + stop-start + SandboxPhase::Stopped) onto upstream NVIDIA/OpenShell
main(36 commits since last sync 2026-05-20).Notable upstream commits absorbed:
a3b16c18feat(auth): per-sandbox authentication to gateway (feat(auth): per-sandbox authentication to gateway NVIDIA/OpenShell#1404)bdaa08fbfix(server): add ConnectSupervisor/RelayStream to SANDBOX_METHODS (fix(server): add ConnectSupervisor and RelayStream to SANDBOX_METHODS NVIDIA/OpenShell#1475)68d42805fix(docker): use host-gateway callbacks on macOS (fix(docker): use host-gateway callbacks on macOS NVIDIA/OpenShell#1516)e7f965a9refactor(sandbox,driver-vm): start moving to rustix (refactor(sandbox,driver-vm): Start moving to rustix (esp over libc unsafe) NVIDIA/OpenShell#1505)528fb291fix(sandbox): allow first-label L7 host wildcards (fix(sandbox): allow first-label L7 host wildcards NVIDIA/OpenShell#1304)9857fa19refactor: deduplicate shared code across ocsf builders + driver crates (refactor: deduplicate shared code across ocsf builders and driver crates NVIDIA/OpenShell#1526)Proto field collisions
Upstream's
a3b16c18addedDriverSandboxSpec.sandbox_token = 11, which collided with fork'sDriverSandboxSpec.volumes = 11(bind-mounts). Resolved by renumbering fork'svolumesto field 12 inproto/compute_driver.proto. No openshell.proto changes affected (fork'sSandboxSpec.volumes = 11is still safe — upstream stopped at 10).Fork's cred-inject 9000+ fields remain unchanged; upstream's new fields stayed in low ranges.
Conflicts resolved
proto/compute_driver.proto— renumbered fork'svolumesto 12crates/openshell-driver-docker/src/lib.rs— merged sandbox_token mount + volumes mount inbuild_binds; kept upstream'sResult<Vec<String>, Status>signaturecrates/openshell-driver-docker/src/tests.rs— kept both fixtures (sandbox_token+volumes), kept upstream'sbuild_binds(&sandbox, &config)arg order, updated fork's new test to matchcrates/openshell-driver-podman/src/container.rs— addedimage_sandbox_userparam tobuild_container_spec_with_token; updated test wrappercrates/openshell-driver-podman/src/driver.rs— combined fork'simage_sandbox_userresolution with upstream'sbuild_container_spec_with_tokencallcrates/openshell-server/src/compute/mod.rs— included bothsandbox_tokenandvolumesindriver_sandbox_spec_from_publiccrates/openshell-cli/tests/sandbox_create_lifecycle_integration.rs— added missing&[]volumes arg in a new test added bya3b16c18.github/workflows/{e2e-label-help,required-ci-gates}.yml— re-deleted (fork's chore commit drops NVIDIA mirror plumbing)Build + tests
Test plan
system z3on Mac (Z3_SYS_Z3_HEADER+LIBRARY_PATH)OPENSHELL_FORK_TAGbump