Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add interactive launch mode. #48

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

WhatAmISupposedToPutHere
Copy link
Collaborator

After launching a process via the server, krun immediately closes and the
launched process' std{in,out,err} go nowhere. Add a --interactive flag that
allocates a pseudo-terminal, routes it to the host and connects to it.

Somewhat like '-it' in docker/podman exec.

@teohhanhui
Copy link
Collaborator

@WhatAmISupposedToPutHere Judging from the commit history, does this PR depend on #47? That's still in draft status anyway.

@WhatAmISupposedToPutHere
Copy link
Collaborator Author

Technically yes, but i am not super sure why that one is marked as draft.

asahilina and others added 2 commits September 27, 2024 21:36
Unconditionally forward a range of ports to sockets named
`$XDG_RUNTIME_DIR/krun/socket/port-$PORT`. This allows applications to
dynamically bind to sockets at those paths and have them forwarded to
the guest as vsock ports, after the krun VM is already running.

The port range is currently hardcoded as 50000..50200.

Signed-off-by: Asahi Lina <[email protected]>
Signed-off-by: Sasha Finkelstein <[email protected]>
After launching a process via the server, krun immediately closes and the
launched process' std{in,out,err} go nowhere. Add a --interactive flag that
allocates a pseudo-terminal, routes it to the host and connects to it.

Somewhat like '-it' in docker/podman exec.

Signed-off-by: Sasha Finkelstein <[email protected]>
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.

3 participants