Skip to content

Commit 9160b9f

Browse files
committed
save: Remove platform from config descriptor
This was brought up by bmitch that its not expected to have a platform object in the config descriptor. Also checked with tianon who agreed, its not _wrong_ but is unexpected and doesn't neccessarily make sense to have it there. Also, while technically incorrect, ECR is throwing an error when it sees this. Signed-off-by: Brian Goff <[email protected]>
1 parent 8599f2a commit 9160b9f

1 file changed

Lines changed: 0 additions & 3 deletions

File tree

image/tarexport/save.go

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -223,8 +223,6 @@ func (s *saveSession) save(outStream io.Writer) error {
223223
})
224224
}
225225

226-
imgPlat := imageDescr.image.Platform()
227-
228226
m := ocispec.Manifest{
229227
Versioned: specs.Versioned{
230228
SchemaVersion: 2,
@@ -234,7 +232,6 @@ func (s *saveSession) save(outStream io.Writer) error {
234232
MediaType: ocispec.MediaTypeImageConfig,
235233
Digest: digest.Digest(imageDescr.image.ID()),
236234
Size: int64(len(imageDescr.image.RawJSON())),
237-
Platform: &imgPlat,
238235
},
239236
Layers: foreign,
240237
}

0 commit comments

Comments
 (0)