Bug 1089414 - Need support for handle based Ops to fetch/modify extended attributes of a file
Summary: Need support for handle based Ops to fetch/modify extended attributes of a file
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: GlusterFS
Classification: Community
Component: libgfapi
Version: mainline
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: Soumya Koduri
QA Contact: Sudhir D
URL:
Whiteboard:
Depends On:
Blocks: 1099878
TreeView+ depends on / blocked
 
Reported: 2014-04-18 20:16 UTC by Soumya Koduri
Modified: 2014-07-11 19:37 UTC (History)
2 users (show)

Fixed In Version: glusterfs-3.5.1
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2014-07-11 19:37:26 UTC
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Embargoed:


Attachments (Terms of Use)

Description Soumya Koduri 2014-04-18 20:16:36 UTC
Description of problem:
Currently there are no handle based ops defined in libgfapi to get/set extended attributes on a file using GlusterFS. 

Need support for the same to be used by NFS-ganesha (one of the ligfapi users).

Comment 1 Anand Avati 2014-04-18 20:20:01 UTC
REVIEW: http://review.gluster.org/7308 (Added Handle-based ops to get/set/remove extended attributes in the libgfapi.) posted (#3) for review on master by soumya k (skoduri)

Comment 2 Anand Avati 2014-04-18 20:21:16 UTC
COMMIT: http://review.gluster.org/7308 committed in master by Vijay Bellur (vbellur) 
------
commit 99d86b1a1afe62c06f7aa2e3d6bb68df7762ce48
Author: Soumya Koduri <skoduri>
Date:   Fri Mar 21 12:57:14 2014 +0530

    Added Handle-based ops to get/set/remove extended attributes in the libgfapi.
    
    Change-Id: I1a8e666018d7b93e0bba2d9882935681da909980
    BUG: 1089414
    Signed-off-by: Soumya Koduri <skoduri>
    Reviewed-on: http://review.gluster.org/7308
    Reviewed-by: Vijay Bellur <vbellur>
    Tested-by: Vijay Bellur <vbellur>

Comment 3 Anand Avati 2014-04-23 07:13:41 UTC
REVIEW: http://review.gluster.org/7529 (libgfapi: A minor bug-fix in glfs_h_setxattr and glfs_h_removexattr calls.) posted (#1) for review on master by soumya k (skoduri)

Comment 4 Anand Avati 2014-04-23 09:14:48 UTC
REVIEW: http://review.gluster.org/7531 (rpcsvc: Ignore INODELK/ENTRYLK/LK for throttling) posted (#1) for review on master by Pranith Kumar Karampuri (pkarampu)

Comment 5 Anand Avati 2014-04-29 01:21:54 UTC
COMMIT: http://review.gluster.org/7529 committed in master by Anand Avati (avati) 
------
commit c85d842c615b65da7f692cef36b5b9f51c62a0ca
Author: Soumya Koduri <skoduri>
Date:   Wed Apr 23 12:41:33 2014 +0530

    libgfapi: A minor bug-fix in glfs_h_setxattr and glfs_h_removexattr calls.
    
    Change-Id: I62f63da37edf722d6d79c75f72ee7403e93e4936
    BUG: 1089414
    Signed-off-by: Soumya Koduri <skoduri>
    Reviewed-on: http://review.gluster.org/7529
    Reviewed-by: Santosh Pradhan <spradhan>
    Tested-by: Meghana M <mmadhusu>
    Reviewed-by: Humble Devassy Chirammal <humble.devassy>
    Reviewed-by: Pranith Kumar Karampuri <pkarampu>
    Reviewed-by: Anand Avati <avati>

Comment 6 Niels de Vos 2014-05-21 11:44:20 UTC
This can not be ON_QA because there is no community glusterfs-3.5qa2-0.425.git9360107 release. If you need this for Red Hat Storage, you should clone this bug.

Comment 7 Anand Avati 2014-05-22 05:56:47 UTC
REVIEW: http://review.gluster.org/7841 (libgfapi: Fixed few memory leaks in glfs_h_getxattr, glfs_h_setxattr calls.) posted (#1) for review on master by soumya k (skoduri)

Comment 8 Anand Avati 2014-05-22 06:18:50 UTC
REVIEW: http://review.gluster.org/7841 (libgfapi: Fixed few memory leaks in glfs_h_getxattr, glfs_h_setxattr calls.           Also replaced the tabs with spaces in that entire file.) posted (#2) for review on master by soumya k (skoduri)

Comment 9 Anand Avati 2014-06-02 08:56:57 UTC
COMMIT: http://review.gluster.org/7841 committed in master by Vijay Bellur (vbellur) 
------
commit 0d26de1b0a4a0fbf7e33dab8bc46c38c179e8645
Author: Soumya Koduri <skoduri>
Date:   Thu May 22 11:22:12 2014 +0530

    libgfapi: Fixed few memory leaks in glfs_h_getxattr, glfs_h_setxattr calls.
              Also replaced the tabs with spaces in that entire file.
    
    Change-Id: Iee6dc031d27fe8d26caf90736d4b2ac7bc8c5206
    BUG: 1089414
    Signed-off-by: Soumya Koduri <skoduri>
    Reviewed-on: http://review.gluster.org/7841
    Reviewed-by: Raghavendra G <rgowdapp>
    Tested-by: Gluster Build System <jenkins.com>
    Reviewed-by: Meghana M <mmadhusu>
    Reviewed-by: Vijay Bellur <vbellur>

Comment 10 Soumya Koduri 2014-06-09 07:15:15 UTC
Verified the fix on the Glusterfs latest master branch available (Tag#v3.5qa2-579-g94fe2d3)


Note You need to log in before you can comment on or make changes to this bug.