Change in ffilz/nfs-ganesha[next]: Merge branch 'seclabel' into for-ffilz
by GerritHub
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>
6 years, 2 months
Announce Push of V2.7.1
by Frank Filz
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
6 years, 2 months
Problems about cache virtual glusterfs ACLs for ganesha in md-cache
by Kinglong Mee
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
6 years, 2 months
Change in ffilz/nfs-ganesha[next]: Move Nb_Worker from NFS_CORE_PARAM to _9P
by GerritHub
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>
6 years, 2 months
Change in ffilz/nfs-ganesha[next]: Drive by log messae formatting fix...
by GerritHub
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>
6 years, 2 months
Change in ffilz/nfs-ganesha[next]: CONFIG: Add CONFIG_ITEM_DEPRECATED for warning deprecated config parms
by GerritHub
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>
6 years, 2 months
Change in ffilz/nfs-ganesha[next]: selinux: add nfs-ganesha-selinux subpackage
by GerritHub
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>
6 years, 2 months
Change in ffilz/nfs-ganesha[next]: Fix uninitialized use compile error
by GerritHub
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>
6 years, 2 months
Ganesha 2.5.4 - usage of device major, minor
by Krishna Harathi
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
6 years, 2 months
Change in ffilz/nfs-ganesha[next]: rados_cluster: implement get_nodeid recovery backend op
by GerritHub
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>
6 years, 2 months