May 2025 - Devel - Nfs Ganesha List Archives

[S] Change in ...nfs-ganesha[next]: ganesha-rados-grace : support non-numeric nodeid

by Sachin Punadikar (GerritHub)

Sachin Punadikar has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1215719?usp=email ) Change subject: ganesha-rados-grace : support non-numeric nodeid ...................................................................... ganesha-rados-grace : support non-numeric nodeid To support backward compatibility, wherein old nodeid (which are non-numeric strings) is required to be removed from rados object storing grace membership, need to allow and process non-numeric nodeid. The tool checks if passed in nodeid is numeric or non-numeric, and then process it correctly. Change-Id: I414ce8c6976e284ac58ecd5acec284af8c269829 Signed-off-by: Sachin Punadikar <sachin.punadikar(a)ibm.com> --- M src/tools/ganesha-rados-grace.c 1 file changed, 27 insertions(+), 3 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/19/1215719/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1215719?usp=email To unsubscribe, or for help writing mail filters, visit https://review.gerrithub.io/settings?usp=email Gerrit-MessageType: newchange Gerrit-Project: ffilz/nfs-ganesha Gerrit-Branch: next Gerrit-Change-Id: I414ce8c6976e284ac58ecd5acec284af8c269829 Gerrit-Change-Number: 1215719 Gerrit-PatchSet: 1 Gerrit-Owner: Sachin Punadikar <sachin.punadikar(a)ibm.com>

1 day, 5 hours

1
0
0 / 0

[M] Change in ...nfs-ganesha[next]: NFS-Ganesha: Add SSSD timeout and skip cache options

by Lior Suliman (GerritHub)

Attention is currently required from: Frank Filz. Hello Frank Filz, I'd like you to do a code review. Please visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1215676?usp=email to review the following change. Change subject: NFS-Ganesha: Add SSSD timeout and skip cache options ...................................................................... NFS-Ganesha: Add SSSD timeout and skip cache options This commit adds 2 new configurations: - Sssd_Implementation_Timeout: The timeout to use for each direct to sssd idmapping operations. Default 30 seconds. - Sssd_Implementation_Skip_Cache: Whether to skip SSSD cache when performing SSSD direct idmapping operations. Default false. Both configurations are effective only when using SSSD pwnam implementation. This is needed to allow clients to use the SSSD pwnam implementation while preventing clients' request from being blocked by potential SSSD issues. Change-Id: I62d1d86db5efb5582d8b87b8913599d8e065045d Signed-off-by: Lior Suliman <liorsu(a)gmail.com> --- M src/doc/man/ganesha-core-config.rst M src/idmapper/sss_nss_idmap.c M src/include/gsh_config.h M src/support/nfs_read_conf.c 4 files changed, 42 insertions(+), 12 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/76/1215676/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1215676?usp=email To unsubscribe, or for help writing mail filters, visit https://review.gerrithub.io/settings?usp=email Gerrit-MessageType: newchange Gerrit-Project: ffilz/nfs-ganesha Gerrit-Branch: next Gerrit-Change-Id: I62d1d86db5efb5582d8b87b8913599d8e065045d Gerrit-Change-Number: 1215676 Gerrit-PatchSet: 1 Gerrit-Owner: Lior Suliman <liorsu(a)google.com> Gerrit-Reviewer: Frank Filz <ffilzlnx(a)mindspring.com> Gerrit-Attention: Frank Filz <ffilzlnx(a)mindspring.com>

1 day, 9 hours

1
0
0 / 0

[S] Change in ...nfs-ganesha[next]: Use idmapper_negative_cache for uid when fetching groups.

by Lior Suliman (GerritHub)

Attention is currently required from: Frank Filz. Hello Frank Filz, I'd like you to do a code review. Please visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1215675?usp=email to review the following change. Change subject: Use idmapper_negative_cache for uid when fetching groups. ...................................................................... Use idmapper_negative_cache for uid when fetching groups. If fetching groups for uid failed, meaning uid does not exist, we would previously loop trying to get the groups for that uid on every request. This commit implements the negative cache, to avoid the loop. Added uid negative lookup, and save into the cache if not found. Change-Id: Ia6ce0d4fb513d4464e0d837956a540d2962b42ee Signed-off-by: Lior Suliman <liorsu(a)gmail.com> --- M src/support/uid2grp.c 1 file changed, 37 insertions(+), 1 deletion(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/75/1215675/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1215675?usp=email To unsubscribe, or for help writing mail filters, visit https://review.gerrithub.io/settings?usp=email Gerrit-MessageType: newchange Gerrit-Project: ffilz/nfs-ganesha Gerrit-Branch: next Gerrit-Change-Id: Ia6ce0d4fb513d4464e0d837956a540d2962b42ee Gerrit-Change-Number: 1215675 Gerrit-PatchSet: 1 Gerrit-Owner: Lior Suliman <liorsu(a)google.com> Gerrit-Reviewer: Frank Filz <ffilzlnx(a)mindspring.com> Gerrit-Attention: Frank Filz <ffilzlnx(a)mindspring.com>

