Project

General

Profile

Actions

Feature #10872

closed

backport POSIX lock support for ceph-fuse client to firefly

Added by Martin Schuppert about 9 years ago. Updated about 9 years ago.

Status:
Resolved
Priority:
Normal
Assignee:
Category:
-
Target version:
-
% Done:

0%

Source:
Development
Tags:
Backport:
Reviewed:
Affected Versions:
Component(FS):
Labels (FS):
Pull request ID:

Description

Description of problem:
In Firefly, the userspace version of the cephfs client does not support POSIX file locking.

The lack of POSIX lock creates various issues, for example, when VM ephemeral disk is created by Nova on a cephfs shared folder, it is possible that many nova-compute services will try to write into the same _base file due to the lock not being implemented.

This request is to check the possibility top backport POSIX locks which have been introduced with 0.88 - http://ceph.com/releases/v0-88-released/

~~
This is the first development release after Giant. The two main features merged this round are the new AsyncMessenger (an alternative implementation of the network layer) from Haomai Wang at UnitedStack, and support for POSIX file locks in ceph-fuse and libcephfs from Yan, Zheng. There is also a big pile of smaller items that re merged while we were stabilizing Giant, including a range of smaller performance and bug fixes and some new tracepoints for LTTNG. - See more at: http://ceph.com/releases/v0-88-released/#sthash.O8ce6HyX.dpuf
~~

Actions #1

Updated by Greg Farnum about 9 years ago

  • Status changed from New to In Progress
  • Assignee set to Greg Farnum

Much to my surprise, a straightforward cherry-pick of the ceph-fuse flock branch to firefly does build after some merge issues. I didn't see anything that would definitely prevent it from working, but we'll need to look through it a lot more carefully. There's an RFC PR here: https://github.com/ceph/ceph/pull/3730

Actions #2

Updated by Greg Farnum about 9 years ago

  • Status changed from In Progress to 4
Actions #3

Updated by Greg Farnum about 9 years ago

  • Status changed from 4 to 7
Actions #4

Updated by Greg Farnum about 9 years ago

  • Status changed from 7 to Pending Backport
  • Assignee changed from Greg Farnum to Loïc Dachary

It's passed a firefly suite run and been through review.

Actions #5

Updated by Greg Farnum about 9 years ago

  • Status changed from Pending Backport to Resolved
  • Assignee changed from Loïc Dachary to Greg Farnum

We're doing another quick point release and Sage told me I could just do this, so I have. Merged to firefly branch as of commit:0820041e6515c82c41b81b9e6825e2dd5fcb8165

Actions

Also available in: Atom PDF