August 2018 - Devel - NFS Ganesha List Archives

Announce Push of V2.7-rc3
by Frank Filz 24 Aug '18

24 Aug '18

Branch next Tag:V2.7-rc3 Release Highlights * rados_cluster: don't lift grace period unless everyone is enforcing Signed-off-by: Frank S. Filz <ffilzlnx(a)mindspring.com> Contents: 32e8a24 Frank S. Filz V2.7-rc3 e3a1341 Jeff Layton rados_cluster: don't lift grace period unless everyone is enforcing

1 0

Change in ffilz/nfs-ganesha[next]: MDCACHE - Take lock when bumping chunk
by GerritHub 22 Aug '18

22 Aug '18

From Daniel Gryniewicz <dang(a)redhat.com>: Daniel Gryniewicz has uploaded this change for review. ( https://review.gerrithub.io/423152 Change subject: MDCACHE - Take lock when bumping chunk ...................................................................... MDCACHE - Take lock when bumping chunk chunk_lru_queue_of() require the QLANE lock. Fixes issue #329 Change-Id: I8a6e01ca5688753bff8b415dbbde70f9b3ed708b Signed-off-by: Daniel Gryniewicz <dang(a)redhat.com> --- M src/FSAL/Stackable_FSALs/FSAL_MDCACHE/mdcache_lru.c 1 file changed, 2 insertions(+), 1 deletion(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/52/423152/1 -- To view, visit https://review.gerrithub.io/423152 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: I8a6e01ca5688753bff8b415dbbde70f9b3ed708b Gerrit-Change-Number: 423152 Gerrit-PatchSet: 1 Gerrit-Owner: Daniel Gryniewicz <dang(a)redhat.com>

1 0

NFSv4.0 and NFSv4.1 performance
by harsh vardhan 22 Aug '18

22 Aug '18

Hello, We observing difference in performance while R/W between NFS4 v4.1 and v4.0 v4.1 is considerable smaller than v4.0(We are trying with authentication as kerberos5) Is this is expected performance degradation from v4.0 to v4.1 ? Thanks and Regards Harsh Vardhan

1 0

Re: [Nfs-ganesha-devel] nfs4 idmapping issue with sssd fully qualified domain names
by Satya Prakash GS 22 Aug '18

22 Aug '18

We hit the exact bug that was mentioned here. https://bugzilla.redhat.com/show_bug.cgi?id=1378557 The issue was happening only with multiple AD domains configured and trust established between them. libnfsidmap was stripping the domain name if a username is passed in fully-qualified domain name format. No-Strip option in idmapd.conf has to be set to "both" to stop nfsidmap from stripping the domain name. Even with this option set, I could not get it working with libnfsidmap-0.25/26. I could only get it working with libnfsidmap-0.27. I compiled it, replaced both libraries, libnfsidmap.so and nsswitch.so and username was properly being passed to the layer below (sssd/winbind). With this fix, id is properly being resolved with sssd and winbind. Thanks, Satya.

1 0

Change in ffilz/nfs-ganesha[next]: SAL: don't lift grace period unless everyone is enforcing
by GerritHub 20 Aug '18

20 Aug '18

From Jeff Layton <jlayton(a)redhat.com>: Jeff Layton has uploaded this change for review. ( https://review.gerrithub.io/422894 Change subject: SAL: don't lift grace period unless everyone is enforcing ...................................................................... SAL: don't lift grace period unless everyone is enforcing When building a new cluster from scratch, we'll generally not have any clients at first. New nodes can end up being added to the cluster, start up and lift the grace period before existing nodes even notice that a new grace period has started. This can cause them to miss an epoch update. Fix this by not allowing a node to lift the grace period unless it's currently being enforced cluster-wide. Change-Id: Ie252883a094bee9f47dc7b8ae987688500a77257 Signed-off-by: Jeff Layton <jlayton(a)redhat.com> --- M src/SAL/nfs4_recovery.c 1 file changed, 4 insertions(+), 0 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/94/422894/1 -- To view, visit https://review.gerrithub.io/422894 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: Ie252883a094bee9f47dc7b8ae987688500a77257 Gerrit-Change-Number: 422894 Gerrit-PatchSet: 1 Gerrit-Owner: Jeff Layton <jlayton(a)redhat.com>

1 0

Coverity covscan for 2018-08-18-27fd5f2c (master branch)
by staticanalysis＠gluster.org 18 Aug '18

18 Aug '18

NFS-Ganesha Coverity covscan results are available from http://download.nfs-ganesha.org/coverity/2018-08-18-27fd5f2c

1 0

Announce Push of V2.7-rc2
by Frank Filz 17 Aug '18

17 Aug '18

Branch next Tag:V2.7-rc2 Release Highlights * Several fs_locations fixes * FSAL_CEPH: add ceph_fs_ prefixes to some operations * abstract_mem: add gsh_memdup * cmake: fix cmake warning about mismatched if/endif labels * FSAL: remove obsolete comment from struct fsal_obj_handle definition * MDCACHE - Use correct DQ macro * FSAL_VFS: fix return value in vfs_open2 * FSAL_MDCACHE: avoid assertion due to wrong check * FSAL_GPFS: several fixes * Skip mounted files when populating POSIX filesystems Signed-off-by: Frank S. Filz <ffilzlnx(a)mindspring.com> Contents: 27fd5f2 Frank S. Filz V2.7-rc2 f8b30d3 Matthew Wong Skip mounted files when populating POSIX filesystems 9887179 Sachin Punadikar GPFS_FSAL:Make use of proper close routine d08cf8d Sachin Punadikar GPFS_FSAL:Print inode in failure cases b488f64 Sachin Punadikar GPFS_FSAL:Ensure fd value is more than 2 660f330 Fatih Acar FSAL_MDCACHE: avoid assertion due to wrong check 4f504ec Daniel Gryniewicz FSAL_VFS: fix return value in vfs_open2 b4a214b Daniel Gryniewicz MDCACHE - Use correct DQ macro a16e7df Jeff Layton FSAL: remove obsolete comment from struct fsal_obj_handle definition 870e1bb Jeff Layton cmake: fix cmake warning about mismatched if/endif labels 9e3a8f8 Jeff Layton FSAL: consolidate is_referral routines 20688cd Jeff Layton FSAL: make fsal_fs_locations_t.server an array of hosts 89d9819 Jeff Layton abstract_mem: add gsh_memdup a455801 Jeff Layton FSAL_CEPH: add ceph_fs_ prefixes to some operations f32d63d Jeff Layton NFS: properly handle NULL fs_locations pointer

1 0

Change in ffilz/nfs-ganesha[next]: FSAL: remove obsolete comment from struct fsal_obj_handle definition
by GerritHub 17 Aug '18

17 Aug '18

From Jeff Layton <jlayton(a)redhat.com>: Jeff Layton has uploaded this change for review. ( https://review.gerrithub.io/422713 Change subject: FSAL: remove obsolete comment from struct fsal_obj_handle definition ...................................................................... FSAL: remove obsolete comment from struct fsal_obj_handle definition Change-Id: I868b9d37e1b91999443866268c8f1da570e80e56 Signed-off-by: Jeff Layton <jlayton(a)redhat.com> --- M src/include/fsal_api.h 1 file changed, 0 insertions(+), 12 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/13/422713/1 -- To view, visit https://review.gerrithub.io/422713 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: I868b9d37e1b91999443866268c8f1da570e80e56 Gerrit-Change-Number: 422713 Gerrit-PatchSet: 1 Gerrit-Owner: Jeff Layton <jlayton(a)redhat.com>

1 0

Change in ffilz/nfs-ganesha[next]: FSAL_VFS: fix return value in vfs_open2
by GerritHub 17 Aug '18

17 Aug '18

From Daniel Gryniewicz <dang(a)redhat.com>: Daniel Gryniewicz has uploaded this change for review. ( https://review.gerrithub.io/422687 Change subject: FSAL_VFS: fix return value in vfs_open2 ...................................................................... FSAL_VFS: fix return value in vfs_open2 If set/getattr fails when handling O_EXCL, vfs_open2 still returns successfully; then it causes NULL ptr dereference when called from FSAL_MDCACHE since it is expecting a valid sub_handle pointer. Fix vfs_open2 to use an return a fsal_status_t Discovered by Fatih Acar <fatih.acar(a)gandi.net> Change-Id: Ib956ad9e8eec247a19cb8d2ed72b55362e6d53f6 Signed-off-by: Daniel Gryniewicz <dang(a)redhat.com> --- M src/FSAL/FSAL_VFS/file.c 1 file changed, 8 insertions(+), 9 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/87/422687/1 -- To view, visit https://review.gerrithub.io/422687 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: Ib956ad9e8eec247a19cb8d2ed72b55362e6d53f6 Gerrit-Change-Number: 422687 Gerrit-PatchSet: 1 Gerrit-Owner: Daniel Gryniewicz <dang(a)redhat.com>

1 0

Change in ffilz/nfs-ganesha[next]: cmake: fix cmake warning about mismatched if/endif labels
by GerritHub 17 Aug '18

17 Aug '18

From Jeff Layton <jlayton(a)redhat.com>: Jeff Layton has uploaded this change for review. ( https://review.gerrithub.io/422665 Change subject: cmake: fix cmake warning about mismatched if/endif labels ...................................................................... cmake: fix cmake warning about mismatched if/endif labels Fix this warning from cmake: CMake Warning (dev) in CMakeLists.txt: A logical block opening on the line /home/jlayton/git/ganesha/src/CMakeLists.txt:470 (if) closes on the line /home/jlayton/git/ganesha/src/CMakeLists.txt:472 (endif) with mis-matching arguments. This warning is for project developers. Use -Wno-dev to suppress it. Change-Id: I6ff4b3b95079909912b38b199f565ea08275e07f Signed-off-by: Jeff Layton <jlayton(a)redhat.com> --- M src/CMakeLists.txt 1 file changed, 1 insertion(+), 1 deletion(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/65/422665/1 -- To view, visit https://review.gerrithub.io/422665 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: I6ff4b3b95079909912b38b199f565ea08275e07f Gerrit-Change-Number: 422665 Gerrit-PatchSet: 1 Gerrit-Owner: Jeff Layton <jlayton(a)redhat.com>

1 0