-
Notifications
You must be signed in to change notification settings - Fork 3.1k
feat(onboard): use OpenShell Docker GPU sandboxes #3001
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
Merged
Merged
Changes from all commits
Commits
Show all changes
72 commits
Select commit
Hold shift + click to select a range
df4f0e8
feat(onboard): use OpenShell docker GPU sandboxes
ericksoa 74fe6eb
fix(onboard): clean up docker gpu check regressions
ericksoa 1e94f2e
fix(test): isolate docker-driver onboarding guards
ericksoa a639877
fix(base): move sandbox images to trixie for openshell
ericksoa dccd4c5
fix(base): include chattr runtime dependency
ericksoa 6b0c902
fix(base): satisfy docker lint for runtime deps
ericksoa 6bc67c8
test(wsl): remove controlling tty in installer atomicity test
ericksoa f803fbc
fix(onboard): reuse docker-driver gateway runtime
ericksoa 9fd1dae
fix(shields): gate docker-driver exec discovery
ericksoa ad613cf
fix(onboard): address docker gpu review feedback
ericksoa 68a8315
fix(e2e): stabilize docker-driver lifecycle tests
ericksoa ff0fbbb
Merge remote-tracking branch 'origin/main' into codex/openshell-docke…
ericksoa 0186100
Merge branch 'main' into codex/openshell-docker-gpu-onboard
ericksoa eca7325
Merge remote-tracking branch 'origin/main' into repair/pr3001-clean-m…
ericksoa d99923c
Merge remote-tracking branch 'origin/main' into codex/openshell-docke…
ericksoa edd1173
ci: force IPv4 for WSL apt setup
ericksoa f6056d5
Merge remote-tracking branch 'origin/main' into codex/openshell-docke…
ericksoa a36c62d
Merge remote-tracking branch 'origin/main' into codex/openshell-docke…
ericksoa 6abace5
fix(cli): preserve onboard passthrough flags
ericksoa f683111
Merge remote-tracking branch 'origin/main' into codex/openshell-docke…
ericksoa 471453e
Merge remote-tracking branch 'origin/main' into codex/openshell-docke…
ericksoa 23afabf
Merge remote-tracking branch 'origin/main' into codex/openshell-docke…
ericksoa a9350d1
Merge remote-tracking branch 'origin/main' into codex/openshell-docke…
ericksoa b07dcc5
Merge remote-tracking branch 'origin/main' into codex/openshell-docke…
ericksoa 57857dc
Merge remote-tracking branch 'origin/main' into codex/openshell-docke…
ericksoa 6fe57b7
fix(onboard): preserve sandbox GPU auto mode on resume
ericksoa 78dc20c
Merge remote-tracking branch 'origin/main' into codex/openshell-docke…
ericksoa f4f38a8
Merge remote-tracking branch 'origin/main' into codex/openshell-docke…
ericksoa fc3dfd4
Merge remote-tracking branch 'origin/main' into codex/openshell-docke…
ericksoa c25df40
Merge remote-tracking branch 'origin/main' into codex/openshell-docke…
ericksoa 0ebfe4c
Merge remote-tracking branch 'origin/main' into codex/openshell-docke…
ericksoa fd3133e
Merge remote-tracking branch 'origin/main' into codex/openshell-docke…
ericksoa 210e9e7
fix(onboard): narrow direct GPU proc writes
ericksoa dd76410
Merge remote-tracking branch 'origin/main' into codex/openshell-docke…
ericksoa 133ab48
fix(agent): bypass base resolver for forced rebuilds
ericksoa e9bf328
fix(onboard): support openshell gateway remove
ericksoa 941f3c1
Merge remote-tracking branch 'origin/main' into codex/openshell-docke…
ericksoa fe3f377
fix(onboard): persist reused gateway during onboard
ericksoa e6f8924
Merge remote-tracking branch 'origin/main' into codex/openshell-docke…
ericksoa 37bd091
fix(hermes): allow current Python runtime in Slack policy
ericksoa effe9a9
fix(e2e): align double-onboard alias with docker gateway
ericksoa 6751206
Merge remote-tracking branch 'origin/main' into codex/openshell-docke…
ericksoa f4c9540
Merge remote-tracking branch 'origin/main' into codex/openshell-docke…
ericksoa c4b608a
Merge remote-tracking branch 'origin/main' into codex/openshell-docke…
ericksoa 85143a3
Merge remote-tracking branch 'origin/main' into codex/openshell-docke…
ericksoa dbc099c
Merge remote-tracking branch 'origin/main' into codex/openshell-docke…
ericksoa 130d74d
Merge remote-tracking branch 'origin/main' into codex/openshell-docke…
ericksoa cd43f8d
Merge remote-tracking branch 'origin/main' into codex/openshell-docke…
ericksoa a43d8f5
fix(messaging): honor OpenShell scoped provider placeholders
ericksoa 2363547
fix(hermes): tolerate runtime-scoped placeholders
ericksoa 782b1b7
test(messaging): quiet loopback rewriter probe
ericksoa 6cd7ddb
test(messaging): retry transient Discord upstream warmup
ericksoa c1a047a
test(messaging): accept scoped Slack placeholders
ericksoa bfe1033
test(messaging): retry Discord proxy warmup responses
ericksoa 28c1091
Merge remote-tracking branch 'origin/main' into codex/openshell-docke…
ericksoa 0eba09a
Merge remote-tracking branch 'origin/main' into codex/openshell-docke…
ericksoa 2bb35c1
fix(onboard): target OpenShell 0.0.37 release
ericksoa 277d481
Merge remote-tracking branch 'origin/main' into codex/openshell-docke…
ericksoa 524cfae
docs(cli): include sandbox GPU onboard flags
ericksoa a88a31e
fix(onboard): restart stale OpenShell docker gateway
ericksoa ec6434b
fix(ci): keep gateway upgrade e2e dev deps
ericksoa 06d942f
Merge remote-tracking branch 'origin/main' into codex/openshell-docke…
ericksoa de2137c
test(hermes): use available python for slack probe
ericksoa 9bccc08
test(e2e): cover OpenShell upgrade cleanup drift
ericksoa 269a378
fix(hermes): persist proxy CA env for Slack probes
ericksoa 4d2c17c
test(hermes): classify Slack Python probe errors
ericksoa 5fc43f9
test(hermes): treat Slack remote close as transient
ericksoa 22f871d
Merge remote-tracking branch 'origin/main' into codex/openshell-docke…
ericksoa 416837e
Merge remote-tracking branch 'origin/main' into codex/openshell-docke…
ericksoa 5bb752a
Merge remote-tracking branch 'origin/main' into codex/openshell-docke…
ericksoa 7ea3e79
fix(e2e): refresh mutable config hash before command exec
ericksoa 1de0193
Merge branch 'main' into codex/openshell-docker-gpu-onboard
cv File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
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
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
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
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
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
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
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
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
Oops, something went wrong.
Oops, something went wrong.
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🧩 Analysis chain
🏁 Script executed:
Repository: NVIDIA/NemoClaw
Length of output: 8361
🏁 Script executed:
Repository: NVIDIA/NemoClaw
Length of output: 145
🏁 Script executed:
Repository: NVIDIA/NemoClaw
Length of output: 41
🏁 Script executed:
Repository: NVIDIA/NemoClaw
Length of output: 369
🏁 Script executed:
Repository: NVIDIA/NemoClaw
Length of output: 88
🏁 Script executed:
rg -n 'HERMES|hermes|BASE_IMAGE|SANDBOX' src/lib/agent-onboard.ts -iRepository: NVIDIA/NemoClaw
Length of output: 3670
🏁 Script executed:
Repository: NVIDIA/NemoClaw
Length of output: 41
🏁 Script executed:
Repository: NVIDIA/NemoClaw
Length of output: 8515
🏁 Script executed:
Repository: NVIDIA/NemoClaw
Length of output: 2051
🏁 Script executed:
Repository: NVIDIA/NemoClaw
Length of output: 1017
🏁 Script executed:
# Check the resolve-hermes-base-image action file completely to see all exports cat .github/actions/resolve-hermes-base-image/action.yamlRepository: NVIDIA/NemoClaw
Length of output: 2275
🏁 Script executed:
Repository: NVIDIA/NemoClaw
Length of output: 41
Export the environment variable name that the onboarding code reads.
The action exports
HERMES_BASE_IMAGE, butagent-onboard.tsresolves pinned base images fromNEMOCLAW_HERMES_SANDBOX_BASE_IMAGE_REF. Unless a workflow remaps the value, the resolved image is never used and the onboarding process falls back to tag discovery or local build.Suggested fix
Also applies to: 62-62
🤖 Prompt for AI Agents