[Bugs] [Bug 1501956] New: gfapi: API needed to set lk_owner

[bugzilla at redhat.com]

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

            Bug ID: 1501956
           Summary: gfapi: API needed to set lk_owner
           Product: GlusterFS
           Version: 3.12
         Component: libgfapi
          Keywords: Triaged
          Severity: high
          Assignee: bugs at gluster.org
          Reporter: skoduri at redhat.com
        QA Contact: bugs at gluster.org
                CC: bugs at gluster.org, ndevos at redhat.com
        Depends On: 1499641
            Blocks: 1500704



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

Description of problem:

Glusterfs server differentiates locks based on both client-id and lock owner.
There is no provision to set lk_owner as of now via gfapi. Hence a new API is
needed to set the same. Since the lk_owner is associated with a particular glfd
(until we get mandatory lk support), the API definition looks like below -

"glfd_set_lk_owner()"


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 Soumya Koduri on 2017-10-09 03:34:27 EDT ---

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

--- Additional comment from Worker Ant on 2017-10-09 06:06:12 EDT ---

REVIEW: https://review.gluster.org/18429 (gfapi: set lkowner in glfd) posted
(#3) for review on master by soumya k (skoduri at redhat.com)

--- Additional comment from Worker Ant on 2017-10-10 07:57:54 EDT ---

REVIEW: https://review.gluster.org/18429 (gfapi: set lkowner in glfd) posted
(#4) for review on master by soumya k (skoduri at redhat.com)

--- Additional comment from Worker Ant on 2017-10-12 06:42:51 EDT ---

REVIEW: https://review.gluster.org/18429 (gfapi: set lkowner in glfd) posted
(#5) for review on master by soumya k (skoduri at redhat.com)

--- Additional comment from Worker Ant on 2017-10-13 03:48:04 EDT ---

COMMIT: https://review.gluster.org/18429 committed in master by Niels de Vos
(ndevos at redhat.com) 
------
commit c22cec3928be6924fb1026ef969cae897efbd76d
Author: Soumya Koduri <skoduri at redhat.com>
Date:   Wed Oct 4 14:34:48 2017 +0530

    gfapi: set lkowner in glfd

    We need a provision to be able to set lkowner (which is
    used to distinguish locks maintained by server) in gfapi.
    Since the same lk_owner need to be used to be able to
    flush the lock while closing the fd, store the lkowner
    in the glfd structure itself.

    A new API has been added to be able to set lkowner in glfd.

    Change-Id: I67591d6b9a89c20b9617d52616513ff9e6c06b47
    BUG: 1499641
    Signed-off-by: Soumya Koduri <skoduri at redhat.com>

--- Additional comment from Worker Ant on 2017-10-13 10:52:51 EDT ---

REVIEW: https://review.gluster.org/18522 (gfapi: update symbol version of
glfd_set_lkowner to 3.10.7) posted (#1) for review on master by soumya k
(skoduri at redhat.com)


Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1499641
[Bug 1499641] gfapi: API needed to set lk_owner
https://bugzilla.redhat.com/show_bug.cgi?id=1500704
[Bug 1500704] gfapi: API needed to set lk_owner
-- 
You are receiving this mail because:
You are the QA Contact for the bug.
You are on the CC list for the bug.
You are the assignee for the bug.