1 day, 9 hours

1
0
0 / 0

[M] Change in ...nfs-ganesha[next]: NFS-Ganesha: Add negative-cache by UIDs

by Lior Suliman (GerritHub)

Attention is currently required from: Frank Filz. Hello Frank Filz, I'd like you to do a code review. Please visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1215674?usp=email to review the following change. Change subject: NFS-Ganesha: Add negative-cache by UIDs ...................................................................... NFS-Ganesha: Add negative-cache by UIDs We need to be able to store the negative answers from idmapper when mapping a uid. This is done by introducing a new negative-cache type. Change-Id: I270a7b1b64747d0cfca5513d4ed6e1649ff1038f Signed-off-by: Lior Suliman <liorsu(a)gmail.com> --- M src/FSAL/localfs.c M src/idmapper/idmapper_monitoring.c M src/idmapper/idmapper_negative_cache.c M src/include/idmapper.h M src/include/idmapper_monitoring.h 5 files changed, 221 insertions(+), 22 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/74/1215674/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1215674?usp=email To unsubscribe, or for help writing mail filters, visit https://review.gerrithub.io/settings?usp=email Gerrit-MessageType: newchange Gerrit-Project: ffilz/nfs-ganesha Gerrit-Branch: next Gerrit-Change-Id: I270a7b1b64747d0cfca5513d4ed6e1649ff1038f Gerrit-Change-Number: 1215674 Gerrit-PatchSet: 1 Gerrit-Owner: Lior Suliman <liorsu(a)google.com> Gerrit-Reviewer: Frank Filz <ffilzlnx(a)mindspring.com> Gerrit-Attention: Frank Filz <ffilzlnx(a)mindspring.com>

1 day, 9 hours

1
0
0 / 0

[S] Change in ...nfs-ganesha[next]: Add config option to select pwnam implementation

by Lior Suliman (GerritHub)

Attention is currently required from: Frank Filz. Hello Frank Filz, I'd like you to do a code review. Please visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1215673?usp=email to review the following change. Change subject: Add config option to select pwnam implementation ...................................................................... Add config option to select pwnam implementation Adds a config to select between the two available implementations for the `pwnam` wrapper: `nsswitch` and `sssd`. This is needed to enable support of ID mapping when using sssd without nsswitch. Change-Id: Ia1fee858d1eef424c559dcaf4f95d6a607159a25 Signed-off-by: Lior Suliman <liorsu(a)gmail.com> --- M src/doc/man/ganesha-core-config.rst M src/idmapper/idmapper.c M src/include/gsh_config.h M src/support/nfs_read_conf.c 4 files changed, 21 insertions(+), 0 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/73/1215673/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1215673?usp=email To unsubscribe, or for help writing mail filters, visit https://review.gerrithub.io/settings?usp=email Gerrit-MessageType: newchange Gerrit-Project: ffilz/nfs-ganesha Gerrit-Branch: next Gerrit-Change-Id: Ia1fee858d1eef424c559dcaf4f95d6a607159a25 Gerrit-Change-Number: 1215673 Gerrit-PatchSet: 1 Gerrit-Owner: Lior Suliman <liorsu(a)google.com> Gerrit-Reviewer: Frank Filz <ffilzlnx(a)mindspring.com> Gerrit-Attention: Frank Filz <ffilzlnx(a)mindspring.com>

1 day, 9 hours

1
0
0 / 0

[XS] Change in ...nfs-ganesha[next]: Minor fix in tracepoint formatting in nfs4_op_sequence.c

by Lior Suliman (GerritHub)

Attention is currently required from: Frank Filz. Hello Frank Filz, I'd like you to do a code review. Please visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1215672?usp=email to review the following change. Change subject: Minor fix in tracepoint formatting in nfs4_op_sequence.c ...................................................................... Minor fix in tracepoint formatting in nfs4_op_sequence.c The names used in the format strings are too long if we include the large base file name. which creates a format string that is larger than 256 chars and gets truncated. Change-Id: I3d8deb7c0d1ba626b367464b1d8cca8a696faf9f Signed-off-by: Lior Suliman <liorsu(a)gmail.com> --- M src/Protocols/NFS/nfs4_op_sequence.c 1 file changed, 2 insertions(+), 2 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/72/1215672/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1215672?usp=email To unsubscribe, or for help writing mail filters, visit https://review.gerrithub.io/settings?usp=email Gerrit-MessageType: newchange Gerrit-Project: ffilz/nfs-ganesha Gerrit-Branch: next Gerrit-Change-Id: I3d8deb7c0d1ba626b367464b1d8cca8a696faf9f Gerrit-Change-Number: 1215672 Gerrit-PatchSet: 1 Gerrit-Owner: Lior Suliman <liorsu(a)google.com> Gerrit-Reviewer: Frank Filz <ffilzlnx(a)mindspring.com> Gerrit-Attention: Frank Filz <ffilzlnx(a)mindspring.com>

