Merge pull request #1138 from percona/PBM-1544-Add-new-column-showing-the-backup-status

PBM-1544 - Add new column showing the backup "Status" to pbm status output

Author: veceraj

cmd/pbm/status.go

@@ -470,22 +470,35 @@ func (s storageStat) String() string {
 		return a.RestoreTS > b.RestoreTS
 	})
 
+	const (
+		statusSuccess = "success"
+		statusFailed  = "failed"
+		statusOngoing = "ongoing"
+	)
+
+	var printStatus string
+
 	for i := range s.Snapshot {
 		ss := &s.Snapshot[i]
 		var status string
 		switch ss.Status {
 		case defs.StatusDone:
 			status = fmt.Sprintf("[restore_to_time: %s]", fmtTS(ss.RestoreTS))
+			printStatus = statusSuccess
 		case defs.StatusCancelled:
 			status = fmt.Sprintf("[!canceled: %s]", fmtTS(ss.RestoreTS))
+			printStatus = statusFailed
 		case defs.StatusError:
 			if errors.Is(ss.Err, errIncompatible) {
 				status = fmt.Sprintf("[incompatible: %s] [%s]", ss.Err.Error(), fmtTS(ss.RestoreTS))
+				printStatus = statusSuccess
 			} else {
 				status = fmt.Sprintf("[ERROR: %s] [%s]", ss.Err.Error(), fmtTS(ss.RestoreTS))
+				printStatus = statusFailed
 			}
 		default:
 			status = fmt.Sprintf("[running: %s / %s]", ss.Status, fmtTS(ss.RestoreTS))
+			printStatus = statusOngoing
 		}
 
 		t := string(ss.Type)
@@ -497,7 +510,7 @@ func (s storageStat) String() string {
 		if ss.StoreName != "" {
 			t += ", *"
 		}
-		ret += fmt.Sprintf("    %s %s <%s> %s\n", ss.Name, storage.PrettySize(ss.Size), t, status)
+		ret += fmt.Sprintf("    %s %s <%s> %s %s\n", ss.Name, storage.PrettySize(ss.Size), t, printStatus, status)
 	}
 
 	if len(s.PITR.Ranges) == 0 {