Bug 1089414

Summary:	Need support for handle based Ops to fetch/modify extended attributes of a file
Product:	[Community] GlusterFS	Reporter:	Soumya Koduri <skoduri>
Component:	libgfapi	Assignee:	Soumya Koduri <skoduri>
Status:	CLOSED CURRENTRELEASE	QA Contact:	Sudhir D <sdharane>
Severity:	unspecified	Docs Contact:
Priority:	unspecified
Version:	mainline	CC:	gluster-bugs, ndevos
Target Milestone:	---
Target Release:	---
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:	glusterfs-3.5.1	Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2014-07-11 19:37:26 UTC	Type:	Bug
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Embargoed:
Bug Depends On:
Bug Blocks:	1099878

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)