January 2020 - Devel - NFS Ganesha List Archives

Change in ...nfs-ganesha[next]: Initialize the client node for comparison
by Daniel Gryniewicz (GerritHub) 16 Jan '20

16 Jan '20

Daniel Gryniewicz has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/481894 ) Change subject: Initialize the client node for comparison ...................................................................... Initialize the client node for comparison During address cleanup, the initialization of the client node was lost, but it's still being used for comparison. Initialize it properly. Change was 9e806a7dc903aef22b92360ddfdf012b2cde5bf1 Change-Id: I24cdf1be5d7d6febe53931e8e1329a8804761291 Signed-off-by: Daniel Gryniewicz <dang(a)redhat.com> --- M src/support/client_mgr.c 1 file changed, 1 insertion(+), 0 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/94/481894/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/481894 To unsubscribe, or for help writing mail filters, visit https://review.gerrithub.io/settings Gerrit-Project: ffilz/nfs-ganesha Gerrit-Branch: next Gerrit-Change-Id: I24cdf1be5d7d6febe53931e8e1329a8804761291 Gerrit-Change-Number: 481894 Gerrit-PatchSet: 1 Gerrit-Owner: Daniel Gryniewicz <dang(a)redhat.com> Gerrit-MessageType: newchange

1 0

Default KRB Principal name with new rpc.gssd
by Pushpesh Sharma 16 Jan '20

16 Jan '20

Hi All, We are trying to use ganehsa with KRB. On NFS client centos7.5 we are joining Active Directory based using sssd(AD based KRB Realm). For root user we are getting krb ticket using kinit and valid AD user. Client principal while joining domain using sssd a default principal of shorthostname$(a)REALM.COM<mailto:shorthostname$@REALM.COM> is always generated. gss.rpcd by default send this principal as principal user name to ganesha server. Client do have other principal like nfs/client_fdqn(a)REALM.COM<mailto:nfs/client_fdqn@REALM.COM>. But as per rpc.gssd documentation<mailto:rpc.gssd%20documentation> as well first choice would be shorthostname$(a)REALM.COM<mailto:shorthostname$@REALM.COM>. Due to this server always recognize root user as someone else i.e. shorthostname$(a)REALM.COM<mailto:shorthostname$@REALM.COM>. We do see in src/idmapper/idmapper.c<https://github.com/nfs-ganesha/nfs-ganesha/blob/next/src/idmapper/idmapper.…> handling for mapping three principal patterns nfs/* , root/*, host/* to uid=0, gid=0. So this left shorthostname$(a)REALM.COM<mailto:horthostname$@REALM.COM> principal not being recognized as root. We are of the opinion that doing a small fix in above idmapper code to add this pattern as well can fix this issue. But we wanted to know any security concern around it? Or if the client behavior can be change in any way, so we don't need this fix? If we try removing this shorthostname$(a)REALM.COM<mailto:shorthostname$@REALM.COM> principal after domain join, sssd cannot be re-loaded and complains about not finding this principal. Thanks -pushpesh

1 0

Change in ...nfs-ganesha[next]: Fix nfs3_readdirplus error return
by Name of user not set (GerritHub) 16 Jan '20

16 Jan '20

wei.qiaomiao(a)zte.com.cn has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/481804 ) Change subject: Fix nfs3_readdirplus error return ...................................................................... Fix nfs3_readdirplus error return in nfs3_readdirplus function, we should use res_readdirplus3 instead of res_readdir3 Change-Id: Ibaeba7c5a6575294ceea099945a16a016ecb168a Signed-off-by: Wei Qiaomiao <wei.qiaomiao(a)zte.com.cn> --- M src/Protocols/NFS/nfs3_readdirplus.c 1 file changed, 1 insertion(+), 2 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/04/481804/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/481804 To unsubscribe, or for help writing mail filters, visit https://review.gerrithub.io/settings Gerrit-Project: ffilz/nfs-ganesha Gerrit-Branch: next Gerrit-Change-Id: Ibaeba7c5a6575294ceea099945a16a016ecb168a Gerrit-Change-Number: 481804 Gerrit-PatchSet: 1 Gerrit-Owner: wei.qiaomiao(a)zte.com.cn Gerrit-MessageType: newchange

1 0

nfs_rpc_process_request spends 150 seconds
by QR 15 Jan '20

15 Jan '20

