[Bugs] [Bug 1332162] New: Support mandatory locking in glusterfs

Mon May 2 11:42:47 UTC 2016

https://bugzilla.redhat.com/show_bug.cgi?id=1332162

            Bug ID: 1332162
           Summary: Support mandatory locking in glusterfs
           Product: GlusterFS
           Version: 3.8.0
         Component: locks
          Keywords: FutureFeature, Reopened, Triaged
          Severity: medium
          Priority: medium
          Assignee: bugs at gluster.org
          Reporter: anoopcs at redhat.com
                CC: anoopcs at redhat.com, bugs at gluster.org,
                    pavan at gluster.com, rjoseph at redhat.com,
                    vijay at dev.gluster.com
        Depends On: 762184 (GLUSTER-452)
            Blocks: 1317278 (glusterfs-3.8.0)

+++ This bug was initially created as a clone of Bug #762184 +++

Need to get mandatory locking supported over glusterfs.

--- Additional comment from Vijay Bellur on 2010-08-28 14:45:51 IST ---

No immediate plans to address this.

--- Additional comment from Anand Avati on 2015-02-27 15:00:45 IST ---

REVIEW: http://review.gluster.org/9768 (features/locks: Fixing
mandatory-locking code path) posted (#1) for review on master by Anoop C S
(achiraya at redhat.com)

--- Additional comment from Anand Avati on 2015-03-12 17:23:57 IST ---

REVIEW: http://review.gluster.org/9768 (features/locks: Fixing
mandatory-locking code path) posted (#2) for review on master by Anoop C S
(achiraya at redhat.com)

--- Additional comment from Anand Avati on 2015-03-30 14:17:13 IST ---

REVIEW: http://review.gluster.org/9768 (features/locks: Fixing
mandatory-locking code path) posted (#3) for review on master by Anoop C S
(achiraya at redhat.com)

--- Additional comment from Anand Avati on 2015-04-29 17:24:35 IST ---

REVIEW: http://review.gluster.org/9768 (features/locks: Fixing
mandatory-locking code path) posted (#4) for review on master by Anoop C S
(achiraya at redhat.com)

--- Additional comment from Anand Avati on 2015-05-04 15:19:28 IST ---

REVIEW: http://review.gluster.org/9768 (features/locks: Fixing
mandatory-locking code path) posted (#5) for review on master by Anoop C S
(achiraya at redhat.com)

--- Additional comment from Anand Avati on 2015-06-04 16:30:05 IST ---

REVIEW: http://review.gluster.org/9768 (features/locks: Fixing
mandatory-locking code path) posted (#7) for review on master by Anoop C S
(achiraya at redhat.com)

--- Additional comment from Anand Avati on 2015-06-04 16:30:08 IST ---

REVIEW: http://review.gluster.org/11082 (features/locks: Include zerofill fop
in locks translator) posted (#2) for review on master by Anoop C S
(achiraya at redhat.com)

--- Additional comment from Anand Avati on 2015-06-05 11:05:42 IST ---

REVIEW: http://review.gluster.org/9768 (features/locks: Fixing
mandatory-locking code path) posted (#8) for review on master by Anoop C S
(achiraya at redhat.com)

--- Additional comment from Anand Avati on 2015-06-05 11:05:45 IST ---

REVIEW: http://review.gluster.org/11082 (features/locks: Include zerofill call
in locks translator) posted (#3) for review on master by Anoop C S
(achiraya at redhat.com)

--- Additional comment from Anand Avati on 2015-06-08 17:23:27 IST ---

REVIEW: http://review.gluster.org/9768 (features/locks: Fixing
mandatory-locking code path) posted (#9) for review on master by Anoop C S
(achiraya at redhat.com)

--- Additional comment from Anand Avati on 2015-06-11 14:43:09 IST ---

REVIEW: http://review.gluster.org/9768 (features/locks: Fixing
mandatory-locking code path) posted (#10) for review on master by Anoop C S
(achiraya at redhat.com)

--- Additional comment from Anand Avati on 2015-08-24 16:52:00 IST ---

REVIEW: http://review.gluster.org/11177 (gfapi: Add API support for applying
mandatory-locks) posted (#2) for review on master by Anoop C S
(anoopcs at redhat.com)

--- Additional comment from Anand Avati on 2015-08-26 15:52:41 IST ---

REVIEW: http://review.gluster.org/11177 (gfapi: Add API support for applying
mandatory-locks) posted (#3) for review on master by Raghavendra Talur
(rtalur at redhat.com)

--- Additional comment from Kaleb KEITHLEY on 2015-10-22 21:16:38 IST ---

because of the large number of bugs filed against mainline version\ is
ambiguous and about to be removed as a choice.

If you believe this is still a bug, please change the status back to NEW and
choose the appropriate, applicable version for it.

--- Additional comment from Anoop C S on 2015-11-24 14:54:04 IST ---

As part of preparing GlusterFS to work with multi-Protocol environment we have
to support basic mandatory locking semantics. Please see the following design
doc for more details.

http://review.gluster.org/#/c/12014/1/in_progress/Mandatory+Locks.md

--- Additional comment from Vijay Bellur on 2015-12-02 06:29:38 IST ---

REVIEW: http://review.gluster.org/9768 (features/locks: Fixing
mandatory-locking code path) posted (#12) for review on master by Anoop C S
(anoopcs at redhat.com)

--- Additional comment from Vijay Bellur on 2015-12-02 06:29:42 IST ---

REVIEW: http://review.gluster.org/11082 (features/locks: Include zerofill fop
in locks translator) posted (#6) for review on master by Anoop C S
(anoopcs at redhat.com)

--- Additional comment from Vijay Bellur on 2015-12-02 13:01:09 IST ---

REVIEW: http://review.gluster.org/11177 (gfapi: Add API support for applying
mandatory-locks) posted (#5) for review on master by Anoop C S
(anoopcs at redhat.com)

--- Additional comment from Vijay Bellur on 2015-12-09 18:02:09 IST ---

REVIEW: http://review.gluster.org/9768 (features/locks: Fixing
mandatory-locking code path) posted (#13) for review on master by Anoop C S
(anoopcs at redhat.com)

--- Additional comment from Vijay Bellur on 2015-12-09 18:02:13 IST ---

REVIEW: http://review.gluster.org/11177 (gfapi: Add API support for applying
mandatory-locks) posted (#6) for review on master by Anoop C S
(anoopcs at redhat.com)

--- Additional comment from Vijay Bellur on 2015-12-09 18:02:17 IST ---

REVIEW: http://review.gluster.org/11082 (features/locks: Include zerofill fop
in locks translator) posted (#7) for review on master by Anoop C S
(anoopcs at redhat.com)

--- Additional comment from Vijay Bellur on 2015-12-09 18:02:20 IST ---

REVIEW: http://review.gluster.org/12924 (tests: Add libgfapi test case for
mandatory locks) posted (#1) for review on master by Anoop C S
(anoopcs at redhat.com)

--- Additional comment from Vijay Bellur on 2016-01-06 12:52:08 IST ---

REVIEW: http://review.gluster.org/9768 (features/locks: Fixing
mandatory-locking code path) posted (#14) for review on master by Anoop C S
(anoopcs at redhat.com)

--- Additional comment from Vijay Bellur on 2016-01-06 12:52:13 IST ---

REVIEW: http://review.gluster.org/11177 (gfapi: Add API support for applying
mandatory-locks) posted (#7) for review on master by Anoop C S
(anoopcs at redhat.com)

--- Additional comment from Vijay Bellur on 2016-01-06 12:52:17 IST ---

REVIEW: http://review.gluster.org/11082 (features/locks: Include zerofill fop
in locks translator) posted (#8) for review on master by Anoop C S
(anoopcs at redhat.com)

--- Additional comment from Vijay Bellur on 2016-01-19 14:39:24 IST ---

REVIEW: http://review.gluster.org/9768 (features/locks: Implement mandatory
locks) posted (#15) for review on master by Anoop C S (anoopcs at redhat.com)

--- Additional comment from Vijay Bellur on 2016-01-19 14:39:28 IST ---

REVIEW: http://review.gluster.org/11177 (libgfapi: Add support for
mandatory-locks) posted (#8) for review on master by Anoop C S
(anoopcs at redhat.com)

--- Additional comment from Vijay Bellur on 2016-02-05 17:24:53 IST ---

REVIEW: http://review.gluster.org/9768 (features/locks: Implement mandatory
locks) posted (#16) for review on master by Anoop C S (anoopcs at redhat.com)

--- Additional comment from Vijay Bellur on 2016-02-05 17:24:56 IST ---

REVIEW: http://review.gluster.org/11177 (libgfapi: Add new api for supporting
mandatory-locks) posted (#9) for review on master by Anoop C S
(anoopcs at redhat.com)

--- Additional comment from Vijay Bellur on 2016-02-10 18:27:22 IST ---

REVIEW: http://review.gluster.org/9768 (features/locks: Implement mandatory
locks) posted (#17) for review on master by Anoop C S (anoopcs at redhat.com)

--- Additional comment from Vijay Bellur on 2016-02-10 18:27:25 IST ---

REVIEW: http://review.gluster.org/11177 (libgfapi: Add new api for supporting
mandatory-locks) posted (#10) for review on master by Anoop C S
(anoopcs at redhat.com)

--- Additional comment from Vijay Bellur on 2016-02-18 17:47:24 IST ---

REVIEW: http://review.gluster.org/9768 (features/locks: Implement mandatory
locks) posted (#18) for review on master by Anoop C S (anoopcs at redhat.com)

--- Additional comment from Vijay Bellur on 2016-02-18 17:47:27 IST ---

REVIEW: http://review.gluster.org/11177 (libgfapi: Add new api for supporting
mandatory-locks) posted (#11) for review on master by Anoop C S
(anoopcs at redhat.com)

--- Additional comment from Vijay Bellur on 2016-02-18 18:26:14 IST ---

REVIEW: http://review.gluster.org/9768 (features/locks: Implement mandatory
locks) posted (#19) for review on master by Anoop C S (anoopcs at redhat.com)

--- Additional comment from Vijay Bellur on 2016-02-18 18:26:18 IST ---

REVIEW: http://review.gluster.org/11177 (libgfapi: Add new api for supporting
mandatory-locks) posted (#12) for review on master by Anoop C S
(anoopcs at redhat.com)

--- Additional comment from Vijay Bellur on 2016-03-01 21:28:44 IST ---

REVIEW: http://review.gluster.org/11177 (libgfapi: Add new api for supporting
mandatory-locks) posted (#13) for review on master by Anoop C S
(anoopcs at redhat.com)

--- Additional comment from Vijay Bellur on 2016-03-03 20:31:04 IST ---

REVIEW: http://review.gluster.org/9768 (features/locks: Implement mandatory
locks) posted (#20) for review on master by Anoop C S (anoopcs at redhat.com)

--- Additional comment from Mike McCune on 2016-03-29 03:45:42 IST ---

This bug was accidentally moved from POST to MODIFIED via an error in
automation, please see mmccune at redhat.com with any questions

--- Additional comment from Vijay Bellur on 2016-04-12 23:37:38 IST ---

REVIEW: http://review.gluster.org/9768 (features/locks: Implement mandatory
locks) posted (#21) for review on master by Anoop C S (anoopcs at redhat.com)

--- Additional comment from Vijay Bellur on 2016-04-13 10:47:56 IST ---

REVIEW: http://review.gluster.org/9768 (features/locks: Implement mandatory
locks) posted (#22) for review on master by Anoop C S (anoopcs at redhat.com)

--- Additional comment from Vijay Bellur on 2016-04-29 11:03:47 IST ---

REVIEW: http://review.gluster.org/9768 (features/locks: Implement mandatory
locks) posted (#23) for review on master by Anoop C S (anoopcs at redhat.com)

--- Additional comment from Vijay Bellur on 2016-05-02 16:48:49 IST ---

COMMIT: http://review.gluster.org/9768 committed in master by Pranith Kumar
Karampuri (pkarampu at redhat.com) 
------
commit 4517bf8dd6de310950cc5a612955aa3a2fddb57e
Author: Anoop C S <anoopcs at redhat.com>
Date:   Wed Feb 4 10:34:33 2015 +0530

    features/locks: Implement mandatory locks

    Initial change to fix/enable the mandatory locking support in GlusterFS
    as per the following design:

    https://review.gluster.org/#/c/12014/

    Accordingly 'locks.mandatory-locking' option is available as part of this
    change which will accept one among the following values:

    * off
    * file
    * forced
    * optimal

    See design doc for more details

    Change-Id: I14c489b3f8af5ebcbfa155a03f0c175e9558ac46
    BUG: 762184
    Signed-off-by: Anoop C S <anoopcs at redhat.com>
    Reviewed-on: http://review.gluster.org/9768
    Smoke: Gluster Build System <jenkins at build.gluster.com>
    CentOS-regression: Gluster Build System <jenkins at build.gluster.com>
    NetBSD-regression: NetBSD Build System <jenkins at build.gluster.org>
    Reviewed-by: Poornima G <pgurusid at redhat.com>
    Reviewed-by: Raghavendra Talur <rtalur at redhat.com>
    Reviewed-by: Rajesh Joseph <rjoseph at redhat.com>
    Reviewed-by: Pranith Kumar Karampuri <pkarampu at redhat.com>

Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=762184
[Bug 762184] Support mandatory locking in glusterfs
https://bugzilla.redhat.com/show_bug.cgi?id=1317278
[Bug 1317278] GlusterFS 3.8.0 tracker
-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are the assignee for the bug.