Modify freeze functionality for updated longhorn-engine logic

Lognhorn 2187

Signed-off-by: Eric Weber <[email protected]>

app/cmd/start.go

@@ -168,17 +168,22 @@ func unfreezeFilesystems() error {
 	// - If it does, we just need to know where in the container we can access the mount points to unfreeze the file
 	//   system.
 	mounter := mount.New("")
-	sourcePoints, err := mounter.List()
+	mountPoints, err := mounter.List()
 	if err != nil {
 		return errors.Wrap(err, "failed to list mount points while starting up")
 	}
 
-	for _, sourcePoint := range sourcePoints {
-		if strings.Contains(sourcePoint.Device, "/dev/longhorn") { // TODO: Find this as a const somewhere.
+	for _, mountPoint := range mountPoints {
+		if strings.Contains(mountPoint.Device, engineutil.DevicePathPrefix) {
 			// We do not actually expect any filesystems to be frozen. This is a best effort attempt to unfreeze them
-			// if somehow instance manager crashed at the wrong moment during a snapshot. Log on success, but not on
-			// failure.
-			engineutil.AttemptUnfreezeFilesystem(sourcePoint.Path, nil, false, logrus.New())
+			// if somehow instance manager crashed at the wrong moment during a snapshot.
+			unfroze, err := engineutil.UnfreezeFilesystem(mountPoint.Path, nil)
+			if err != nil {
+				logrus.WithError(err).Warnf("Failed to unfreeze filesystem mounted at %v", mountPoint)
+			}
+			if unfroze {
+				logrus.Warnf("Unfroze filesystem mounted at %v", mountPoint)
+			}
 		}
 	}
 	return nil