Return error when NodePublishVolume mount fails

c3y1huang · c3y1huang · commit 4b49fd597337 · 2020-02-11T15:24:11.000+08:00
Signed-off-by: Chin-Ya Huang &lt;chin-ya.huang@suse.com&gt;
diff --git a/pkg/hostpath/nodeserver.go b/pkg/hostpath/nodeserver.go
@@ -181,6 +181,7 @@ func (ns *nodeServer) NodePublishVolume(ctx context.Context, req *csi.NodePublis
 					errList.WriteString(fmt.Sprintf(" :%s", rmErr.Error()))
 				}
 			}
+			return nil, status.Error(codes.Internal, fmt.Sprintf("failed to mount device: %s at %s: %s", path, targetPath, errList.String()))
 		}
 	}
 

Original file line number	Diff line number	Diff line change
`@@ -181,6 +181,7 @@ func (ns nodeServer) NodePublishVolume(ctx context.Context, req csi.NodePublis`
`181`	`181`	`errList.WriteString(fmt.Sprintf(" :%s", rmErr.Error()))`
`182`	`182`	`}`
`183`	`183`	`}`
	`184`	`+ return nil, status.Error(codes.Internal, fmt.Sprintf("failed to mount device: %s at %s: %s", path, targetPath, errList.String()))`
`184`	`185`	`}`
`185`	`186`	`}`
`186`	`187`