TrueCloudLab/distribution
13
0
Fork 3
Code Issues 3 Pull requests Projects Releases 2 Packages Wiki Activity Actions

Remove err nil check

Browse source 
since type checking nil will not panic and return appropriately

Signed-off-by: Manish Tomar <manish.tomar@docker.com>
This commit is contained in:
Manish Tomar 2019-02-04 16:42:44 -08:00
parent da8db4666b
commit 48818fdea7
1 changed files with 7 additions and 9 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

16
registry/storage/garbagecollect.go
View file

@ -98,15 +98,13 @@ func MarkAndSweep(ctx context.Context, storageDriver driver.StorageDriver, regis
return nil return nil
}) })
if err != nil { // In certain situations such as unfinished uploads, deleting all
// In certain situations such as unfinished uploads, deleting all // tags in S3 or removing the _manifests folder manually, this
// tags in S3 or removing the _manifests folder manually, this // error may be of type PathNotFound.
// error may be of type PathNotFound. //
// // In these cases we can continue marking other manifests safely.
// In these cases we can continue marking other manifests safely. if _, ok := err.(driver.PathNotFoundError); ok {
if _, ok := err.(driver.PathNotFoundError); ok { return nil
return nil
}
} }
return err return err