October 2018 - Devel - NFS Ganesha List Archives

Change in ffilz/nfs-ganesha[next]: NFS: remove ATTR_ACL if request empty acl
by GerritHub 26 Oct '18

26 Oct '18

From Kinglong Mee <kinglongmee(a)gmail.com>: Kinglong Mee has uploaded this change for review. ( https://review.gerrithub.io/430920 Change subject: NFS: remove ATTR_ACL if request empty acl ...................................................................... NFS: remove ATTR_ACL if request empty acl glusterfs_process_acl :FSAL :CRIT :setattr acl is NULL Change-Id: I0a4d345888d621524bda488bb1f99297876e60ee Signed-off-by: Kinglong Mee <kinglongmee(a)gmail.com> --- M src/Protocols/NFS/nfs_proto_tools.c 1 file changed, 7 insertions(+), 1 deletion(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/20/430920/1 -- To view, visit https://review.gerrithub.io/430920 To unsubscribe, or for help writing mail filters, visit https://review.gerrithub.io/settings Gerrit-Project: ffilz/nfs-ganesha Gerrit-Branch: next Gerrit-MessageType: newchange Gerrit-Change-Id: I0a4d345888d621524bda488bb1f99297876e60ee Gerrit-Change-Number: 430920 Gerrit-PatchSet: 1 Gerrit-Owner: Kinglong Mee <kinglongmee(a)gmail.com>

1 0

Change in ffilz/nfs-ganesha[next]: FSAL_GLUSTER: Copy user creds and lease_id after reopening fd
by GerritHub 26 Oct '18

26 Oct '18

From Soumya <skoduri(a)redhat.com>: Soumya has uploaded this change for review. ( https://review.gerrithub.io/430902 Change subject: FSAL_GLUSTER: Copy user creds and lease_id after reopening fd ...................................................................... FSAL_GLUSTER: Copy user creds and lease_id after reopening fd After OPEN upgrade/downgrade, copy user_creds and leaseid as well into the newly opened file descriptor. Change-Id: I44534efc12c05db090a50c156de659c9a4057705 Signed-off-by: Soumya Koduri <skoduri(a)redhat.com> --- M src/FSAL/FSAL_GLUSTER/handle.c 1 file changed, 7 insertions(+), 0 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/02/430902/1 -- To view, visit https://review.gerrithub.io/430902 To unsubscribe, or for help writing mail filters, visit https://review.gerrithub.io/settings Gerrit-Project: ffilz/nfs-ganesha Gerrit-Branch: next Gerrit-MessageType: newchange Gerrit-Change-Id: I44534efc12c05db090a50c156de659c9a4057705 Gerrit-Change-Number: 430902 Gerrit-PatchSet: 1 Gerrit-Owner: Soumya <skoduri(a)redhat.com>

1 0

Change in ffilz/nfs-ganesha[next]: Acquire state's fdlock in close2
by GerritHub 26 Oct '18

26 Oct '18

From Soumya <skoduri(a)redhat.com>: Soumya has uploaded this change for review. ( https://review.gerrithub.io/430893 Change subject: Acquire state's fdlock in close2 ...................................................................... Acquire state's fdlock in close2 Before closing the file descriptor, make sure no other thread is using it by acquiring fdlock. Change-Id: I6c1ba6315349eae6f9303f2a8bb1777f94711e5a Signed-off-by: Soumya Koduri <skoduri(a)redhat.com> --- M src/FSAL/FSAL_CEPH/handle.c M src/FSAL/FSAL_GLUSTER/handle.c M src/FSAL/FSAL_GPFS/file.c M src/FSAL/FSAL_VFS/file.c 4 files changed, 33 insertions(+), 3 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/93/430893/1 -- To view, visit https://review.gerrithub.io/430893 To unsubscribe, or for help writing mail filters, visit https://review.gerrithub.io/settings Gerrit-Project: ffilz/nfs-ganesha Gerrit-Branch: next Gerrit-MessageType: newchange Gerrit-Change-Id: I6c1ba6315349eae6f9303f2a8bb1777f94711e5a Gerrit-Change-Number: 430893 Gerrit-PatchSet: 1 Gerrit-Owner: Soumya <skoduri(a)redhat.com>

1 0

created NFS-Ganesha team in Launchpad
by Kaleb S. KEITHLEY 25 Oct '18

25 Oct '18

FYI Up to now I've been using the Gluster team in Ubuntu Launchpad to build nfs-ganesha and ntirpc "community" packages for Ubuntu. Someone may have looked at that and thought that those packages were only for use with GlusterFS, however that was never the actually the case. Regardless—— Today I created the NFS-Ganesha team in Ubuntu Launchpad at [1]. From today "community" packages (that I build anyway) will be built and distributed from the NFS-Ganesha team's PPAs. (BTW, this is not a closed team, anyone may ask to be added to the team.) The existing packages in the gluster team PPAs at [2] will remain indefinitely. NFS-Ganesha packages for Fedora, Debian, CentOS and RHEL, and selected SUSE distributions from [3], [4], and [5]. [1] https://launchpad.net/~nfs-ganesha [2] https://launchpad.net/~gluster [3] https://download.nfs-ganesha.org/ [4] https://wiki.centos.org/SpecialInterestGroup/Storage [5] https://build.opensuse.org/project/subprojects/home:glusterfs -- Kaleb

2 1

Change in ffilz/nfs-ganesha[next]: Fix xprt initial refs
by GerritHub 25 Oct '18

25 Oct '18

From Daniel Gryniewicz <dang(a)redhat.com>: Daniel Gryniewicz has uploaded this change for review. ( https://review.gerrithub.io/430791 Change subject: Fix xprt initial refs ...................................................................... Fix xprt initial refs Pull up fix for xprt initial refs. There should have been 2 on create: one for the hash table, and one for the caller. Fix Ganesha to release the ref it should have gotten, but was not. Change-Id: I1e64e13de6d470590f83d21f903e1d28d4ed3645 Signed-off-by: Daniel Gryniewicz <dang(a)redhat.com> --- M src/MainNFSD/nfs_rpc_dispatcher_thread.c M src/libntirpc 2 files changed, 5 insertions(+), 1 deletion(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/91/430791/1 -- To view, visit https://review.gerrithub.io/430791 To unsubscribe, or for help writing mail filters, visit https://review.gerrithub.io/settings Gerrit-Project: ffilz/nfs-ganesha Gerrit-Branch: next Gerrit-MessageType: newchange Gerrit-Change-Id: I1e64e13de6d470590f83d21f903e1d28d4ed3645 Gerrit-Change-Number: 430791 Gerrit-PatchSet: 1 Gerrit-Owner: Daniel Gryniewicz <dang(a)redhat.com>

1 0

Change in ffilz/nfs-ganesha[next]: Add support for LTTng on external ntirpc
by GerritHub 25 Oct '18

25 Oct '18

From Daniel Gryniewicz <dang(a)redhat.com>: Daniel Gryniewicz has uploaded this change for review. ( https://review.gerrithub.io/430790 Change subject: Add support for LTTng on external ntirpc ...................................................................... Add support for LTTng on external ntirpc Change-Id: Ic41345ecc18d18a0249ae404910a40a63a432b68 Signed-off-by: Daniel Gryniewicz <dang(a)redhat.com> --- M src/CMakeLists.txt M src/cmake/modules/FindNTIRPC.cmake 2 files changed, 6 insertions(+), 0 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/90/430790/1 -- To view, visit https://review.gerrithub.io/430790 To unsubscribe, or for help writing mail filters, visit https://review.gerrithub.io/settings Gerrit-Project: ffilz/nfs-ganesha Gerrit-Branch: next Gerrit-MessageType: newchange Gerrit-Change-Id: Ic41345ecc18d18a0249ae404910a40a63a432b68 Gerrit-Change-Number: 430790 Gerrit-PatchSet: 1 Gerrit-Owner: Daniel Gryniewicz <dang(a)redhat.com>

1 0

Change in ffilz/nfs-ganesha[next]: fsal_rename : check conflicts for delegation in rename operation
by GerritHub 25 Oct '18

25 Oct '18

From Jiffin Tony Thottan <jthottan(a)redhat.com>: Jiffin Tony Thottan has uploaded this change for review. ( https://review.gerrithub.io/430711 Change subject: fsal_rename : check conflicts for delegation in rename operation ...................................................................... fsal_rename : check conflicts for delegation in rename operation TODO * Currently recall will happen only for the source entry, need to do it for destination(if exists) as per RFC * Do recall need to send if parent directory is renamed Change-Id: I88584b8891f644cd20c9e528b424b7b9701e20ea Signed-off-by: Jiffin Tony Thottan <jthottan(a)redhat.com> --- M src/FSAL/fsal_helper.c 1 file changed, 6 insertions(+), 1 deletion(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/11/430711/1 -- To view, visit https://review.gerrithub.io/430711 To unsubscribe, or for help writing mail filters, visit https://review.gerrithub.io/settings Gerrit-Project: ffilz/nfs-ganesha Gerrit-Branch: next Gerrit-MessageType: newchange Gerrit-Change-Id: I88584b8891f644cd20c9e528b424b7b9701e20ea Gerrit-Change-Number: 430711 Gerrit-PatchSet: 1 Gerrit-Owner: Jiffin Tony Thottan <jthottan(a)redhat.com>

1 0

Change in ffilz/nfs-ganesha[next]: fsal_setattr : check conflicts for delegations in chmod
by GerritHub 25 Oct '18

25 Oct '18

From Jiffin Tony Thottan <jthottan(a)redhat.com>: Jiffin Tony Thottan has uploaded this change for review. ( https://review.gerrithub.io/430710 Change subject: fsal_setattr : check conflicts for delegations in chmod ...................................................................... fsal_setattr : check conflicts for delegations in chmod Also replaced existing conflict check for size in nfs3_setattr and nfs4_op_setattr by the check in fsal_setattr Change-Id: I2638ba5c8e8313794672b66007b9e9f7f2cdfdf8 Signed-off-by: Jiffin Tony Thottan <jthottan(a)redhat.com> --- M src/FSAL/fsal_helper.c M src/Protocols/NFS/nfs3_setattr.c M src/Protocols/NFS/nfs4_op_setattr.c 3 files changed, 4 insertions(+), 19 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/10/430710/1 -- To view, visit https://review.gerrithub.io/430710 To unsubscribe, or for help writing mail filters, visit https://review.gerrithub.io/settings Gerrit-Project: ffilz/nfs-ganesha Gerrit-Branch: next Gerrit-MessageType: newchange Gerrit-Change-Id: I2638ba5c8e8313794672b66007b9e9f7f2cdfdf8 Gerrit-Change-Number: 430710 Gerrit-PatchSet: 1 Gerrit-Owner: Jiffin Tony Thottan <jthottan(a)redhat.com>

1 0

Change in ffilz/nfs-ganesha[next]: fsal_link : conflict checks for delegation in link operation
by GerritHub 25 Oct '18

25 Oct '18

From Jiffin Tony Thottan <jthottan(a)redhat.com>: Jiffin Tony Thottan has uploaded this change for review. ( https://review.gerrithub.io/430709 Change subject: fsal_link : conflict checks for delegation in link operation ...................................................................... fsal_link : conflict checks for delegation in link operation Change-Id: I3a0c913dc58abe9baf78e6ce71cc0c9422eb5e81 Signed-off-by: Jiffin Tony Thottan <jthottan(a)redhat.com> --- M src/FSAL/fsal_helper.c 1 file changed, 5 insertions(+), 0 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/09/430709/1 -- To view, visit https://review.gerrithub.io/430709 To unsubscribe, or for help writing mail filters, visit https://review.gerrithub.io/settings Gerrit-Project: ffilz/nfs-ganesha Gerrit-Branch: next Gerrit-MessageType: newchange Gerrit-Change-Id: I3a0c913dc58abe9baf78e6ce71cc0c9422eb5e81 Gerrit-Change-Number: 430709 Gerrit-PatchSet: 1 Gerrit-Owner: Jiffin Tony Thottan <jthottan(a)redhat.com>

1 0

Change in ffilz/nfs-ganesha[next]: fsal_remove : check conflicts for delegation in remove operation
by GerritHub 25 Oct '18

25 Oct '18

From Jiffin Tony Thottan <jthottan(a)redhat.com>: Jiffin Tony Thottan has uploaded this change for review. ( https://review.gerrithub.io/430708 Change subject: fsal_remove : check conflicts for delegation in remove operation ...................................................................... fsal_remove : check conflicts for delegation in remove operation Change-Id: Icbfc189c414840f54a96f37476ebf67f47c53d7b Signed-off-by: Jiffin Tony Thottan <jthottan(a)redhat.com> --- M src/FSAL/fsal_helper.c 1 file changed, 7 insertions(+), 0 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/08/430708/1 -- To view, visit https://review.gerrithub.io/430708 To unsubscribe, or for help writing mail filters, visit https://review.gerrithub.io/settings Gerrit-Project: ffilz/nfs-ganesha Gerrit-Branch: next Gerrit-MessageType: newchange Gerrit-Change-Id: Icbfc189c414840f54a96f37476ebf67f47c53d7b Gerrit-Change-Number: 430708 Gerrit-PatchSet: 1 Gerrit-Owner: Jiffin Tony Thottan <jthottan(a)redhat.com>

1 0