Mark batch job status properly (#5066)

Client side overall status was always shown as `in-progress` even if job was successful or failed. Signed-off-by: Shubhendu Ram Tripathi <[email protected]>
minio · Oct 21, 2024 · a814b5d · a814b5d
1 parent 65aa514
commit a814b5d
Showing 1 changed file with 11 additions and 2 deletions.
diff --git a/cmd/batch-status.go b/cmd/batch-status.go
@@ -128,10 +128,19 @@ func mainBatchStatus(ctx *cli.Context) error {
 						return
 					}
 
-					printMsg(batchJobStatusMessage{
+					m := batchJobStatusMessage{
 						Status: "in-progress",
 						Metric: job,
-					})
+					}
+					switch {
+					case job.Complete:
+						m.Status = "complete"
+					case job.Failed:
+						m.Status = "failed"
+					default:
+						// leave as is with in-progress
+					}
+					printMsg(m)
 					if job.Complete || job.Failed {
 						cancel()
 						return