avoid double MRF heal when there is versions disparity (#18585)

cmd/erasure-multipart.go

@@ -1290,13 +1290,15 @@ func (er erasureObjects) CompleteMultipartUpload(ctx context.Context, bucket str
 		})
 	}
 
-	// Check if there is any offline disk and add it to the MRF list
+	if !opts.Speedtest && !versionsDisparity {
-	for _, disk := range onlineDisks {
+		// Check if there is any offline disk and add it to the MRF list
-		if disk != nil && disk.IsOnline() {
+		for _, disk := range onlineDisks {
-			continue
+			if disk != nil && disk.IsOnline() {
+				continue
+			}
+			er.addPartial(bucket, object, fi.VersionID)
+			break
 		}
-		er.addPartial(bucket, object, fi.VersionID)
-		break
 	}
 
 	for i := 0; i < len(onlineDisks); i++ {

cmd/erasure-object.go

@@ -1531,18 +1531,21 @@ func (er erasureObjects) putObject(ctx context.Context, bucket string, object st
 
 	// For speedtest objects do not attempt to heal them.
 	if !opts.Speedtest {
-		// Whether a disk was initially or becomes offline
+		// When there is versions disparity we are healing
-		// during this upload, send it to the MRF list.
+		// the content implicitly for all versions, we can
-		for i := 0; i < len(onlineDisks); i++ {
+		// avoid triggering another MRF heal for offline drives.
-			if onlineDisks[i] != nil && onlineDisks[i].IsOnline() {
+		if !versionsDisparity {
-				continue
+			// Whether a disk was initially or becomes offline
+			// during this upload, send it to the MRF list.
+			for i := 0; i < len(onlineDisks); i++ {
+				if onlineDisks[i] != nil && onlineDisks[i].IsOnline() {
+					continue
+				}
+
+				er.addPartial(bucket, object, fi.VersionID)
+				break
 			}
-
+		} else {
-			er.addPartial(bucket, object, fi.VersionID)
-			break
-		}
-
-		if versionsDisparity {
 			globalMRFState.addPartialOp(partialOperation{
 				bucket:      bucket,
 				object:      object,