Skip to content

Commit

Permalink
Fix hwmgr job handling
Browse files Browse the repository at this point in the history
Signed-off-by: Don Penney <[email protected]>
  • Loading branch information
donpenney committed Dec 23, 2024
1 parent 14e9a79 commit d1ff852
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions adaptors/dell-hwmgr/hwmgrclient/client.go
Original file line number Diff line number Diff line change
Expand Up @@ -321,15 +321,15 @@ func (c *HardwareManagerClient) CheckJobStatus(ctx context.Context, jobId string
failReason = "unknown"
}
c.Logger.InfoContext(ctx, "Job has failed", slog.Any("status", status), slog.String("failReason", failReason))
return JobStatusFailed, failReason, fmt.Errorf("job has failed: %s", failReason)
return JobStatusFailed, failReason, nil
default:
if status.Brief.FailReason != nil {
failReason = *status.Brief.FailReason
} else {
failReason = "unknown"
}
c.Logger.InfoContext(ctx, "Job status is unknown", slog.Any("status", status), slog.String("failReason", failReason))
return JobStatusUnknown, failReason, fmt.Errorf("job status is unknown: %s", failReason)
return JobStatusUnknown, failReason, nil
}

return JobStatusCompleted, failReason, nil
Expand Down

0 comments on commit d1ff852

Please sign in to comment.