Skip to content

feat(openshell-saw): switch OpenShell images to ODH/quay.io builds - #24

Open
mtalvi wants to merge 1 commit into
validatedpatterns-sandbox:mainfrom
mtalvi:images
Open

feat(openshell-saw): switch OpenShell images to ODH/quay.io builds#24
mtalvi wants to merge 1 commit into
validatedpatterns-sandbox:mainfrom
mtalvi:images

Conversation

@mtalvi

@mtalvi mtalvi commented Aug 12, 2026

Copy link
Copy Markdown
Collaborator

Related Ticket:
APPENG-5962

Summary

Replaces the upstream NVIDIA OpenShell images (ghcr.io/nvidia/openshell/*) and upstream PyPI CLI package with Red Hat's ODH builds (quay.io/opendatahub/odh-openshell-*) and the RHAIV pip index, aligning the pattern with Red Hat's supported supply chain. Values-only change — no template or script changes required.

Value Before (NVIDIA upstream) After (ODH/RHAIV)
openshell.gatewayImage ghcr.io/nvidia/openshell/gateway:0.0.96 quay.io/opendatahub/odh-openshell-gateway:v0.0.99-rhaiv.0
openshell.supervisorImage ghcr.io/nvidia/openshell/supervisor:0.0.96 quay.io/opendatahub/odh-openshell-supervisor:v0.0.99-rhaiv.0
openshell.version (pip) 0.0.96 0.0.99+rhaiv.0
openshell.pipIndexUrl "" (PyPI default) https://packages.redhat.com/api/pypi/public-rhai/rhoai/3.6-EA1/cpu-ubi9-test/simple/

Old NVIDIA values are kept as commented-out fallback lines for easy rollback.

Why

OpenShell itself does not require the NVIDIA images — the "missing Docker-driver binaries" issue that originally motivated using them is a NemoClaw-specific preflight check, not an OpenShell limitation, and is already handled by the existing NEMOCLAW_OPENSHELL_GATEWAY_BIN/NEMOCLAW_OPENSHELL_SANDBOX_BIN overrides in this chart. Switching to ODH aligns with Red Hat's supported supply chain and enables consistent versioning across the stack.

Testing performed

Tested end-to-end on a live OpenShift cluster using a fresh, isolated sandbox (kept separate from any existing sandbox so nothing else was disturbed). All image builds/pushes during testing went to a personal quay repo, never the shared team registry.

Headline result: the previously-reported ODH supervisor permission-denied blocker (EACCES on /sandbox/.openclaw/openclaw.json and /sandbox/.profile, reported against v0.0.96-rhaiv.0) does not reproduce with v0.0.99-rhaiv.0. Verified directly: created a sandbox from the real NemoClaw sandbox image with the ODH supervisor installed, confirmed the process runs as the unprivileged sandbox user (not root), both previously-broken config files are readable, and OpenClaw starts cleanly. No image-level mitigation (e.g. changes to the sandbox image's chown step) was needed — the fix appears to have already landed upstream between v0.0.96 and v0.0.99.

Picture:
dashboard-0 99

Final status against acceptance criteria

  • All three OpenShell components (gateway, supervisor, CLI) use ODH/quay.io builds
  • openshell-gateway --version shows the RHAIV version (0.0.99-rhaiv.0)
  • openshell --version shows the RHAIV version (0.0.99+rhaiv.0)
  • openshell provider create works
  • openshell sandbox create works, sandbox reaches Ready
  • TUI works (make openshell-saw-tui) — including a live end-to-end chat round-trip through a real inference provider
  • GUI/dashboard works — correct Keycloak/PKCE login redirect
  • External CLI works (openshell --gateway-insecure sandbox list)
  • Setup completes (OK/PARTIAL — same as today)
  • Version is configurable via values.yaml
  • ODH supervisor EACCES blocker does not reproduce with v0.0.99-rhaiv.0 — no mitigation needed

Rollback

Swap the comment markers back in charts/openshell-saw/values.yaml to restore the NVIDIA image lines (kept in the file, commented out).

Replace upstream NVIDIA OpenShell gateway/supervisor/CLI images with
Red Hat's ODH builds (quay.io/opendatahub/odh-openshell-*, RHAIV pip
index) to align with the supported supply chain. Pinned to
v0.0.99-rhaiv.0, which resolves the supervisor privilege-dropping
issue previously reported against v0.0.96-rhaiv.0.

Co-authored-by: Cursor <cursoragent@cursor.com>
@mtalvi
mtalvi requested a review from sauagarwa August 12, 2026 16:48
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.

1 participant