Skip to content

fix: TUI and GUI commands fail - workspace routing, UID mismatch, inference route - #43

Merged
LinoyBitan1 merged 2 commits into
validatedpatterns-sandbox:mainfrom
LinoyBitan1:fix/sandbox-deployment-fixes
Aug 25, 2026
Merged

fix: TUI and GUI commands fail - workspace routing, UID mismatch, inference route#43
LinoyBitan1 merged 2 commits into
validatedpatterns-sandbox:mainfrom
LinoyBitan1:fix/sandbox-deployment-fixes

Conversation

@LinoyBitan1

@LinoyBitan1 LinoyBitan1 commented Aug 24, 2026

Copy link
Copy Markdown
Collaborator

Summary

Fix three bugs that break all four TUI/GUI commands after a fresh validated-pattern deploy.

Jira: APPENG-6158

What's broken on main

After ./pattern.sh make install and make openshell-saw-configure-gateway:

  • make nemoclaw-tui SANDBOX_NAME=cuda-sandboxsandbox not found
  • make nemoclaw-gui SANDBOX_NAME=cuda-sandboxCould not extract dashboard token
  • make openclaw-tui SANDBOX_NAME=notebookUnsafe fallback OpenClaw temp dir
  • make openclaw-gui SANDBOX_NAME=notebookCould not extract token

Fixes

1. Workspace routingnemoclaw-tui, nemoclaw-gui, and openshell-saw-gui.sh pass --workspace to sandbox list but not to ssh-proxy or sandbox exec. The proxy defaults to workspace default and can't find cuda-sandbox in cuda-dev. Fixed by adding --workspace to all connection commands.

2. UID mismatch — The openclaw-openshell image owns /sandbox as UID 65532. The supervisor assigns a dynamic UID at runtime (1000, 998, …). OpenClaw fs-safe requires the temp dir to be owned by the running user — the mismatch crashes it. Fixed by running docker exec -u 0 chown -R sandbox:sandbox /sandbox after the sandbox reaches Ready.

3. Inference 503apply_bom.py calls openshell inference set without --workspace (so only default gets a route) and without --system (so inference.local has no backend). Fixed by passing --workspace on the user route and --system on the system route.

Changes

  • Makefile-quickstart--workspace on ssh-proxy in nemoclaw-tui; pass WORKSPACE to GUI script
  • openshell-saw-gui.sh--workspace on sandbox exec, ssh-proxy, and SSH hostname
  • apply_bom.pychown_sandbox_home() after sandbox Ready; --workspace and --system on inference set
  • README.mdWORKSPACE=cuda-dev on NemoClaw commands; sandbox list --workspace cuda-dev

Verification (fresh deploy)

  • make nemoclaw-tui SANDBOX_NAME=cuda-sandbox WORKSPACE=cuda-dev — connects, inference responds
  • make openclaw-tui SANDBOX_NAME=notebook — starts without UID crash
  • make nemoclaw-gui SANDBOX_NAME=cuda-sandbox WORKSPACE=cuda-dev GUI_PORT=18789 — works
  • make openclaw-gui SANDBOX_NAME=notebook GUI_PORT=18790 — works
  • Setup Job completes 10/10 ALL PASSED

@LinoyBitan1
LinoyBitan1 force-pushed the fix/sandbox-deployment-fixes branch from 12e5835 to 0674111 Compare August 24, 2026 08:47
@LinoyBitan1 LinoyBitan1 changed the title fix: TUI and GUI commands fail fix: TUI and GUI commands fail - workspace routing, UID mismatch, inference route Aug 24, 2026
@LinoyBitan1
LinoyBitan1 marked this pull request as ready for review August 24, 2026 08:51
@LinoyBitan1
LinoyBitan1 force-pushed the fix/sandbox-deployment-fixes branch from 0674111 to 6d6f056 Compare August 24, 2026 08:56
@LinoyBitan1
LinoyBitan1 requested review from mtalvi and sauagarwa and removed request for mtalvi August 25, 2026 05:20

@mtalvi mtalvi left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM!
Very nice work @LinoyBitan1 !
Yes the issue was testing on a fresh cluster vs an existing one.

mtalvi

This comment was marked as duplicate.

@LinoyBitan1
LinoyBitan1 merged commit d2b623a into validatedpatterns-sandbox:main Aug 25, 2026
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