1 day, 9 hours

1
0
0 / 0

[XS] Change in ...nfs-ganesha[next]: Coverity 502601: Fix null dereference in nfsacl_setacl() Avoid passin...

by Name of user not set (GerritHub)

Vidya.Thumukunta(a)ibm.com has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1215644?usp=email ) Change subject: Coverity 502601: Fix null dereference in nfsacl_setacl() Avoid passing NULL acl_access to nfs3_acl_2_fsal_acl(). ...................................................................... Coverity 502601: Fix null dereference in nfsacl_setacl() Avoid passing NULL acl_access to nfs3_acl_2_fsal_acl(). Change-Id: I1a5d69ca36455170747e0eaa14226dbcbd0b6074 Signed-off-by: VidyaThumukunta <Vidya.Thumukunta(a)ibm.com> --- M src/Protocols/NFSACL/nfsacl_setacl.c 1 file changed, 3 insertions(+), 2 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/44/1215644/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1215644?usp=email To unsubscribe, or for help writing mail filters, visit https://review.gerrithub.io/settings?usp=email Gerrit-MessageType: newchange Gerrit-Project: ffilz/nfs-ganesha Gerrit-Branch: next Gerrit-Change-Id: I1a5d69ca36455170747e0eaa14226dbcbd0b6074 Gerrit-Change-Number: 1215644 Gerrit-PatchSet: 1 Gerrit-Owner: Vidya.Thumukunta(a)ibm.com

1 day, 14 hours

1
0
0 / 0

We have a new community chat to replace IRC

by Frank Filz

We have a new community chat set up. You may be able to get directly to it with https://app.element.io/#/room/#devel:nfs-ganesha.org or sign on to matrix.org. Channels are #devel:nfs-ganesha.org, #support, and #ntirpc Thanks Frank

4 days

1
0
0 / 0

[S] Change in ...nfs-ganesha[next]: send tlv_headers to connection register

by Name of user not set (GerritHub)

roybabayov(a)gmail.com has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1215456?usp=email ) Change subject: send tlv_headers to connection register ...................................................................... send tlv_headers to connection register The tlv headers can contain extra identification of the client that might be required for the FSAL to register the connection. For example, in Google cloud, the tlv contain the pscConnectionId that identifies the client network - https://cloud.google.com/vpc/docs/about-vpc-hosted-services#proxy-protocol Change-Id: Iff0119221344610ebc829c859d02bb5cbfe56642 Signed-off-by: Roy Babayov <roybabayov(a)google.com> --- M src/RPCAL/connection_manager.c M src/include/connection_manager.h 2 files changed, 19 insertions(+), 7 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/56/1215456/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1215456?usp=email To unsubscribe, or for help writing mail filters, visit https://review.gerrithub.io/settings?usp=email Gerrit-MessageType: newchange Gerrit-Project: ffilz/nfs-ganesha Gerrit-Branch: next Gerrit-Change-Id: Iff0119221344610ebc829c859d02bb5cbfe56642 Gerrit-Change-Number: 1215456 Gerrit-PatchSet: 1 Gerrit-Owner: roybabayov(a)gmail.com

4 days, 13 hours

1
0
0 / 0

[M] Change in ...nfs-ganesha[next]: scripts: Clean up ganesha_mgr help

by Martin Schwenke (GerritHub)

Martin Schwenke has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1215437?usp=email ) Change subject: scripts: Clean up ganesha_mgr help ...................................................................... scripts: Clean up ganesha_mgr help Use a multiline f-string to neaten the usage message. Add a couple of functions to print help messages and exit. Use them to reduce duplication and increase consistency of both the content of error messages and the code that produces them. Consistently print the name of the actual script being run. Signed-off-by: Martin Schwenke <mschwenke(a)ddn.com> Change-Id: I9da99cf52e37e496f45578c2a18cb1226a066ae2 --- M src/scripts/ganeshactl/ganesha_mgr.py 1 file changed, 95 insertions(+), 124 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/37/1215437/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1215437?usp=email To unsubscribe, or for help writing mail filters, visit https://review.gerrithub.io/settings?usp=email Gerrit-MessageType: newchange Gerrit-Project: ffilz/nfs-ganesha Gerrit-Branch: next Gerrit-Change-Id: I9da99cf52e37e496f45578c2a18cb1226a066ae2 Gerrit-Change-Number: 1215437 Gerrit-PatchSet: 1 Gerrit-Owner: Martin Schwenke <martin(a)meltin.net>

5 days, 16 hours

1
0
0 / 0

2025

2024

2023

2022

2021

2020

2019

2018

Devel May 2025