Merge pull request #9 from vngcloud/fixbug-mount-root-node

tvt286 · web-flow · commit c0cfecb92569 · 2025-09-04T21:43:23.000+07:00
[fix] improve error handling in getDevicePath by providing detailed e…
diff --git a/pkg/driver/node.go b/pkg/driver/node.go
@@ -758,10 +758,9 @@ func (s *nodeService) isMounted(_ string, target string) (bool, error) {
 }
 
 func (s *nodeService) getDevicePath(volumeID string) (string, error) {
-	var devicePath string
 	devicePath, err := s.mounter.GetDevicePathBySerialID(volumeID)
 	if err != nil {
-		klog.Warningf("Couldn't get device path from mount: %v", err)
+		return "", fmt.Errorf("failed to get device path for volume %s: %w", volumeID, err)
 	}
 
 	return devicePath, nil

Original file line number	Diff line number	Diff line change
`@@ -758,10 +758,9 @@ func (s *nodeService) isMounted(_ string, target string) (bool, error) {`
`758`	`758`	`}`
`759`	`759`
`760`	`760`	`func (s *nodeService) getDevicePath(volumeID string) (string, error) {`
`761`		`- var devicePath string`
`762`	`761`	`devicePath, err := s.mounter.GetDevicePathBySerialID(volumeID)`
`763`	`762`	`if err != nil {`
`764`		`- klog.Warningf("Couldn't get device path from mount: %v", err)`
	`763`	`+ return "", fmt.Errorf("failed to get device path for volume %s: %w", volumeID, err)`
`765`	`764`	`}`
`766`	`765`
`767`	`766`	`return devicePath, nil`