Skip to content

Conversation

ehdr
Copy link
Contributor

@ehdr ehdr commented Sep 16, 2025

Describe your changes

So you can connect to a running Sandbox, or start a shell from a Modal Image ID (in addition to a image registry tag as today).

Changelog

  • Added modal shell support for connecting to a running Sandbox (modal shell sb-id), and starting a shell with a Modal Image ID (--image im-id) (in addition to image registry tag as today).

@mwaskom
Copy link
Contributor

mwaskom commented Sep 16, 2025

Hm, I'm not sure about --image=image-id

@ehdr
Copy link
Contributor Author

ehdr commented Sep 16, 2025

Hm, I'm not sure about --image=image-id

Why not?

@mwaskom
Copy link
Contributor

mwaskom commented Sep 16, 2025

Why not?

Sorry for the drive-by. It seems a bit simpler to me for the positional argument to modal shell accept "any [sensible] Modal ID" and for --image= to continue meaning "a dockerhub image reference, passed to modal.Image.from_registry.

But it's not a super strong opinion.

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.

2 participants