Skip to content

fixing DOCKER_HOST problem on macOS docker desktop#138

Closed
nsisodiya wants to merge 1 commit into
NVIDIA:mainfrom
nsisodiya:fix/docker-host
Closed

fixing DOCKER_HOST problem on macOS docker desktop#138
nsisodiya wants to merge 1 commit into
NVIDIA:mainfrom
nsisodiya:fix/docker-host

Conversation

@nsisodiya

Copy link
Copy Markdown

No description provided.

@wscurran

Copy link
Copy Markdown
Contributor

Thanks for the suggested fix. Can you provide a brief explanation of the problem and solution? Thanks!

@cv

cv commented Mar 21, 2026

Copy link
Copy Markdown
Collaborator

Hey @nsisodiya — the DOCKER_HOST fix for macOS Docker Desktop is definitely something our Mac users would benefit from. Since you opened this, we've added CI pipelines and shipped a bunch of new stuff, so the codebase looks pretty different now. Could you rebase this on the latest main when you get a chance? That way we can properly evaluate it. Thanks for the contribution!

mafueee pushed a commit to mafueee/NemoClaw that referenced this pull request Mar 28, 2026
…IA#138)

Closes NVIDIA#27

Add remove() methods to TracingLogBus, PlatformEventBus, and
SandboxWatchBus to clean up entries when sandboxes are deleted.
Wire cleanup into both handle_deleted (K8s reconciler) and
delete_sandbox (gRPC handler). Reorder watch_sandbox to validate
sandbox existence before subscribing to buses, preventing entries
for non-existent IDs. Add one-time sandbox validation at stream
open in push_sandbox_logs.

Co-authored-by: John Myers <johntmyers@users.noreply.github.com>
@wscurran

Copy link
Copy Markdown
Contributor

This PR was labeled status: needs-info on March 20 while we waited for additional context to move forward with review. We haven't heard back in 26 days, so we're closing for now. If you're able to provide the requested information, please feel free to reopen — the contribution is appreciated.

@wscurran wscurran closed this Apr 15, 2026
@wscurran wscurran added needs: info Waiting on author for missing details and removed status: needs-info labels Jun 3, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs: info Waiting on author for missing details

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants