Continue to unpublish if volume is already unmounted

Closes trident#572
NetApp · Oct 19, 2021 · 10efaf2 · 10efaf2
1 parent 89c5b93
commit 10efaf2
Showing 1 changed file with 6 additions and 6 deletions.
diff --git a/frontend/csi/node_server.go b/frontend/csi/node_server.go
@@ -185,12 +185,12 @@ func (p *Plugin) NodeUnpublishVolume(
 	}
 
 	if notMountPoint {
-		return nil, status.Error(codes.NotFound, "volume not mounted")
-	}
-
-	if err = utils.WaitForUmount(ctx, targetPath, umountMaxDuration); err != nil {
-		Logc(ctx).WithFields(log.Fields{"path": targetPath, "error": err}).Error("unable to unmount volume.")
-		return nil, status.Errorf(codes.InvalidArgument, "unable to unmount volume; %s", err)
+		Logc(ctx).Debug( "Volume not mounted, proceeding to unpublish volume")
+	} else {
+		if err = utils.WaitForUmount(ctx, targetPath, umountMaxDuration); err != nil {
+			Logc(ctx).WithFields(log.Fields{"path": targetPath, "error": err}).Error("unable to unmount volume.")
+			return nil, status.Errorf(codes.InvalidArgument, "unable to unmount volume; %s", err)
+		}
 	}
 
 	// As per the CSI spec SP i.e. Trident is responsible for deleting the target path,