docs: add reconnect/restart steps after reboot to troubleshooting guide#474
docs: add reconnect/restart steps after reboot to troubleshooting guide#474RitikKadyan wants to merge 1 commit intoNVIDIA:mainfrom
Conversation
Adds a 'Reconnecting after a reboot' section to the troubleshooting guide with step-by-step instructions for restoring a NemoClaw session after a machine restart. Fixes NVIDIA#469
📝 WalkthroughWalkthroughDocumentation enhancement adding a "Reconnecting after a reboot" troubleshooting section to guide users through recovery steps, including Docker verification, sandbox inspection, and reconnection procedures using nemoclaw commands. Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~8 minutes Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
📝 Coding Plan
Comment Tip CodeRabbit can scan for known vulnerabilities in your dependencies using OSV Scanner.OSV Scanner will automatically detect and report security vulnerabilities in your project's dependencies. No additional configuration is required. |
There was a problem hiding this comment.
🧹 Nitpick comments (1)
docs/reference/troubleshooting.md (1)
163-163: Use active voice in this sentence.“has been removed entirely” is passive; please rewrite in active voice to match the docs style requirements.
As per coding guidelines, "Active voice required. Flag passive constructions."
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@docs/reference/troubleshooting.md` at line 163, Rewrite the sentence "This shows whether your sandbox still exists but is stopped, or has been removed entirely." to use active voice; replace the passive clause "has been removed entirely" with an active construction such as "you have removed it entirely" so the full sentence reads: "This shows whether your sandbox still exists but is stopped, or you have removed it entirely."
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Nitpick comments:
In `@docs/reference/troubleshooting.md`:
- Line 163: Rewrite the sentence "This shows whether your sandbox still exists
but is stopped, or has been removed entirely." to use active voice; replace the
passive clause "has been removed entirely" with an active construction such as
"you have removed it entirely" so the full sentence reads: "This shows whether
your sandbox still exists but is stopped, or you have removed it entirely."
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
Run ID: 24318169-7da9-46c2-b756-d5da9ca5533e
📒 Files selected for processing (1)
docs/reference/troubleshooting.md
|
Thanks for adding reconnect/restart steps to the troubleshooting guide, this should help users get back up and running after a reboot. |
Of course! |
…0.0.15 EXPERIMENTAL — POC branch to validate three-tier config resolution: 1. Frozen openclaw.json (gateway.auth.token, CORS — always immutable) 2. Policy defaults (config_overrides in openclaw-sandbox.yaml) 3. User runtime overrides (nemoclaw config-set → overrides file → hot-reload) OpenClaw shim patch (patches/openclaw-config-overrides.patch): - Adds OPENCLAW_CONFIG_OVERRIDES_FILE env var support to config loader - Deep-merges overrides onto frozen config, stripping gateway.* for security - Adds overrides file to chokidar watcher for hot-reload OpenShell minimum bumped to v0.0.15: - Auto-TLS termination (PR #544) — removes need for tls: terminate - Security hardening SEC-002–010 (PR #548) - Runtime settings channel (PR #474) - Version check now enforced in onboard preflight Policy changes: - Remove 35 deprecated tls: terminate annotations (base + all presets) - Remove permissive wildcard L7 rules from claude_code/nvidia endpoints - Add config_overrides section defining mutable fields + defaults New commands: - nemoclaw <sandbox> config-set --key <path> --value <value> - nemoclaw <sandbox> config-get [--key <path>]
|
Hey @RitikKadyan — thanks for working on this! Just a heads-up that we have an overlapping PR at #911 that also addresses #469. After digging into the
Since both PRs target the same issue, it might make sense to consolidate. Happy to coordinate either way! |
* feat(gateway/sandbox): add global and sandbox runtime settings flow
Adds a "Reconnecting after a reboot" section to the troubleshooting guide with step-by-step instructions for restoring a NemoClaw session after a machine restart.
Fixes #469
Summary
Adds a "Reconnecting after a reboot" section to the troubleshooting guide with step-by-step instructions for restoring a NemoClaw session after a machine restart. Also clarifies the existing "Sandbox shows as stopped" entry to distinguish reboot scenarios from other causes.
Related Issue
Fixes #469
Changes
docs/reference/troubleshooting.mdopenshell sandbox listandnemoclaw status, reconnecting withnemoclaw <name> connect, re-running onboard if needed, and restarting auxiliary servicesType of Change
Testing
make checkpasses.npm testpasses.make docsbuilds without warnings. (for doc-only changes)Note: These steps are based on the existing CLI reference and README documentation. I have not tested the full reboot flow end-to-end on a live NemoClaw installation. Would appreciate a maintainer verifying the exact reconnect behavior before merging.
Checklist
General
Code Changes
make formatapplied (TypeScript and Python).Doc Changes
update-docsagent skill to draft changes while complying with the style guide. For example, prompt your agent with "/update-docscatch up the docs for the new changes I made in this PR."Summary by CodeRabbit