In Windows nfs client, nfs_rpc_process_request(lookup/readdir etc) sometimes takes more than 150 seconds(v3_full). I think this is weird, because 150 seconds too long. Is anyone know what's going on during the period? Thanks in advance. Ganesha server info ganesha version: V2.7.6 FSAL : In housenfs client info nfs version : nfs v3 client info : Windows server 2012

2 1

Change in ...nfs-ganesha[next]: Remove unused DUPREQ_DELETED and timestamp
by Malahal (GerritHub) 13 Jan '20

13 Jan '20

Malahal has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/481400 ) Change subject: Remove unused DUPREQ_DELETED and timestamp ...................................................................... Remove unused DUPREQ_DELETED and timestamp Change-Id: Ied7184a23011b6c647a67f76788f6dc01194f337 Signed-off-by: Malahal Naineni <malahal(a)us.ibm.com> --- M src/RPCAL/nfs_dupreq.c M src/include/nfs_dupreq.h 2 files changed, 2 insertions(+), 6 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/00/481400/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/481400 To unsubscribe, or for help writing mail filters, visit https://review.gerrithub.io/settings Gerrit-Project: ffilz/nfs-ganesha Gerrit-Branch: next Gerrit-Change-Id: Ied7184a23011b6c647a67f76788f6dc01194f337 Gerrit-Change-Number: 481400 Gerrit-PatchSet: 1 Gerrit-Owner: Malahal <malahal(a)gmail.com> Gerrit-MessageType: newchange

1 0

Change in ...nfs-ganesha[next]: Fix use of dupreqs with DUPREQ_DELETED state
by Malahal (GerritHub) 13 Jan '20

13 Jan '20

Malahal has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/481399 ) Change subject: Fix use of dupreqs with DUPREQ_DELETED state ...................................................................... Fix use of dupreqs with DUPREQ_DELETED state DUPREQ_DELETED is set just before deleting the dupreq in nfs_dupreq_delete(). This is called on requests that get dropped! So these dupreqs should not be used but nfs_dupreq_start() ends up using it as its state is not DUPREQ_START. No need to set DUPREQ_DELETED as the dupreq is getting deleted anyway. Change-Id: Iecd82fae45400d30cea31b4afb1995a0c2d795c0 Signed-off-by: Malahal Naineni <malahal(a)us.ibm.com> --- M src/RPCAL/nfs_dupreq.c 1 file changed, 0 insertions(+), 3 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/99/481399/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/481399 To unsubscribe, or for help writing mail filters, visit https://review.gerrithub.io/settings Gerrit-Project: ffilz/nfs-ganesha Gerrit-Branch: next Gerrit-Change-Id: Iecd82fae45400d30cea31b4afb1995a0c2d795c0 Gerrit-Change-Number: 481399 Gerrit-PatchSet: 1 Gerrit-Owner: Malahal <malahal(a)gmail.com> Gerrit-MessageType: newchange

1 0

Change in ...nfs-ganesha[next]: DNS related stats are added to ganesha_stats auth command
by Trishali Nayar (GerritHub) 11 Jan '20

11 Jan '20

Trishali Nayar has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/480042 ) Change subject: DNS related stats are added to ganesha_stats auth command ...................................................................... DNS related stats are added to ganesha_stats auth command Change-Id: I4e3228e1d9dfb1514040a00d2a55f69ba4f811dd Signed-off-by: Trishali Nayar <ntrishal(a)in.ibm.com> --- M src/Protocols/NFS/nfs4_op_exchange_id.c M src/Protocols/NLM/nlm_async.c M src/RPCAL/gss_credcache.c M src/SAL/nfs4_recovery.c M src/idmapper/idmapper.c M src/include/common_utils.h M src/include/idmapper.h M src/include/server_stats_private.h M src/scripts/ganeshactl/Ganesha/glib_dbus_stats.py M src/support/exports.c M src/support/nfs_ip_name.c 11 files changed, 133 insertions(+), 11 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/42/480042/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/480042 To unsubscribe, or for help writing mail filters, visit https://review.gerrithub.io/settings Gerrit-Project: ffilz/nfs-ganesha Gerrit-Branch: next Gerrit-Change-Id: I4e3228e1d9dfb1514040a00d2a55f69ba4f811dd Gerrit-Change-Number: 480042 Gerrit-PatchSet: 1 Gerrit-Owner: Trishali Nayar <ntrishal(a)in.ibm.com> Gerrit-MessageType: newchange

