Skip to content

Commit

Permalink
Merge pull request #883 from nalind/manifest-list-reserialize
Browse files Browse the repository at this point in the history
copy: avoid unnecessary re-encoding of source lists
  • Loading branch information
rhatdan authored Apr 7, 2020
2 parents 075eb89 + 5a69db8 commit 29b4cbb
Showing 1 changed file with 3 additions and 0 deletions.
3 changes: 3 additions & 0 deletions copy/copy.go
Original file line number Diff line number Diff line change
Expand Up @@ -484,6 +484,9 @@ func (c *copier) copyMultipleImages(ctx context.Context, policyContext *signatur
return nil, "", errors.Errorf("Error: manifest list must be converted to type %q to be written to destination, but that would invalidate signatures", thisListType)
}
logrus.Debugf("Manifest list has been updated")
} else {
// We can just use the original value, so use it instead of the one we just rebuilt, so that we don't change the digest.
attemptedManifestList = manifestList
}

// Save the manifest list.
Expand Down

0 comments on commit 29b4cbb

Please sign in to comment.