Merge pull request #65 from donpenney/fix-job-handling

Fix hwmgr job handling
openshift-kni · Dec 23, 2024 · 7865e5a · 7865e5a
2 parents 14e9a79 + d1ff852
commit 7865e5a
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/adaptors/dell-hwmgr/hwmgrclient/client.go b/adaptors/dell-hwmgr/hwmgrclient/client.go
@@ -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