Merge pull request #1908 from leiyiz/release-1.14.2-exp

experimental fix, remove deviceInUse check

Author: pwschuurman

pkg/gce-pd-csi-driver/node.go

@@ -19,7 +19,6 @@ import (
 	"errors"
 	"fmt"
 	"os"
-	"path/filepath"
 	"regexp"
 	"runtime"
 	"strconv"
@@ -468,21 +467,24 @@ func (ns *GCENodeServer) NodeUnstageVolume(ctx context.Context, req *csi.NodeUns
 type ignoreableError struct{ error }
 
 func (ns *GCENodeServer) confirmDeviceUnused(volumeID string) error {
-	devicePath, err := getDevicePath(ns, volumeID, "" /* partition, which is unused */)
-	if err != nil {
-		return &ignoreableError{fmt.Errorf("failed to find device path for volume %s: %v", volumeID, err.Error())}
-	}
-
-	devFsPath, err := filepath.EvalSymlinks(devicePath)
-	if err != nil {
-		return &ignoreableError{fmt.Errorf("filepath.EvalSymlinks(%q) failed: %v", devicePath, err)}
-	}
+	// 1.14-exp release. DO NOT INCLUDE in regular release.
+	//
 
-	if inUse, err := ns.DeviceUtils.IsDeviceFilesystemInUse(ns.Mounter, devicePath, devFsPath); err != nil {
-		return &ignoreableError{fmt.Errorf("failed to check if device %s (aka %s) is in use: %v", devicePath, devFsPath, err)}
-	} else if inUse {
-		return fmt.Errorf("device %s (aka %s) is still in use", devicePath, devFsPath)
-	}
+	// devicePath, err := getDevicePath(ns, volumeID, "" /* partition, which is unused */)
+	// if err != nil {
+	// 	return &ignoreableError{fmt.Errorf("failed to find device path for volume %s: %v", volumeID, err.Error())}
+	// }
+
+	// devFsPath, err := filepath.EvalSymlinks(devicePath)
+	// if err != nil {
+	// 	return &ignoreableError{fmt.Errorf("filepath.EvalSymlinks(%q) failed: %v", devicePath, err)}
+	// }
+
+	// if inUse, err := ns.DeviceUtils.IsDeviceFilesystemInUse(ns.Mounter, devicePath, devFsPath); err != nil {
+	// 	return &ignoreableError{fmt.Errorf("failed to check if device %s (aka %s) is in use: %v", devicePath, devFsPath, err)}
+	// } else if inUse {
+	// 	return fmt.Errorf("device %s (aka %s) is still in use", devicePath, devFsPath)
+	// }
 
 	return nil
 }