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 support for force vsock on firecracker pilot #35

Merged
merged 2 commits into from
Jan 29, 2024

Conversation

schaefi
Copy link
Contributor

@schaefi schaefi commented Jan 26, 2024

The vsock communication between guest and host was established with the firecracker pilot only for resume type registrations. For those it's required because you cannot mux one serial terminal between parallel applications running at the same time in the guest. So in resume mode each call has its own socket and channels. However, there is no reason to not also support that type of data flow between guest and host for one time calls which fires up the VM, starts the application and shuts down the VM. This commit adds support for forcing the vsock communication also for the simple app registration. The default stays on serial though.

@schaefi schaefi self-assigned this Jan 26, 2024
The vsock communication between guest and host was established
with the firecracker pilot only for resume type registrations.
For those it's required because you cannot mux one serial
terminal between parallel applications running at the same
time in the guest. So in resume mode each call has its own
socket and channels. However, there is no reason to not also
support that type of data flow between guest and host for one
time calls which fires up the VM, starts the application and
shuts down the VM. This commit adds support for forcing the
vsock communication also for the simple app registration.
The default stays on serial though.
@schaefi schaefi force-pushed the support_vsock_for_one_time_calls branch from b8c9f1c to 84cf185 Compare January 26, 2024 11:50
Add information about console settings and limitations with
regards to firecracker VMs
@schaefi schaefi merged commit 1bca764 into main Jan 29, 2024
7 checks passed
@schaefi schaefi deleted the support_vsock_for_one_time_calls branch January 29, 2024 14:46
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