mirrors/ceph-csi
1
0
Fork 0
mirror of https://github.com/ceph/ceph-csi.git synced 2025-05-29 18:16:42 +00:00
Code Issues Packages Projects Releases Wiki Activity

rbd: use os.Remove to remove directory

Browse Source 
using os.RemoveAll will remove everything
in the director after the Umount we should
be using os.Remove only to remove the empty
directory

Signed-off-by: Madhu Rajanna <madhupr007@gmail.com>
(cherry picked from commit 00d252e4ac0ed9ddb8e7cc9f6e8ffbf97b430080)
This commit is contained in:
Madhu Rajanna 2024-11-20 10:15:40 +01:00 committed by mergify[bot]
parent b939471ac2
commit 590955ea76
1 changed files with 2 additions and 2 deletions

4
internal/rbd/nodeserver.go

@ -938,7 +938,7 @@ func (ns *NodeServer) NodeUnpublishVolume(
return nil, status.Error(codes.NotFound, err.Error()) return nil, status.Error(codes.NotFound, err.Error())
} }
if !isMnt { if !isMnt {
if err = os.RemoveAll(targetPath); err != nil { if err = os.Remove(targetPath); err != nil {
return nil, status.Error(codes.Internal, err.Error()) return nil, status.Error(codes.Internal, err.Error())
} }
@ -949,7 +949,7 @@ func (ns *NodeServer) NodeUnpublishVolume(
return nil, status.Error(codes.Internal, err.Error()) return nil, status.Error(codes.Internal, err.Error())
} }
if err = os.RemoveAll(targetPath); err != nil { if err = os.Remove(targetPath); err != nil {
return nil, status.Error(codes.Internal, err.Error()) return nil, status.Error(codes.Internal, err.Error())
} }