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

Upgrade firecracker from v1.1.0 to v.1.4.1 #768

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

Conversation

Kern--
Copy link
Contributor

@Kern-- Kern-- commented Sep 22, 2023

Issue #, if available:

Description of changes:
carry of #757

Upgrade firecracker from v1.1.0 to v.1.4.1

Firecracker diff firecracker-microvm/firecracker@v1.1.0...v1.4.1

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

fangn2 and others added 2 commits September 22, 2023 21:04
Update firecracker-go-sdk dependency

Remove devtool strip which has been removed in latest firecracker release

Signed-off-by: Tony Fang <[email protected]>
Before the change, we relied on Firecracker's devtool to build a custom
kernel with our configs (which add FUSE support). The devtool's
build_kernel function is broken in Firecracker 1.4.1 and is completely
replaced by a non-containerized build script in main.

Rather than trying to get Firecracker's internal dev tooling to work for
our purposes, this change takes just the necessary scripts from
Firecracker to build a kernel in a container.

Signed-off-by: Kern Walster <[email protected]>
@Kern-- Kern-- requested a review from a team as a code owner September 22, 2023 21:09
For some reason, waiting for the task exit before waiting for the
rngtest outside the container causes rngtest's stdin to block forever.

Signed-off-by: Kern Walster <[email protected]>
@swagatbora90
Copy link
Contributor

@Kern-- PR needs a rebase.

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