csi: Remove NOT_FOUND error from ControllerUnpublishVolume() implemen…

…tation

Now the provisioner won't wait for Detach
(ControllerUnpublishVolume()) complete before invoking DeleteVolume.
Then Detach will get stuck in the NOT_FOUND error since there is no
way to verify the volume before detaching.

For the details, See
container-storage-interface/spec#382

And the spec doc has removed NOT_FOUND error for
ControllerUnpublishVolume():
https://github.com/container-storage-interface/spec/blob/release-1.2/spec.md#controllerunpublishvolume

Signed-off-by: Shuo Wu <shuo@rancher.com>

csi/controller_server.go

@@ -225,11 +225,15 @@ func (cs *ControllerServer) ControllerUnpublishVolume(ctx context.Context, req *
 	if err != nil {
 		return nil, status.Error(codes.Internal, err.Error())
 	}
+
+	// VOLUME_NOT_FOUND is no longer the ControllerUnpublishVolume error
+	// See https://github.com/container-storage-interface/spec/issues/382 for details
 	if existVol == nil {
 		msg := fmt.Sprintf("ControllerUnpublishVolume: the volume %s not exists", req.GetVolumeId())
 		logrus.Warn(msg)
-		return nil, status.Error(codes.NotFound, msg)
+		return &csi.ControllerUnpublishVolumeResponse{}, nil
 	}
+
 	if existVol.State == string(types.VolumeStateDetaching) {
 		return nil, status.Errorf(codes.Aborted, "The volume %s is detaching", req.GetVolumeId())
 	}