Skip to content
This repository has been archived by the owner on Jun 18, 2022. It is now read-only.

Commit

Permalink
Merge pull request #132 from StrongMonkey/fix-2
Browse files Browse the repository at this point in the history
small fix
  • Loading branch information
ibuildthecloud authored Jan 19, 2017
2 parents 2c1bfd1 + 46999e4 commit e6ff976
Showing 1 changed file with 1 addition and 2 deletions.
3 changes: 1 addition & 2 deletions core/storage/storage.go
Original file line number Diff line number Diff line change
Expand Up @@ -109,7 +109,7 @@ func DoVolumeRemove(volume model.Volume, storagePool model.StoragePool, progress
}
errorList := []error{}
for i := 0; i < 3; i++ {
if err := utils.RemoveContainer(dockerClient, container.ID); !engineCli.IsErrContainerNotFound(err) {
if err := utils.RemoveContainer(dockerClient, container.ID); err != nil && !engineCli.IsErrContainerNotFound(err) {
errorList = append(errorList, err)
} else {
break
Expand Down Expand Up @@ -145,7 +145,6 @@ func DoVolumeRemove(volume model.Volume, storagePool model.StoragePool, progress
return errors.Wrap(err, constants.DoVolumeRemoveError+"failed to remove directory")
}
}
return errors.Wrap(existErr, constants.DoVolumeRemoveError+"failed to find the path")
}
return nil
}

0 comments on commit e6ff976

Please sign in to comment.