Merge 25d25b45eef552dbee5cde1730b5320a5407dfc6 into c2e23d3301b1be2b2ad667184030087f92ad2470

This commit is contained in:
wxiaoguang 2025-02-19 09:50:31 +01:00 committed by GitHub
commit 0d9f0759ab
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194

View File

@ -356,10 +356,6 @@ func createPackageAndVersion(ctx context.Context, mci *manifestCreationInfo, met
return nil, err return nil, err
} }
if err = packages_service.DeletePackageVersionAndReferences(ctx, pv); err != nil {
return nil, err
}
// keep download count on overwrite // keep download count on overwrite
_pv.DownloadCount = pv.DownloadCount _pv.DownloadCount = pv.DownloadCount
@ -418,12 +414,10 @@ func createFileFromBlobReference(ctx context.Context, pv, uploadVersion *package
} }
var err error var err error
if pf, err = packages_model.TryInsertFile(ctx, pf); err != nil { if pf, err = packages_model.TryInsertFile(ctx, pf); err != nil {
if errors.Is(err, packages_model.ErrDuplicatePackageFile) { if !errors.Is(err, packages_model.ErrDuplicatePackageFile) {
// Skip this blob because the manifest contains the same filesystem layer multiple times. log.Error("Error inserting package file: %v", err)
return nil return err
} }
log.Error("Error inserting package file: %v", err)
return err
} }
props := map[string]string{ props := map[string]string{
@ -437,13 +431,6 @@ func createFileFromBlobReference(ctx context.Context, pv, uploadVersion *package
} }
} }
// Remove the file from the blob upload version
if uploadVersion != nil && ref.File.File != nil && uploadVersion.ID == ref.File.File.VersionID {
if err := packages_service.DeletePackageFile(ctx, ref.File.File); err != nil {
return err
}
}
return nil return nil
} }