mirrors/ceph-csi
1
0
Fork 0
mirror of https://github.com/ceph/ceph-csi.git synced 2024-10-18 13:19:57 +00:00
Code Issues Packages Projects Releases Wiki Activity

cephfs: correct error code for volumegroupsnapshot

Browse Source 
this commit corrects the error code used by
groupvolumelocks to acquire snapshots

Signed-off-by: yati1998 <ypadia@redhat.com>
This commit is contained in:
yati1998 2024-09-24 17:25:13 +05:30 committed by mergify[bot]
parent 29aecd345f
commit dbd8462bcc
1 changed files with 4 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
internal/cephfs/groupcontrollerserver.go
View File

@ -88,9 +88,9 @@ func (cs *ControllerServer) CreateVolumeGroupSnapshot(
requestName := req.GetName() requestName := req.GetName()
// Existence and conflict checks // Existence and conflict checks
if acquired := cs.VolumeGroupLocks.TryAcquire(requestName); !acquired { if acquired := cs.VolumeGroupLocks.TryAcquire(requestName); !acquired {
log.ErrorLog(ctx, util.VolumeOperationAlreadyExistsFmt, requestName) log.ErrorLog(ctx, util.SnapshotOperationAlreadyExistsFmt, requestName)
return nil, status.Errorf(codes.Aborted, util.VolumeOperationAlreadyExistsFmt, requestName) return nil, status.Errorf(codes.Aborted, util.SnapshotOperationAlreadyExistsFmt, requestName)
} }
defer cs.VolumeGroupLocks.Release(requestName) defer cs.VolumeGroupLocks.Release(requestName)
@ -708,9 +708,9 @@ func (cs *ControllerServer) DeleteVolumeGroupSnapshot(ctx context.Context,
groupSnapshotID := req.GetGroupSnapshotId() groupSnapshotID := req.GetGroupSnapshotId()
// Existence and conflict checks // Existence and conflict checks
if acquired := cs.VolumeGroupLocks.TryAcquire(groupSnapshotID); !acquired { if acquired := cs.VolumeGroupLocks.TryAcquire(groupSnapshotID); !acquired {
log.ErrorLog(ctx, util.VolumeOperationAlreadyExistsFmt, groupSnapshotID) log.ErrorLog(ctx, util.SnapshotOperationAlreadyExistsFmt, groupSnapshotID)
return nil, status.Errorf(codes.Aborted, util.VolumeOperationAlreadyExistsFmt, groupSnapshotID) return nil, status.Errorf(codes.Aborted, util.SnapshotOperationAlreadyExistsFmt, groupSnapshotID)
} }
defer cs.VolumeGroupLocks.Release(groupSnapshotID) defer cs.VolumeGroupLocks.Release(groupSnapshotID)