1 0

Announce Push of V4-dev.1
by Frank Filz 10 Jan '20

10 Jan '20

Branch next Tag:V4-dev.1 Merge Highlights * Support for POSIX ACL xattrs for CEPH * Remove unused DUPREQ_ERROR and DUPREQ_INSERT_MALLOC_ERROR * Fix responding with NFS version mismatch. * Fix get_state_obj_export_owner_refs() fail case * Fix accessing object handle after freeing its last state * core: multiple definitions of 'common' data compiling with gcc-10 * CEPH: Fix size of handle * Fix double couting of READ, WRITE op * Fix typo in lib_dbus_stats.py * Throttle the health warning Signed-off-by: Frank S. Filz <ffilzlnx(a)mindspring.com> Contents: 9903c67 Frank S. Filz V4-dev.1 91f0302 Daniel Gryniewicz Throttle the health warning db67193 Sachin Punadikar Fix typo in lib_dbus_stats.py 16b6c6b Sachin Punadikar Fix double couting of READ, WRITE ops 4bfdb31 Wei Qiaomiao CEPH: Fix size of handle fc62d0d Kaleb S. KEITHLEY core: multiple definitions of 'common' data compiling with gcc-10 2a9d219 Shangzhong Zhu Enable ACL disable option for FSAL_CEPH abbe630 Shangzhong Zhu Add get and set ACLs attr support for FSAL_CEPH d354583 Shangzhong Zhu Add get and set ACLs API for FSAL_CEPH fe895ee Shangzhong Zhu Convert POSIX ACL to extented attribute a3f51c0 Malahal Naineni Fix accessing object handle after freeing its last state 42afe22 Malahal Naineni Fix get_state_obj_export_owner_refs() fail case fe8cebb Malahal Naineni Fix responding with NFS version mismatch. 1c3789b Malahal Naineni Remove unused DUPREQ_ERROR and DUPREQ_INSERT_MALLOC_ERROR

1 0

Change in ...nfs-ganesha[next]: Throttle the health warning
by Daniel Gryniewicz (GerritHub) 10 Jan '20

10 Jan '20

Daniel Gryniewicz has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/479965 ) Change subject: Throttle the health warning ...................................................................... Throttle the health warning If there is a period of low traffic from clients, then the heartbeat can fire just after the first op in a while. This causes a suprious health warn message in the logs. Raise the threshold for health warn to 2, rather than 1, to avoid these messages. Change-Id: Id3ade7b919290a71409f0f668735a3fc1ca89e5d Signed-off-by: Daniel Gryniewicz <dang(a)redhat.com> --- M src/MainNFSD/nfs_init.c 1 file changed, 5 insertions(+), 2 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/65/479965/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/479965 To unsubscribe, or for help writing mail filters, visit https://review.gerrithub.io/settings Gerrit-Project: ffilz/nfs-ganesha Gerrit-Branch: next Gerrit-Change-Id: Id3ade7b919290a71409f0f668735a3fc1ca89e5d Gerrit-Change-Number: 479965 Gerrit-PatchSet: 1 Gerrit-Owner: Daniel Gryniewicz <dang(a)redhat.com> Gerrit-MessageType: newchange

1 0

Change in ...nfs-ganesha[next]: Fix typo
by Sachin Punadikar (GerritHub) 09 Jan '20

09 Jan '20

Sachin Punadikar has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/479775 ) Change subject: Fix typo ...................................................................... Fix typo "compound" was mis-spelled as "compond". Fixed it. Change-Id: I4c49e48eefb353f5ce8f8cfde1322c3fca9c82a7 Signed-off-by: Sachin Punadikar <psachin(a)in.ibm.com> --- M src/scripts/ganeshactl/Ganesha/glib_dbus_stats.py 1 file changed, 1 insertion(+), 1 deletion(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/75/479775/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/479775 To unsubscribe, or for help writing mail filters, visit https://review.gerrithub.io/settings Gerrit-Project: ffilz/nfs-ganesha Gerrit-Branch: next Gerrit-Change-Id: I4c49e48eefb353f5ce8f8cfde1322c3fca9c82a7 Gerrit-Change-Number: 479775 Gerrit-PatchSet: 1 Gerrit-Owner: Sachin Punadikar <psachin(a)in.ibm.com> Gerrit-MessageType: newchange

1 0