October 2018 - Devel - NFS Ganesha List Archives

Change in ffilz/nfs-ganesha[next]: Merge branch 'seclabel' into for-ffilz
by GerritHub 12 Oct '18

12 Oct '18

From Jeff Layton <jlayton(a)redhat.com>: Jeff Layton has uploaded this change for review. ( https://review.gerrithub.io/429278 Change subject: Merge branch 'seclabel' into for-ffilz ...................................................................... Merge branch 'seclabel' into for-ffilz Change-Id: Ia7715df087a5e0f9ecf1a2bb5d1de7170df235d4 Signed-off-by: Jeff Layton <jlayton(a)redhat.com> --- 1 file changed, 0 insertions(+), 0 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/78/429278/1 -- To view, visit https://review.gerrithub.io/429278 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: Ia7715df087a5e0f9ecf1a2bb5d1de7170df235d4 Gerrit-Change-Number: 429278 Gerrit-PatchSet: 1 Gerrit-Owner: Jeff Layton <jlayton(a)redhat.com>

1 0

Announce Push of V2.7.1
by Frank Filz 12 Oct '18

12 Oct '18

Branch next Tag:V2.7.1 Release Highlights * Some doc and config cleanup * two packaging/build fixes * some compiler error/warning fixes Signed-off-by: Frank S. Filz <ffilzlnx(a)mindspring.com> Contents: af26bf4 Frank S. Filz V2.7.1 a7911fc Kaleb S. KEITHLEY selinux: add nfs-ganesha-selinux subpackage c607e43 Daniel Gryniewicz Fix uninitialized use compile error 2bf45bb Kinglong Mee nfs_main: fix uninitialised log_path and dump_trace 366d045 Jeff Layton tools: make sure ganesha-rados-grace is installed if built 76955c3 Jeff Layton config_samples: update the ceph.conf example config 4a34f1b Jeff Layton doc: clean up ganesha-rados-grace document

1 0

Problems about cache virtual glusterfs ACLs for ganesha in md-cache
by Kinglong Mee 12 Oct '18

12 Oct '18

Cc nfs-ganesha, Md-cache has option "cache-posix-acl" that controls caching of posix ACLs ("system.posix_acl_access"/"system.posix_acl_default") and virtual glusterfs ACLs ("glusterfs.posix.acl"/"glusterfs.posix.default_acl") now. But, _posix_xattr_get_set does not fill virtual glusterfs ACLs when lookup requests. So, md-cache caches bad virtual glusterfs ACLs. After I turn on "cache-posix-acl" option to cache ACLs at md-cache, nfs client gets many EIO errors. https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/427305 There are two chooses for cache virtual glusterfs ACLs in md-cache, 1. Cache it separately as posix ACLs (a new option maybe "cache-glusterfs-acl" is added); And make sure _posix_xattr_get_set fills them when lookup requests. 2. Does not cache it, only cache posix ACLs; If gfapi request it, md-cache lookup according posix ACL at cache, if exist, make the virtual glusterfs ACL locally and return to gfapi; otherwise, send the request to glusterfsd. Virtual glusterfs ACLs are another format of posix ACLs, there are larger than posix ACLs, and always exist no matter the really posix ACL exist or not. So, I'd prefer #2. Any comments are welcome. thanks, Kinglong Mee

4 9

Change in ffilz/nfs-ganesha[next]: Move Nb_Worker from NFS_CORE_PARAM to _9P
by GerritHub 11 Oct '18

11 Oct '18

From Frank Filz <ffilzlnx(a)mindspring.com>: Frank Filz has uploaded this change for review. ( https://review.gerrithub.io/429066 Change subject: Move Nb_Worker from NFS_CORE_PARAM to _9P ...................................................................... Move Nb_Worker from NFS_CORE_PARAM to _9P Also fix documentation, the block is _9P not 9P. Change-Id: Ieb85fdc3760f42bd46b1653a5e5612802ab6af9c Signed-off-by: Frank S. Filz <ffilzlnx(a)mindspring.com> --- M src/MainNFSD/9p_dispatcher.c M src/MainNFSD/nfs_init.c M src/Protocols/9P/9p_read_conf.c M src/config_samples/config.txt M src/doc/man/ganesha-9p-config.rst M src/doc/man/ganesha-core-config.rst M src/include/9p.h M src/include/gsh_config.h M src/support/nfs_read_conf.c 9 files changed, 20 insertions(+), 17 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/66/429066/1 -- To view, visit https://review.gerrithub.io/429066 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: Ieb85fdc3760f42bd46b1653a5e5612802ab6af9c Gerrit-Change-Number: 429066 Gerrit-PatchSet: 1 Gerrit-Owner: Frank Filz <ffilzlnx(a)mindspring.com>

1 0

Change in ffilz/nfs-ganesha[next]: Drive by log messae formatting fix...
by GerritHub 11 Oct '18

11 Oct '18

From Frank Filz <ffilzlnx(a)mindspring.com>: Frank Filz has uploaded this change for review. ( https://review.gerrithub.io/429065 Change subject: Drive by log messae formatting fix... ...................................................................... Drive by log messae formatting fix... Change-Id: Ie137a78acc545a0587a58057f526ff9dbccc4b3a Signed-off-by: Frank S. Filz <ffilzlnx(a)mindspring.com> --- M src/support/exports.c 1 file changed, 1 insertion(+), 1 deletion(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/65/429065/1 -- To view, visit https://review.gerrithub.io/429065 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: Ie137a78acc545a0587a58057f526ff9dbccc4b3a Gerrit-Change-Number: 429065 Gerrit-PatchSet: 1 Gerrit-Owner: Frank Filz <ffilzlnx(a)mindspring.com>

1 0

Change in ffilz/nfs-ganesha[next]: CONFIG: Add CONFIG_ITEM_DEPRECATED for warning deprecated config parms
by GerritHub 11 Oct '18

11 Oct '18

From Frank Filz <ffilzlnx(a)mindspring.com>: Frank Filz has uploaded this change for review. ( https://review.gerrithub.io/429064 Change subject: CONFIG: Add CONFIG_ITEM_DEPRECATED for warning deprecated config parms ...................................................................... CONFIG: Add CONFIG_ITEM_DEPRECATED for warning deprecated config parms An optional message can be provided. This will spit out a warning in the log file, similar to an unknown config item, however it is explicitly identified as deprecated and provides the option of adding a message if a parameter has been moved or changed. Change-Id: I968929da06cd531eca5df71c754a1f8f05ab74eb Signed-off-by: Frank S. Filz <ffilzlnx(a)mindspring.com> --- M src/config_parsing/config_parsing.c M src/include/config_parsing.h 2 files changed, 35 insertions(+), 1 deletion(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/64/429064/1 -- To view, visit https://review.gerrithub.io/429064 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: I968929da06cd531eca5df71c754a1f8f05ab74eb Gerrit-Change-Number: 429064 Gerrit-PatchSet: 1 Gerrit-Owner: Frank Filz <ffilzlnx(a)mindspring.com>

1 0

Change in ffilz/nfs-ganesha[next]: selinux: add nfs-ganesha-selinux subpackage
by GerritHub 11 Oct '18

11 Oct '18

From <kaleb(a)redhat.com>: kaleb(a)redhat.com has uploaded this change for review. ( https://review.gerrithub.io/429053 Change subject: selinux: add nfs-ganesha-selinux subpackage ...................................................................... selinux: add nfs-ganesha-selinux subpackage Around October 2017 the ganesha developers negotiated with the selinux team to have ganesha in nfsd_t domain with the (explicit) understanding or belief that nfsd_t was an unconfined domain. It turns out that nfsd_t is not actually unconfined. But consider that samba runs, AIUI, in an unconfined domain. Samba and Ganesha are very similar in that they are both network file system daemons that run in user space. The purpose of this patch is to initiate the migration of ganesha selinux bits from the monolithic selinux-policy-targeted packages in Fedora and RHEL to ganesha. This first patch lifts the ganesha selinux bits (mostly*) unmodified from the RHEL 7.5 selinux package. These will eventually be modified such that ganesha runs in an unconfined, or nearly unconfined domain. For now it is enough to start the process of removing the ganesha bits from the Fedora and RHEL packages starting with Fedora 30 and RHEL 8. * logging attributes are changed from /var/log/ganesha.log and /var/log/ganesha-gfapi.log to /var/log/ganesha/*.log. RHEL is expected to catch up in 7.6. Change-Id: Ibafd259ae4107b28b1d1eca4fa6abf91b2aadd46 Signed-off-by: Kaleb S. KEITHLEY <kkeithle(a)redhat.com> --- M src/nfs-ganesha.spec-in.cmake A src/selinux/ganesha.fc A src/selinux/ganesha.if A src/selinux/ganesha.te 4 files changed, 330 insertions(+), 14 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/53/429053/1 -- To view, visit https://review.gerrithub.io/429053 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: Ibafd259ae4107b28b1d1eca4fa6abf91b2aadd46 Gerrit-Change-Number: 429053 Gerrit-PatchSet: 1 Gerrit-Owner: Anonymous Coward <kaleb(a)redhat.com>

1 0

Change in ffilz/nfs-ganesha[next]: Fix uninitialized use compile error
by GerritHub 11 Oct '18

11 Oct '18

From Daniel Gryniewicz <dang(a)redhat.com>: Daniel Gryniewicz has uploaded this change for review. ( https://review.gerrithub.io/428978 Change subject: Fix uninitialized use compile error ...................................................................... Fix uninitialized use compile error Some versions of GCC seem to tag these as uninitialized errors. Initialize is just in case. Issue #355 Change-Id: I5c29652b6656310c3648ef184d7ecccd818020ec Signed-off-by: Daniel Gryniewicz <dang(a)redhat.com> --- M src/FSAL/FSAL_PROXY/handle.c M src/Protocols/NFS/nfs_proto_tools.c 2 files changed, 2 insertions(+), 2 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/78/428978/1 -- To view, visit https://review.gerrithub.io/428978 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: I5c29652b6656310c3648ef184d7ecccd818020ec Gerrit-Change-Number: 428978 Gerrit-PatchSet: 1 Gerrit-Owner: Daniel Gryniewicz <dang(a)redhat.com>

1 0

Ganesha 2.5.4 - usage of device major, minor
by Krishna Harathi 11 Oct '18

11 Oct '18

We are using Ganesha 2.5.4 VFS FSAL with FUSE based filesystem. During our testing of deleting existing exports and creating new ones, found that if a device major and minor is reused, clients get ESTALE for accessing a newly created export (nfs2 below). This seems to cause the following log entry, and explains the ESTALE response. 04/10/2018 T15:16:59.769027-0700 : nfs-ganesha-26627[sigmgr] 1595 :claim_posix_filesystems :FSAL :INFO :Root fs for export /exports/nfs1 is /exports/nfs2 We use our own Exportid and unique FSID configured for each export in the configuration file. I would like to know more about the intent and purpose of the usage of device major and minor of an export in this context. Any help in fixing this reuse issue is also appreciated. Thanks. Regards. Krishna Harathi

2 1

Change in ffilz/nfs-ganesha[next]: rados_cluster: implement get_nodeid recovery backend op
by GerritHub 11 Oct '18

11 Oct '18

From Jeff Layton <jlayton(a)redhat.com>: Jeff Layton has uploaded this change for review. ( https://review.gerrithub.io/428919 Change subject: rados_cluster: implement get_nodeid recovery backend op ...................................................................... rados_cluster: implement get_nodeid recovery backend op Change-Id: If0374d4d9684c5d0ac8579db5c78df2086c81439 Signed-off-by: Jeff Layton <jlayton(a)redhat.com> --- M src/SAL/recovery/recovery_rados_cluster.c 1 file changed, 7 insertions(+), 0 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/19/428919/1 -- To view, visit https://review.gerrithub.io/428919 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: If0374d4d9684c5d0ac8579db5c78df2086c81439 Gerrit-Change-Number: 428919 Gerrit-PatchSet: 1 Gerrit-Owner: Jeff Layton <jlayton(a)redhat.com>

1 0