Skip to content

Commit 2a9c4da

Browse files
aledbfroboquat
authored andcommitted
[registry-facade] Do not log warning from local store already exist error
1 parent 557b970 commit 2a9c4da

File tree

1 file changed

+4
-2
lines changed

1 file changed

+4
-2
lines changed

components/registry-facade/pkg/registry/manifest.go

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -327,7 +327,7 @@ func DownloadConfig(ctx context.Context, fetch FetcherFunc, ref string, desc oci
327327

328328
return w.Commit(ctx, int64(len(buf)), digest.FromBytes(buf), content.WithLabels(contentTypeLabel(desc.MediaType)))
329329
}()
330-
if err != nil {
330+
if err != nil && !strings.Contains(err.Error(), "already exists") {
331331
log.WithError(err).WithField("ref", ref).WithField("desc", desc).Warn("cannot cache config")
332332
}
333333
}
@@ -501,7 +501,9 @@ func DownloadManifest(ctx context.Context, fetch FetcherFunc, desc ociv1.Descrip
501501
// time one wishes to resolve desc.
502502
w, err := opts.Store.Writer(ctx, content.WithDescriptor(desc), content.WithRef(desc.Digest.String()))
503503
if err != nil {
504-
log.WithError(err).WithField("desc", *rdesc).Warn("cannot create store writer")
504+
if err != nil && !strings.Contains(err.Error(), "already exists") {
505+
log.WithError(err).WithField("desc", *rdesc).Warn("cannot create store writer")
506+
}
505507
} else {
506508
_, err = io.Copy(w, bytes.NewReader(inpt))
507509
if err != nil {

0 commit comments

Comments
 (0)