diff --git a/internal/cephfs/controllerserver.go b/internal/cephfs/controllerserver.go index 4a4277c3e..25c298bcf 100644 --- a/internal/cephfs/controllerserver.go +++ b/internal/cephfs/controllerserver.go @@ -595,7 +595,7 @@ func (cs *ControllerServer) CreateSnapshot( info, err := parentVolOptions.getSubVolumeInfo(ctx, volumeID(vid.FsSubvolName)) if err != nil { // Check error code value against ErrInvalidCommand to understand the cluster - // support it or not, its safe to evaluat as the filtering + // support it or not, It's safe to evaluate as the filtering // is already done from getSubVolumeInfo() and send out the error here. if errors.Is(err, ErrInvalidCommand) { return nil, status.Error( diff --git a/internal/cephfs/fsjournal.go b/internal/cephfs/fsjournal.go index 9ee2e0c44..31dbad94e 100644 --- a/internal/cephfs/fsjournal.go +++ b/internal/cephfs/fsjournal.go @@ -345,7 +345,7 @@ volume names as requested by the CSI drivers. Hence, these need to be invoked on respective CSI driver generated volume name based locks are held, as otherwise racy access to these omaps may end up leaving them in an inconsistent state. -These functions also cleanup omap reservations that are stale. I.e when omap entries exist and +These functions also cleanup omap reservations that are stale. I.e. when omap entries exist and backing subvolumes are missing, or one of the omaps exist and the next is missing. This is because, the order of omap creation and deletion are inverse of each other, and protected by the request name lock, and hence any stale omaps are leftovers from incomplete transactions and are diff --git a/internal/cephfs/nodeserver.go b/internal/cephfs/nodeserver.go index 067b9adb8..d321ed681 100644 --- a/internal/cephfs/nodeserver.go +++ b/internal/cephfs/nodeserver.go @@ -308,7 +308,7 @@ func (ns *NodeServer) NodeUnpublishVolume( return nil, status.Error(codes.Internal, err.Error()) } - util.DebugLog(ctx, "cephfs: successfully unbinded volume %s from %s", req.GetVolumeId(), targetPath) + util.DebugLog(ctx, "cephfs: successfully unbounded volume %s from %s", req.GetVolumeId(), targetPath) return &csi.NodeUnpublishVolumeResponse{}, nil }