fix: use getTask instead of taskStore.Get, for the error cause type (#571)

Signed-off-by: zzy987 <67889264+zzy987@users.noreply.github.com>
This commit is contained in:
zzy987 2021-08-24 15:18:55 +08:00 committed by Gaius
parent 22ad61b7f6
commit b11344aa27
No known key found for this signature in database
GPG Key ID: 8B4E5D1290FA2FFB
2 changed files with 2 additions and 5 deletions

View File

@ -100,10 +100,7 @@ func (cleaner *Cleaner) GC(storagePattern string, force bool) ([]string, error)
walkTaskIds[taskID] = true
// we should return directly when we success to get info which means it is being used
if _, err := cleaner.taskMgr.Get(taskID); err == nil || !cdnerrors.IsDataNotFound(err) {
if err != nil {
logger.GcLogger.With("type", storagePattern).Errorf("failed to get taskID(%s): %v", taskID, err)
}
if cleaner.taskMgr.Exist(taskID) {
return nil
}

View File

@ -182,7 +182,7 @@ func (tm Manager) Get(taskID string) (*types.SeedTask, error) {
}
func (tm Manager) Exist(taskID string) bool {
_, err := tm.taskStore.Get(taskID)
_, err := tm.getTask(taskID)
return err == nil || !cdnerrors.IsDataNotFound(err)
}