Alternate implementation of GetLock

Author: glpatcern

pkg/storage/fs/cephfs/cephfs.go

@@ -700,14 +700,25 @@ func (fs *cephfs) GetLock(ctx context.Context, ref *provider.Reference) (*provid
 		return nil, getRevaError(err)
 	}
 
+	// prepare the metadata
 	var l *provider.Lock
+	buf, errXattr := cv.mount.GetXattr(path, xattrLock)
+	if errXattr == nil {
+		if l, err = decodeLock(string(buf)); err != nil {
+			return nil, errors.Wrap(err, "cephfs: malformed lock payload")
+		}
+	}
+
 	user.op(func(cv *cacheVal) {
 		var file *goceph.File
 		defer closeFile(file)
 		if file, err = cv.mount.Open(path, os.O_RDWR, fs.conf.FilePerms); err != nil {
-			// TODO(lopresti) if user has read-only permissions, here we fail because
-			// we want to try and grab a lock to probe if a lock existed. Alternatively,
-			// we could just return the metadata if present.
+			// try and open with read-only permissions: if this succeeds, we just return
+			// the metadata as is, otherwise we return the error on Open()
+			_, err = cv.mount.Open(path, os.O_RDONLY, fs.conf.FilePerms)
+			if err != nil {
+				l = nil
+			}
 			return
 		}
 
@@ -718,21 +729,14 @@ func (fs *cephfs) GetLock(ctx context.Context, ref *provider.Reference) (*provid
 			return
 		}
 
-		buf, err := cv.mount.GetXattr(path, xattrLock)
-		if err != nil {
+		if errXattr != nil {
 			// error here means we have a "foreign" flock with no CS3 metadata
 			err = nil
 			l = new(provider.Lock)
 			l.AppName = "External"
 			return
 		}
 
-		if l, err = decodeLock(string(buf)); err != nil {
-			l = nil
-			err = errors.Wrap(err, "cephfs: malformed lock payload")
-			return
-		}
-
 		if time.Unix(int64(l.Expiration.Seconds), 0).After(time.Now()) {
 			// the lock expired, drop
 			fs.UnsetArbitraryMetadata(ctx, ref, []string{xattrLock})