[Bugs] [Bug 1369026] New: RFE: Lease support for gluster

bugzilla at redhat.com bugzilla at redhat.com
Mon Aug 22 11:01:50 UTC 2016


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

            Bug ID: 1369026
           Summary: RFE: Lease support for gluster
           Product: GlusterFS
           Version: mainline
         Component: core
          Assignee: bugs at gluster.org
          Reporter: spalai at redhat.com
                CC: bugs at gluster.org, pgurusid at redhat.com,
                    skoduri at redhat.com
        Depends On: 1319992
            Blocks: 1337638



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

Description of problem:
This is a but for tracking the lease feature for Gluster.
Design doc can be found at http://review.gluster.org/#/c/11980/

Version-Release number of selected component (if applicable):


How reproducible:


Steps to Reproduce:
1.
2.
3.

Actual results:


Expected results:


Additional info:

--- Additional comment from Mike McCune on 2016-03-29 04:17:33 MVT ---

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-04 16:49:03 MVT ---

REVIEW: http://review.gluster.org/13901 (rpc: Change the way client uuid is
built) posted (#1) for review on master by Poornima G (pgurusid at redhat.com)

--- Additional comment from Vijay Bellur on 2016-04-18 13:46:22 MVT ---

REVIEW: http://review.gluster.org/11596 (core: add lease fop) posted (#19) for
review on master by Poornima G (pgurusid at redhat.com)

--- Additional comment from Vijay Bellur on 2016-04-18 13:46:25 MVT ---

REVIEW: http://review.gluster.org/11597 (Protocol: Add lease() fop) posted
(#18) for review on master by Poornima G (pgurusid at redhat.com)

--- Additional comment from Vijay Bellur on 2016-04-18 13:46:28 MVT ---

REVIEW: http://review.gluster.org/11598 (posix: Add lease() fop) posted (#18)
for review on master by Poornima G (pgurusid at redhat.com)

--- Additional comment from Vijay Bellur on 2016-04-18 13:46:30 MVT ---

REVIEW: http://review.gluster.org/11599 (io-threads: Add lease() fop) posted
(#18) for review on master by Poornima G (pgurusid at redhat.com)

--- Additional comment from Vijay Bellur on 2016-04-18 13:46:33 MVT ---

REVIEW: http://review.gluster.org/11643 (Leases: Add a server side xlator to
handle lease requests) posted (#17) for review on master by Poornima G
(pgurusid at redhat.com)

--- Additional comment from Vijay Bellur on 2016-04-18 13:46:35 MVT ---

REVIEW: http://review.gluster.org/11600 (gfapi: add lease() fop) posted (#18)
for review on master by Poornima G (pgurusid at redhat.com)

--- Additional comment from Vijay Bellur on 2016-04-18 13:46:38 MVT ---

REVIEW: http://review.gluster.org/11720 (dht: Add lease() fop) posted (#16) for
review on master by Poornima G (pgurusid at redhat.com)

--- Additional comment from Vijay Bellur on 2016-04-18 13:46:40 MVT ---

REVIEW: http://review.gluster.org/12496 (tests: Add lease test case) posted
(#14) for review on master by Poornima G (pgurusid at redhat.com)

--- Additional comment from Vijay Bellur on 2016-04-18 13:46:43 MVT ---

REVIEW: http://review.gluster.org/12876 (gfapi: Add api to set lkowner and
leaseid) posted (#10) for review on master by Poornima G (pgurusid at redhat.com)

--- Additional comment from Vijay Bellur on 2016-04-18 13:46:45 MVT ---

REVIEW: http://review.gluster.org/11722 (glusterd: volgen and volume set
changes for leases) posted (#17) for review on master by Poornima G
(pgurusid at redhat.com)

--- Additional comment from Vijay Bellur on 2016-04-18 13:46:47 MVT ---

REVIEW: http://review.gluster.org/14019 (gfapi: Lease recall cbk) posted (#1)
for review on master by Poornima G (pgurusid at redhat.com)

--- Additional comment from Vijay Bellur on 2016-04-18 13:50:38 MVT ---

REVIEW: http://review.gluster.org/11596 (core: add lease fop) posted (#20) for
review on master by Poornima G (pgurusid at redhat.com)

--- Additional comment from Vijay Bellur on 2016-04-18 13:50:40 MVT ---

REVIEW: http://review.gluster.org/14019 (gfapi: Lease recall cbk) posted (#2)
for review on master by Poornima G (pgurusid at redhat.com)

--- Additional comment from Vijay Bellur on 2016-04-18 13:50:43 MVT ---

REVIEW: http://review.gluster.org/11597 (Protocol: Add lease() fop) posted
(#19) for review on master by Poornima G (pgurusid at redhat.com)

--- Additional comment from Vijay Bellur on 2016-04-18 13:50:45 MVT ---

REVIEW: http://review.gluster.org/11598 (posix: Add lease() fop) posted (#19)
for review on master by Poornima G (pgurusid at redhat.com)

--- Additional comment from Vijay Bellur on 2016-04-18 13:50:48 MVT ---

REVIEW: http://review.gluster.org/11599 (io-threads: Add lease() fop) posted
(#19) for review on master by Poornima G (pgurusid at redhat.com)

--- Additional comment from Vijay Bellur on 2016-04-18 13:50:50 MVT ---

REVIEW: http://review.gluster.org/11643 (Leases: Add a server side xlator to
handle lease requests) posted (#18) for review on master by Poornima G
(pgurusid at redhat.com)

--- Additional comment from Vijay Bellur on 2016-04-18 13:50:53 MVT ---

REVIEW: http://review.gluster.org/11600 (gfapi: add lease() fop) posted (#19)
for review on master by Poornima G (pgurusid at redhat.com)

--- Additional comment from Vijay Bellur on 2016-04-18 13:50:55 MVT ---

REVIEW: http://review.gluster.org/11720 (dht: Add lease() fop) posted (#17) for
review on master by Poornima G (pgurusid at redhat.com)

--- Additional comment from Vijay Bellur on 2016-04-18 13:50:58 MVT ---

REVIEW: http://review.gluster.org/12496 (tests: Add lease test case) posted
(#15) for review on master by Poornima G (pgurusid at redhat.com)

--- Additional comment from Vijay Bellur on 2016-04-18 13:51:00 MVT ---

REVIEW: http://review.gluster.org/12876 (gfapi: Add api to set lkowner and
leaseid) posted (#11) for review on master by Poornima G (pgurusid at redhat.com)

--- Additional comment from Vijay Bellur on 2016-04-18 13:51:02 MVT ---

REVIEW: http://review.gluster.org/11722 (glusterd: volgen and volume set
changes for leases) posted (#18) for review on master by Poornima G
(pgurusid at redhat.com)

--- Additional comment from Vijay Bellur on 2016-04-21 15:01:41 MVT ---

COMMIT: http://review.gluster.org/11596 committed in master by Pranith Kumar
Karampuri (pkarampu at redhat.com) 
------
commit 63918789fbb5f3a2d381e0f960f2dcbaf1b7ff72
Author: Poornima G <pgurusid at redhat.com>
Date:   Thu Jul 9 15:06:49 2015 +0530

    core: add lease fop

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

--- Additional comment from Vijay Bellur on 2016-04-21 16:32:01 MVT ---

COMMIT: http://review.gluster.org/11599 committed in master by Pranith Kumar
Karampuri (pkarampu at redhat.com) 
------
commit ba9c40ab4ee238bb6f5e6e275f339484690bc84e
Author: Poornima G <pgurusid at redhat.com>
Date:   Thu Jul 9 15:28:59 2015 +0530

    io-threads: Add lease() fop

    Change-Id: Ie4921867948d23b8b6c570196e88680cdb5ebfbc
    BUG: 1319992
    Signed-off-by: Poornima G <pgurusid at redhat.com>
    Reviewed-on: http://review.gluster.org/11599
    Smoke: Gluster Build System <jenkins at build.gluster.com>
    NetBSD-regression: NetBSD Build System <jenkins at build.gluster.org>
    CentOS-regression: Gluster Build System <jenkins at build.gluster.com>
    Reviewed-by: Raghavendra Talur <rtalur at redhat.com>

--- Additional comment from Vijay Bellur on 2016-04-21 16:32:16 MVT ---

COMMIT: http://review.gluster.org/11598 committed in master by Pranith Kumar
Karampuri (pkarampu at redhat.com) 
------
commit f8a6cd1d2715fc13f130b7c76d8aea75de8b5133
Author: Poornima G <pgurusid at redhat.com>
Date:   Thu Jul 9 15:31:30 2015 +0530

    posix: Add lease() fop

    Change-Id: I20007d7ff4536ae8180425c84a1aa7863a6f2447
    BUG: 1319992
    Signed-off-by: Poornima G <pgurusid at redhat.com>
    Reviewed-on: http://review.gluster.org/11598
    Smoke: Gluster Build System <jenkins at build.gluster.com>
    NetBSD-regression: NetBSD Build System <jenkins at build.gluster.org>
    CentOS-regression: Gluster Build System <jenkins at build.gluster.com>
    Reviewed-by: Raghavendra Talur <rtalur at redhat.com>

--- Additional comment from Vijay Bellur on 2016-04-25 19:30:19 MVT ---

COMMIT: http://review.gluster.org/11720 committed in master by Jeff Darcy
(jdarcy at redhat.com) 
------
commit 3021a9ab666e9f1a5116df84e77659aca613e955
Author: Poornima G <pgurusid at redhat.com>
Date:   Mon Jul 20 14:29:24 2015 +0530

    dht: Add lease() fop

    Change-Id: I0bbc2c2ef115c78393f6570815a5b80316e7e4be
    BUG: 1319992
    Signed-off-by: Poornima G <pgurusid at redhat.com>
    Reviewed-on: http://review.gluster.org/11720
    Smoke: Gluster Build System <jenkins at build.gluster.com>
    NetBSD-regression: NetBSD Build System <jenkins at build.gluster.org>
    CentOS-regression: Gluster Build System <jenkins at build.gluster.com>
    Reviewed-by: Raghavendra Talur <rtalur at redhat.com>
    Reviewed-by: Rajesh Joseph <rjoseph at redhat.com>
    Reviewed-by: Jeff Darcy <jdarcy at redhat.com>

--- Additional comment from Vijay Bellur on 2016-04-27 15:49:58 MVT ---

REVIEW: http://review.gluster.org/11597 (Protocol: Add lease() fop) posted
(#20) for review on master by Poornima G (pgurusid at redhat.com)

--- Additional comment from Vijay Bellur on 2016-04-27 16:01:42 MVT ---

REVIEW: http://review.gluster.org/11597 (Protocol: Add lease fop) posted (#21)
for review on master by Poornima G (pgurusid at redhat.com)

--- Additional comment from Vijay Bellur on 2016-04-27 16:01:44 MVT ---

REVIEW: http://review.gluster.org/11643 (Leases: Add a server side xlator to
handle lease requests) posted (#19) for review on master by Poornima G
(pgurusid at redhat.com)

--- Additional comment from Vijay Bellur on 2016-04-29 10:27:52 MVT ---

REVIEW: http://review.gluster.org/11597 (Protocol: Add lease fop) posted (#22)
for review on master by Poornima G (pgurusid at redhat.com)

--- Additional comment from Vijay Bellur on 2016-04-29 22:07:05 MVT ---

REVIEW: http://review.gluster.org/11597 (Protocol: Add lease fop) posted (#23)
for review on master by Poornima G (pgurusid at redhat.com)

--- Additional comment from Vijay Bellur on 2016-04-30 10:37:43 MVT ---

COMMIT: http://review.gluster.org/11597 committed in master by Pranith Kumar
Karampuri (pkarampu at redhat.com) 
------
commit 3c8922d123471de0c5da499420fcac6b10179786
Author: Poornima G <pgurusid at redhat.com>
Date:   Thu Jul 9 15:22:43 2015 +0530

    Protocol: Add lease fop

    Change-Id: I64c361d3e4ae86d57dc18bb887758d044c861237
    BUG: 1319992
    Signed-off-by: Poornima G <pgurusid at redhat.com>
    Reviewed-on: http://review.gluster.org/11597
    Smoke: Gluster Build System <jenkins at build.gluster.com>
    NetBSD-regression: NetBSD Build System <jenkins at build.gluster.org>
    CentOS-regression: Gluster Build System <jenkins at build.gluster.com>
    Reviewed-by: Rajesh Joseph <rjoseph at redhat.com>
    Reviewed-by: Pranith Kumar Karampuri <pkarampu at redhat.com>

--- Additional comment from Vijay Bellur on 2016-04-30 10:39:09 MVT ---

COMMIT: http://review.gluster.org/11643 committed in master by Pranith Kumar
Karampuri (pkarampu at redhat.com) 
------
commit 9b60198b1a22228f85615e377c9b0cbe1e4f4ce6
Author: Poornima G <pgurusid at redhat.com>
Date:   Wed May 27 12:55:50 2015 +0530

    Leases: Add a server side xlator to handle lease requests

    Before this patch, there was an effort to implement leases
    in upcall xlator, these patches by Soumya and me can be
    found @ http://review.gluster.org/#/c/10084/

    Change-Id: I926728c7ec690727a8971039b240655882d02059
    BUG: 1319992
    Signed-off-by: Poornima G <pgurusid at redhat.com>
    Reviewed-on: http://review.gluster.org/11643
    Smoke: Gluster Build System <jenkins at build.gluster.com>
    NetBSD-regression: NetBSD Build System <jenkins at build.gluster.org>
    CentOS-regression: Gluster Build System <jenkins at build.gluster.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>

--- Additional comment from Vijay Bellur on 2016-05-01 19:52:31 MVT ---

REVIEW: http://review.gluster.org/11722 (glusterd: volgen and volume set
changes for leases) posted (#19) for review on master by Poornima G
(pgurusid at redhat.com)

--- Additional comment from Vijay Bellur on 2016-05-05 13:57:30 MVT ---

REVIEW: http://review.gluster.org/11722 (glusterd: volgen and volume set
changes for leases) posted (#20) for review on master by Poornima G
(pgurusid at redhat.com)

--- Additional comment from Vijay Bellur on 2016-05-10 10:56:42 MVT ---

REVIEW: http://review.gluster.org/14275 (leases: Fix the notify being lost)
posted (#1) for review on master by Poornima G (pgurusid at redhat.com)

--- Additional comment from Vijay Bellur on 2016-05-11 12:00:05 MVT ---

REVIEW: http://review.gluster.org/13901 (rpc: Change the way client uuid is
built) posted (#2) for review on master by Susant Palai (spalai at redhat.com)

--- Additional comment from Vijay Bellur on 2016-05-12 09:36:47 MVT ---

REVIEW: http://review.gluster.org/14275 (leases: Fix the notify being lost)
posted (#2) for review on master by Poornima G (pgurusid at redhat.com)

--- Additional comment from Vijay Bellur on 2016-05-16 10:18:45 MVT ---

COMMIT: http://review.gluster.org/14275 committed in master by Raghavendra G
(rgowdapp at redhat.com) 
------
commit 58c577994b2bf67b6f8d9749d9c7d678edfc9ceb
Author: Poornima G <pgurusid at redhat.com>
Date:   Tue May 10 01:50:08 2016 -0400

    leases: Fix the notify being lost

    The notify() implemented doesn't really add any value
    and the EXIT_IF_LEASES_OFF in notify() can cause the
    notifications to be lost if leases if off.

    Hence remove the notify definition from leases,
    also fix a log level

    Change-Id: I8bf30d26f08034a16a450ed811dcf8ac57d70a94
    BUG: 1319992
    Signed-off-by: Poornima G <pgurusid at redhat.com>
    Reviewed-on: http://review.gluster.org/14275
    Smoke: Gluster Build System <jenkins at build.gluster.com>
    NetBSD-regression: NetBSD Build System <jenkins at build.gluster.org>
    Reviewed-by: Raghavendra Talur <rtalur at redhat.com>
    CentOS-regression: Gluster Build System <jenkins at build.gluster.com>
    Reviewed-by: Raghavendra G <rgowdapp at redhat.com>

--- Additional comment from Vijay Bellur on 2016-05-16 10:47:58 MVT ---

REVIEW: http://review.gluster.org/11600 (gfapi: add lease() fop) posted (#20)
for review on master by Poornima G (pgurusid at redhat.com)

--- Additional comment from Vijay Bellur on 2016-05-16 10:48:00 MVT ---

REVIEW: http://review.gluster.org/12496 (tests: Add lease test case) posted
(#16) for review on master by Poornima G (pgurusid at redhat.com)

--- Additional comment from Vijay Bellur on 2016-05-16 10:48:03 MVT ---

REVIEW: http://review.gluster.org/12876 (gfapi: Add api to set lkowner and
leaseid) posted (#12) for review on master by Poornima G (pgurusid at redhat.com)

--- Additional comment from Vijay Bellur on 2016-05-16 10:48:08 MVT ---

REVIEW: http://review.gluster.org/11722 (glusterd: volgen and volume set
changes for leases) posted (#21) for review on master by Poornima G
(pgurusid at redhat.com)

--- Additional comment from Poornima G on 2016-05-16 11:00:51 MVT ---

TODOs in the descending order of priority:

1. Support of leases for EC and Sharded volumes.

2. Lease healing in case of replicated(-like) volumes

3. In case of network disconnect between gfapi client and gluster-server
process, we should replay leases in case if server flushes them. Currently its
been looked as part of Lease healing mentioned in (3). So we shall not have
this support in case of plane distributed volume.

4.  In case of replicated(-like) volumes, we should be able to filter out
duplicate LEASE_RECALL requests sent by them.

5. In case of NFS-Ganesha restart, we shall need support on glusterfs-server to
be able to support reclaim of lost state by NFS clients. But if we failover
only after gluster-server flushes the locks, we should be good.

6. If the client is actively flushing the I/Os, we could extend recall timer.

7. During rebalance/date-tiering, we should migrate lease. But at worst
scenario, we could recall lease before migrating data.

--- Additional comment from Vijay Bellur on 2016-05-16 17:36:28 MVT ---

REVIEW: http://review.gluster.org/14360 (leases: Do not init the threads when
lease is disabled) posted (#1) for review on master by Poornima G
(pgurusid at redhat.com)

--- Additional comment from Vijay Bellur on 2016-05-17 10:30:58 MVT ---

REVIEW: http://review.gluster.org/14360 (leases: Do not init the threads when
lease is disabled) posted (#2) for review on master by Poornima G
(pgurusid at redhat.com)

--- Additional comment from Vijay Bellur on 2016-05-17 10:33:41 MVT ---

REVIEW: http://review.gluster.org/14370 (leases: Send "this" as cookie to the
timer handler) posted (#1) for review on master by Poornima G
(pgurusid at redhat.com)

--- Additional comment from Vijay Bellur on 2016-05-17 15:13:09 MVT ---

REVIEW: http://review.gluster.org/14374 (leases: Fix failure of RD_lease
request in certain scenario) posted (#1) for review on master by Poornima G
(pgurusid at redhat.com)

--- Additional comment from Vijay Bellur on 2016-05-17 15:50:06 MVT ---

REVIEW: http://review.gluster.org/14370 (leases: Send "this" as cookie to the
timer handler) posted (#2) for review on master by Poornima G
(pgurusid at redhat.com)

--- Additional comment from Vijay Bellur on 2016-05-19 09:45:17 MVT ---

REVIEW: http://review.gluster.org/14360 (leases: Do not init the threads when
lease is disabled) posted (#3) for review on master by Poornima G
(pgurusid at redhat.com)

--- Additional comment from Vijay Bellur on 2016-05-19 10:31:08 MVT ---

REVIEW: http://review.gluster.org/14360 (leases: Do not init the threads when
lease is disabled) posted (#4) for review on master by Poornima G
(pgurusid at redhat.com)

--- Additional comment from Vijay Bellur on 2016-05-19 14:28:06 MVT ---

COMMIT: http://review.gluster.org/14374 committed in master by Pranith Kumar
Karampuri (pkarampu at redhat.com) 
------
commit 005d18a623dc49397e2a83a7fb80607ae0a9aed9
Author: Poornima G <pgurusid at redhat.com>
Date:   Tue May 17 05:47:55 2016 -0400

    leases: Fix failure of RD_lease request in certain scenario

    Issue:
    When RW_lease exists for a given lease id and another request for
    RD_lease comes in with the same lease id, it gets rejected.

    Fix:
    Allow the RD_lease when RW_lease is present iff its from the
    same lease id.

    Change-Id: Id8ad542c649497f5e58f2acf5681c640d9202046
    BUG: 1319992
    Signed-off-by: Poornima G <pgurusid at redhat.com>
    Reviewed-on: http://review.gluster.org/14374
    Smoke: Gluster Build System <jenkins at build.gluster.com>
    NetBSD-regression: NetBSD Build System <jenkins at build.gluster.org>
    Reviewed-by: Raghavendra Talur <rtalur at redhat.com>
    CentOS-regression: Gluster Build System <jenkins at build.gluster.com>

--- Additional comment from Vijay Bellur on 2016-05-19 14:29:14 MVT ---

COMMIT: http://review.gluster.org/14370 committed in master by Pranith Kumar
Karampuri (pkarampu at redhat.com) 
------
commit 84924ee0ef7cb557c54a9d467364ded86b34f40d
Author: Poornima G <pgurusid at redhat.com>
Date:   Tue May 17 01:22:37 2016 -0400

    leases: Send "this" as cookie to the timer handler

    Issue:
    timer-wheel implementation doesn't set the THIS to point
    to the xlator who registered the timer, before calling
    the handler. Hence referencing to THIS is any timer handler
    will point to the global_xlator.

    Fix: This is a bug from the timer wheel, but until that gets
    fixed passing "this" as a cookie to the timer handler.

    Change-Id: Ife1be56dc100372f9211e8fc7a885ac717cbcf47
    BUG: 1319992
    Signed-off-by: Poornima G <pgurusid at redhat.com>
    Reviewed-on: http://review.gluster.org/14370
    Smoke: Gluster Build System <jenkins at build.gluster.com>
    NetBSD-regression: NetBSD Build System <jenkins at build.gluster.org>
    CentOS-regression: Gluster Build System <jenkins at build.gluster.com>
    Reviewed-by: Raghavendra Talur <rtalur at redhat.com>
    Reviewed-by: Pranith Kumar Karampuri <pkarampu at redhat.com>

--- Additional comment from Vijay Bellur on 2016-05-19 19:39:44 MVT ---

COMMIT: http://review.gluster.org/14360 committed in master by Jeff Darcy
(jdarcy at redhat.com) 
------
commit 6f1a71210ee0a0f3741b5ece3b5240c1e4b5fa6d
Author: Poornima G <pgurusid at redhat.com>
Date:   Mon May 16 08:27:41 2016 -0400

    leases: Do not init the threads when lease is disabled

    Change-Id: I08c4caf94bf3dfceba6f7d3cc8945c61d9b12dbc
    BUG: 1319992
    Signed-off-by: Poornima G <pgurusid at redhat.com>
    Reviewed-on: http://review.gluster.org/14360
    Smoke: Gluster Build System <jenkins at build.gluster.com>
    NetBSD-regression: NetBSD Build System <jenkins at build.gluster.org>
    CentOS-regression: Gluster Build System <jenkins at build.gluster.com>
    Reviewed-by: Jeff Darcy <jdarcy at redhat.com>

--- Additional comment from Vijay Bellur on 2016-05-19 22:50:20 MVT ---

REVIEW: http://review.gluster.org/11600 (gfapi: add lease() fop) posted (#21)
for review on master by Poornima G (pgurusid at redhat.com)

--- Additional comment from Vijay Bellur on 2016-05-19 22:50:22 MVT ---

REVIEW: http://review.gluster.org/12496 (tests: Add lease test case) posted
(#17) for review on master by Poornima G (pgurusid at redhat.com)

--- Additional comment from Vijay Bellur on 2016-05-19 22:50:25 MVT ---

REVIEW: http://review.gluster.org/12876 (gfapi: Add api to set lkowner and
leaseid) posted (#13) for review on master by Poornima G (pgurusid at redhat.com)

--- Additional comment from Vijay Bellur on 2016-05-19 22:50:29 MVT ---

REVIEW: http://review.gluster.org/11722 (glusterd: volgen and volume set
changes for leases) posted (#22) for review on master by Poornima G
(pgurusid at redhat.com)

--- Additional comment from Vijay Bellur on 2016-05-19 22:50:32 MVT ---

REVIEW: http://review.gluster.org/14434 (leases: Fix the recall code path)
posted (#1) for review on master by Poornima G (pgurusid at redhat.com)

--- Additional comment from Vijay Bellur on 2016-05-19 22:58:10 MVT ---

REVIEW: http://review.gluster.org/14434 (leases: Fix the recall code path)
posted (#2) for review on master by Poornima G (pgurusid at redhat.com)

--- Additional comment from Vijay Bellur on 2016-05-19 22:58:13 MVT ---

REVIEW: http://review.gluster.org/11600 (gfapi: add lease() fop) posted (#22)
for review on master by Poornima G (pgurusid at redhat.com)

--- Additional comment from Vijay Bellur on 2016-05-19 22:58:16 MVT ---

REVIEW: http://review.gluster.org/12496 (tests: Add lease test case) posted
(#18) for review on master by Poornima G (pgurusid at redhat.com)

--- Additional comment from Vijay Bellur on 2016-05-19 22:58:20 MVT ---

REVIEW: http://review.gluster.org/12876 (gfapi: Add api to set lkowner and
leaseid) posted (#14) for review on master by Poornima G (pgurusid at redhat.com)

--- Additional comment from Vijay Bellur on 2016-05-19 22:58:23 MVT ---

REVIEW: http://review.gluster.org/11722 (glusterd: volgen and volume set
changes for leases) posted (#23) for review on master by Poornima G
(pgurusid at redhat.com)

--- Additional comment from Vijay Bellur on 2016-06-01 12:33:14 MVT ---

COMMIT: http://review.gluster.org/14434 committed in master by Pranith Kumar
Karampuri (pkarampu at redhat.com) 
------
commit b06a77f3446d622f8159f08508f4d5063ec50339
Author: Poornima G <pgurusid at redhat.com>
Date:   Thu May 19 05:04:13 2016 -0400

    leases: Fix the recall code path

    1. Replace frame->op usage with frame->root->op, as frame->op
       is not filled with appropriate value in all cases
    2. Add few log messages
    3. Fix boolean assignment

    Change-Id: I340f2200c1fcc4f4ce5a139b0fd22508cb8ac1e3
    BUG: 1319992
    Signed-off-by: Poornima G <pgurusid at redhat.com>
    Reviewed-on: http://review.gluster.org/14434
    Smoke: Gluster Build System <jenkins at build.gluster.com>
    NetBSD-regression: NetBSD Build System <jenkins at build.gluster.org>
    Reviewed-by: Raghavendra Talur <rtalur at redhat.com>
    CentOS-regression: Gluster Build System <jenkins at build.gluster.com>
    Reviewed-by: Pranith Kumar Karampuri <pkarampu at redhat.com>

--- Additional comment from Vijay Bellur on 2016-06-02 11:50:59 MVT ---

REVIEW: http://review.gluster.org/11600 (gfapi: add lease() fop) posted (#23)
for review on master by Poornima G (pgurusid at redhat.com)

--- Additional comment from Vijay Bellur on 2016-06-02 11:51:02 MVT ---

REVIEW: http://review.gluster.org/12496 (tests: Add lease test case) posted
(#19) for review on master by Poornima G (pgurusid at redhat.com)

--- Additional comment from Vijay Bellur on 2016-06-02 11:51:05 MVT ---

REVIEW: http://review.gluster.org/12876 (gfapi: Add api to set lkowner and
leaseid) posted (#15) for review on master by Poornima G (pgurusid at redhat.com)

--- Additional comment from Vijay Bellur on 2016-06-02 11:51:08 MVT ---

REVIEW: http://review.gluster.org/11722 (glusterd: volgen and volume set
changes for leases) posted (#24) for review on master by Poornima G
(pgurusid at redhat.com)

--- Additional comment from Vijay Bellur on 2016-06-03 09:58:47 MVT ---

COMMIT: http://review.gluster.org/11722 committed in master by Atin Mukherjee
(amukherj at redhat.com) 
------
commit bc525b01002d7649be0ac28c5761c8ce92b6a658
Author: Poornima G <pgurusid at redhat.com>
Date:   Mon Jul 20 16:44:32 2015 +0530

    glusterd: volgen and volume set changes for leases

    Add leases xlator in volgen and also add corresponding volume set options

    Change-Id: Ic5de50cdb87eaf6a833e739bc7e08fecbeca3de3
    BUG: 1319992
    Signed-off-by: Poornima G <pgurusid at redhat.com>
    Reviewed-on: http://review.gluster.org/11722
    NetBSD-regression: NetBSD Build System <jenkins at build.gluster.org>
    CentOS-regression: Gluster Build System <jenkins at build.gluster.com>
    Smoke: Gluster Build System <jenkins at build.gluster.com>
    Reviewed-by: Atin Mukherjee <amukherj at redhat.com>

--- Additional comment from Vijay Bellur on 2016-06-05 09:21:49 MVT ---

REVIEW: http://review.gluster.org/14647 (protocol: Add framework to send
transaction id with recall) posted (#1) for review on master by Poornima G
(pgurusid at redhat.com)

--- Additional comment from Vijay Bellur on 2016-06-05 09:46:00 MVT ---

REVIEW: http://review.gluster.org/14647 (protocol: Add framework to send
transaction id with recall) posted (#2) for review on master by Poornima G
(pgurusid at redhat.com)

--- Additional comment from Vijay Bellur on 2016-06-05 10:06:56 MVT ---

REVIEW: http://review.gluster.org/14647 (protocol: Add framework to send
transaction id with recall) posted (#3) for review on master by Poornima G
(pgurusid at redhat.com)

--- Additional comment from Vijay Bellur on 2016-06-10 16:03:45 MVT ---

COMMIT: http://review.gluster.org/14647 committed in master by Raghavendra G
(rgowdapp at redhat.com) 
------
commit 3bc419d8014877f64e57dc3368b0609396ec0d12
Author: Poornima G <pgurusid at redhat.com>
Date:   Sun Jun 5 00:06:11 2016 -0400

    protocol: Add framework to send transaction id with recall

    Issue:
    The upcall(cache invalidation/recall) event is sent from the bricks
    to clients. In AFR/EC setup, it can so happen that all the bricks
    will send the upcall for the same event, and if AFR/EC doesn't filter
    out these duplicate notifications, the logic above cluster xlators
    can fail.

    Solution:
    Use transaction id to filter out duplicate notifications.

    This patch adds framework for duplicate notifications.
    AFR/EC can build up on this patch for deduping the notifications

    Change-Id: I66b08e63b8799bc5932f2b2545376138a5701168
    BUG: 1319992
    Signed-off-by: Poornima G <pgurusid at redhat.com>
    Reviewed-on: http://review.gluster.org/14647
    Smoke: Gluster Build System <jenkins at build.gluster.com>
    NetBSD-regression: NetBSD Build System <jenkins at build.gluster.org>
    CentOS-regression: Gluster Build System <jenkins at build.gluster.com>
    Reviewed-by: Raghavendra Talur <rtalur at redhat.com>
    Reviewed-by: Raghavendra G <rgowdapp at redhat.com>

--- Additional comment from Niels de Vos on 2016-06-16 17:31:57 MVT ---

This bug is getting closed because a release has been made available that
should address the reported issue. In case the problem is still not fixed with
glusterfs-3.8.0, please open a new bug report.

glusterfs-3.8.0 has been announced on the Gluster mailinglists [1], packages
for several distributions should become available in the near future. Keep an
eye on the Gluster Users mailinglist [2] and the update infrastructure for your
distribution.

[1] http://blog.gluster.org/2016/06/glusterfs-3-8-released/
[2] http://thread.gmane.org/gmane.comp.file-systems.gluster.user

--- Additional comment from Vijay Bellur on 2016-08-01 10:47:29 MVT ---

REVIEW: http://review.gluster.org/13901 (rpc: Change the way client uuid is
built) posted (#3) for review on master by Susant Palai (spalai at redhat.com)

--- Additional comment from Worker Ant on 2016-08-22 15:00:07 MVT ---

REVIEW: http://review.gluster.org/13901 (rpc: Change the way client uuid is
built) posted (#4) for review on master by Susant Palai (spalai at redhat.com)


Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1319992
[Bug 1319992] RFE: Lease support for gluster
https://bugzilla.redhat.com/show_bug.cgi?id=1337638
[Bug 1337638] Leases: Fix lease failures in certain scenarios
-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are the assignee for the bug.


More information about the Bugs mailing list