Skip to content

Commit

Permalink
os/arch now required by buildkit on image spec
Browse files Browse the repository at this point in the history
Related buildkit change:
moby/buildkit#4601
  • Loading branch information
coryb committed May 17, 2024
1 parent 352c220 commit 5ccf4d7
Showing 1 changed file with 7 additions and 0 deletions.
7 changes: 7 additions & 0 deletions codegen/builtin_fs.go
Original file line number Diff line number Diff line change
Expand Up @@ -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),
Expand Down

0 comments on commit 5ccf4d7

Please sign in to comment.