Merge pull request #354 from coryb/os-required

os/arch now required by buildkit on image spec
openllb · May 17, 2024 · 748fac9 · 748fac9
2 parents 352c220 + 5ccf4d7
commit 748fac9
Showing 1 changed file with 7 additions and 0 deletions.
diff --git a/codegen/builtin_fs.go b/codegen/builtin_fs.go
@@ -761,6 +761,13 @@ func (dp DockerPush) Call(ctx context.Context, cln *client.Client, val Value, op
 	}
 	exportFS.Image.ContainerConfig.Labels = exportFS.Image.Config.Labels
 
+	if exportFS.Image.OS == "" {
+		exportFS.Image.OS = exportFS.Platform.OS
+	}
+	if exportFS.Image.Architecture == "" {
+		exportFS.Image.Architecture = exportFS.Platform.Architecture
+	}
+
 	var dgst string
 	exportFS.SolveOpts = append(exportFS.SolveOpts,
 		solver.WithImageSpec(exportFS.Image),