Skip to content

docs, examples: OpenShell and Substrate on one mesh, on your Kubernetes or on GKE - #746

Closed
aojea wants to merge 1 commit into
google:docs-openshell-sandbox-guidefrom
aojea:ntk-openshell-to-substrate
Closed

aojea wants to merge 1 commit into
google:docs-openshell-sandbox-guidefrom
aojea:ntk-openshell-to-substrate

Conversation

@aojea

@aojea aojea commented Oct 18, 2026 •

Copy link
Copy Markdown
Collaborator

Closed; superseded by a later pull request.

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Code Review

This pull request introduces a comprehensive guide and reproducible examples for running NVIDIA OpenShell sandboxes and Agent Substrate actors on a single Agent Mesh, including a new Kubernetes-based OpenShell example and updates to existing examples to support remote hubs and version pinning. The review feedback identifies a security issue in both the openshell-kubernetes and substrate-actor Makefiles, where sensitive tokens are generated with insecure file permissions (umask 022). It is recommended to restrict these permissions using umask 077 to ensure the tokens are readable only by the owner.

Comment thread development/examples/openshell-kubernetes/Makefile Outdated
Comment thread development/examples/substrate-actor/Makefile Outdated
@aojea aojea closed this Oct 18, 2026
@aojea
aojea deleted the ntk-openshell-to-substrate branch October 18, 2026 12:54
@aojea
aojea restored the ntk-openshell-to-substrate branch October 18, 2026 13:42
@aojea aojea reopened this Oct 18, 2026
@aojea
aojea force-pushed the ntk-openshell-to-substrate branch from 188ee3f to 78b3725 Compare October 18, 2026 13:43
@aojea aojea closed this Oct 18, 2026
@aojea
aojea deleted the ntk-openshell-to-substrate branch October 18, 2026 13:43
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