Skip to content

fix: rename /etc/navigator policy paths to /etc/openshell#28

Merged
drew merged 1 commit intomainfrom
fix/rename-navigator-policy-paths-to-openshell
Mar 13, 2026
Merged

fix: rename /etc/navigator policy paths to /etc/openshell#28
drew merged 1 commit intomainfrom
fix/rename-navigator-policy-paths-to-openshell

Conversation

@drew
Copy link
Copy Markdown
Collaborator

@drew drew commented Mar 13, 2026

Summary

  • Renames all sandbox policy paths from /etc/navigator/ to /etc/openshell/ to complete the OpenShell rebrand
  • Renames TLS credential paths from /etc/navigator-tls/ to /etc/openshell-tls/

Changed files

File Change
sandboxes/base/Dockerfile COPY policy.yaml /etc/navigator/.../etc/openshell/...
sandboxes/openclaw/Dockerfile COPY policy.yaml /etc/navigator/.../etc/openshell/...
sandboxes/nemoclaw/Dockerfile chown /etc/navigator/etc/openshell
sandboxes/nemoclaw/policy-proxy.js Default policy path + TLS well-known paths
sandboxes/nemoclaw/nemoclaw-start.sh _POLICY_SRC path

Complete the OpenShell rebrand by updating all sandbox policy and TLS
paths from the legacy /etc/navigator(-tls) prefix to /etc/openshell(-tls).
@drew drew merged commit b53684f into main Mar 13, 2026
5 checks passed
factory-octavian pushed a commit to factory-octavian/OpenShell-Community that referenced this pull request Apr 1, 2026
Closes NVIDIA#28

## Summary
- `--update-kube-config` and `--get-kubeconfig` appended a `-remote` suffix to the cluster name when resolving the stored kubeconfig path, but the deploy step writes the kubeconfig under the raw name — so the suffixed path never existed, causing both flags to fail for remote clusters.
- Removed the suffix logic so the lookup path matches the deploy path. The `-remote` suffix is still applied to internal kubeconfig YAML entries (cluster/context/user names) by `rewrite_kubeconfig_remote`, which is unaffected.
- Inlined the `is_remote` variable since it was only used to compute the display-only `location` string.
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