feat: upgrade default Firecracker v1.12 version (#2245)

bchalios · web-flow · commit 26a0e91b7f32 · 2026-03-30T00:04:38.000-07:00
* feat: upgrade default Firecracker v1.12 version We updated Firecracker version v1.12 to fix an issue with the first /init call to envd[1]. Use that one as the default v1.12 version. Also, make smoke tests look for the new format of Firecracker artifacts which includes an architecture suffix[2]. [1] e2b-dev/fc-versions@09c01f1 [2] e2b-dev/fc-versions@f612c4a Signed-off-by: Babis Chalios <babis.chalios@e2b.dev> * fix(smoke-test): use correct asset name for FC v1.10 binaries FC v1.10 binary releases were just building a single binary (firecracker) since we were only building for x86. Now we build for both x86 and ARM and we upload both binaries (with appropriate suffixes). Take that into account for as long as we support Firecracker v1.10. Once we stop supporting those, we can drop the workaround. Signed-off-by: Babis Chalios <babis.chalios@e2b.dev> * fix: create-build command Firecracker download Firecracker releases now include both x86 and arm64 binaries with -amd64 and -arm64 suffixes respectively in the fc-version release assets. The only exception is v1.10 for which we still use an old release. Teach create-build command how to download artifacts using these asset names. Signed-off-by: Babis Chalios <babis.chalios@e2b.dev> --------- Signed-off-by: Babis Chalios <babis.chalios@e2b.dev>
diff --git a/.github/actions/build-sandbox-template/action.yml b/.github/actions/build-sandbox-template/action.yml
@@ -8,7 +8,7 @@ runs:
       env:
         TEMPLATE_ID: "2j6ly824owf4awgai1xo"
         KERNEL_VERSION: "vmlinux-6.1.158"
-        FIRECRACKER_VERSION: "v1.12.1_a41d3fb"
+        FIRECRACKER_VERSION: "v1.12.1_210cbac"
       run: |
         # Generate an unique build ID for the template for this run
         export BUILD_ID=$(uuidgen)
diff --git a/packages/orchestrator/cmd/create-build/main.go b/packages/orchestrator/cmd/create-build/main.go
@@ -441,7 +441,15 @@ func setupFC(ctx context.Context, dir, version string) error {
 		return nil
 	}
 
-	fcURL := fmt.Sprintf("https://github.com/e2b-dev/fc-versions/releases/download/%s/firecracker", version)
+	// Old releases in https://github.com/e2b-dev/fc-versions/releases don't build
+	// x86_64 and aarch64 binaries. They just build the former and the asset's name
+	// is just 'firecracker'
+	// TODO: Drop this work-around once we remove support for Firecracker v1.10
+	assetName := "firecracker-amd64"
+	if strings.HasPrefix(version, "v1.10") {
+		assetName = "firecracker"
+	}
+	fcURL := fmt.Sprintf("https://github.com/e2b-dev/fc-versions/releases/download/%s/%s", version, assetName)
 	fmt.Printf("⬇ Downloading Firecracker %s...\n", version)
 
 	return download(ctx, fcURL, dstPath, 0o755)
diff --git a/packages/orchestrator/cmd/smoketest/smoke_test.go b/packages/orchestrator/cmd/smoketest/smoke_test.go
@@ -8,6 +8,7 @@ import (
 	"os"
 	"os/exec"
 	"path/filepath"
+	"strings"
 	"testing"
 	"time"
 
@@ -362,8 +363,18 @@ func downloadKernel(t *testing.T, dataDir string) {
 
 func downloadFC(t *testing.T, dataDir, version string) {
 	t.Helper()
+
+	// Old releases in https://github.com/e2b-dev/fc-versions/releases don't build
+	// x86_64 and aarch64 binaries. They just build the former and the asset's name
+	// is just 'firecracker'
+	// TODO: Drop this work-around once we remove support for Firecracker v1.10
+	assetName := "firecracker-amd64"
+	if strings.HasPrefix(version, "v1.10") {
+		assetName = "firecracker"
+	}
+
 	dst := filepath.Join(dataDir, "fc-versions", version, "firecracker")
-	url := fmt.Sprintf("https://github.com/e2b-dev/fc-versions/releases/download/%s/firecracker", version)
+	url := fmt.Sprintf("https://github.com/e2b-dev/fc-versions/releases/download/%s/%s", version, assetName)
 	downloadFile(t, url, dst, 0o755)
 }
 
diff --git a/packages/shared/pkg/featureflags/flags.go b/packages/shared/pkg/featureflags/flags.go
@@ -240,7 +240,7 @@ const (
 // TODO: The short tag here has only 7 characters — the one from our build pipeline will likely have exactly 8 so this will break.
 const (
 	DefaultFirecackerV1_10Version = "v1.10.1_30cbb07"
-	DefaultFirecackerV1_12Version = "v1.12.1_a41d3fb"
+	DefaultFirecackerV1_12Version = "v1.12.1_210cbac"
 	DefaultFirecrackerVersion     = DefaultFirecackerV1_12Version
 )
 
@@ -259,7 +259,7 @@ var (
 )
 
 // ResolveFirecrackerVersion resolves the firecracker version using the FirecrackerVersions feature flag.
-// The buildVersion format is "v1.12.1_a41d3fb" — we extract "v1.12" as the lookup key.
+// The buildVersion format is "v1.12.1_210cbac" — we extract "v1.12" as the lookup key.
 func ResolveFirecrackerVersion(ctx context.Context, ff *Client, buildVersion string) string {
 	parts := strings.Split(buildVersion, "_")
 	if len(parts) < 2 {
diff --git a/tests/integration/seed.go b/tests/integration/seed.go
@@ -208,7 +208,7 @@ INSERT INTO env_builds (
 	cluster_node_id, version, created_at, updated_at
 ) VALUES ($1, $2, $3, $4, $5, $6, $7, $8, $9, $10, $11, $12, $13, CURRENT_TIMESTAMP)
 `, build.id, "FROM e2bdev/base:latest", dbtypes.BuildStatusUploaded,
-				2, 512, 512, 1982, "vmlinux-6.1.102", "v1.12.1_a41d3fb", pkg.Version,
+				2, 512, 512, 1982, "vmlinux-6.1.102", "v1.12.1_210cbac", pkg.Version,
 				"integration-test-node", templates.TemplateV1Version, build.createdAt)
 		} else {
 			err = db.TestsRawSQL(ctx, `
@@ -218,7 +218,7 @@ INSERT INTO env_builds (
 	cluster_node_id, version, updated_at
 ) VALUES ($1, $2, $3, $4, $5, $6, $7, $8, $9, $10, $11, $12, CURRENT_TIMESTAMP)
 `, build.id, "FROM e2bdev/base:latest", dbtypes.BuildStatusUploaded,
-				2, 512, 512, 1982, "vmlinux-6.1.102", "v1.12.1_a41d3fb", pkg.Version,
+				2, 512, 512, 1982, "vmlinux-6.1.102", "v1.12.1_210cbac", pkg.Version,
 				"integration-test-node", templates.TemplateV1Version)
 		}
 		if err != nil {
