use isNotMountPoint instead of isNotLikelyMountPoint to keep idempotent behaviour of the driver because later method can't detect bind mounts.

mucahitkurt · mucahitkurt · commit 692c215f6e6a · 2019-10-01T21:48:10.000+03:00
Signed-off-by: Mucahit Kurt &lt;mucahitkurt@gmail.com&gt;
diff --git a/pkg/hostpath/nodeserver.go b/pkg/hostpath/nodeserver.go
@@ -133,7 +133,7 @@ func (ns *nodeServer) NodePublishVolume(ctx context.Context, req *csi.NodePublis
 			return nil, status.Error(codes.InvalidArgument, "cannot publish a non-mount volume as mount volume")
 		}
 
-		notMnt, err := mount.New("").IsLikelyNotMountPoint(targetPath)
+		notMnt, err := mount.New("").IsNotMountPoint(targetPath)
 		if err != nil {
 			if os.IsNotExist(err) {
 				if err = os.MkdirAll(targetPath, 0750); err != nil {

Original file line number	Diff line number	Diff line change
`@@ -133,7 +133,7 @@ func (ns nodeServer) NodePublishVolume(ctx context.Context, req csi.NodePublis`
`133`	`133`	`return nil, status.Error(codes.InvalidArgument, "cannot publish a non-mount volume as mount volume")`
`134`	`134`	`}`
`135`	`135`
`136`		`- notMnt, err := mount.New("").IsLikelyNotMountPoint(targetPath)`
	`136`	`+ notMnt, err := mount.New("").IsNotMountPoint(targetPath)`
`137`	`137`	`if err != nil {`
`138`	`138`	`if os.IsNotExist(err) {`
`139`	`139`	`if err = os.MkdirAll(targetPath, 0750); err != nil {`