Skip to content

Commit

Permalink
Merge pull request #1880 from mtrmac/encrypt-comments
Browse files Browse the repository at this point in the history
Remove some inaccurate comments
  • Loading branch information
vrothberg committed Mar 13, 2023
2 parents d12f543 + 2cac6eb commit 3d74c68
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 3 deletions.
2 changes: 0 additions & 2 deletions copy/copy.go
Original file line number Diff line number Diff line change
Expand Up @@ -101,8 +101,6 @@ type Options struct {

// If OciEncryptConfig is non-nil, it indicates that an image should be encrypted.
// The encryption options is derived from the construction of EncryptConfig object.
// Note: During initial encryption process of a layer, the resultant digest is not known
// during creation, so newDigestingReader has to be set with validateDigest = false
OciEncryptConfig *encconfig.EncryptConfig
// OciEncryptLayers represents the list of layers to encrypt.
// If nil, don't encrypt any layers.
Expand Down
1 change: 0 additions & 1 deletion copy/single.go
Original file line number Diff line number Diff line change
Expand Up @@ -364,7 +364,6 @@ func (ic *imageCopier) copyLayers(ctx context.Context) error {
return err
}
srcInfosUpdated := false
// If we only need to check authorization, no updates required.
if updatedSrcInfos != nil && !reflect.DeepEqual(srcInfos, updatedSrcInfos) {
if ic.cannotModifyManifestReason != "" {
return fmt.Errorf("Copying this image would require changing layer representation, which we cannot do: %q", ic.cannotModifyManifestReason)
Expand Down

0 comments on commit 3d74c68

Please sign in to comment.