June 2026 - Devel - NFS Ganesha List Archives

[S] Change in ...nfs-ganesha[next]: FSAL_GPFS: fix pynfs SATT12x failure for oversized SETATTR size
by Name of user not set (GerritHub) 15 Jun '26

15 Jun '26

Vidya.Thumukunta(a)ibm.com has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1239136?usp=email ) Change subject: FSAL_GPFS: fix pynfs SATT12x failure for oversized SETATTR size ...................................................................... FSAL_GPFS: fix pynfs SATT12x failure for oversized SETATTR size Return NFS4ERR_FBIG for sizes not representable as off_t instead of returning EINVAL from GPFS, aligning behavior with VFS FSAL. Change-Id: Icf45b6544794a7f1f3b2994565cab78a552c1a58 Signed-off-by: VidyaThumukunta <vidya.thumukunta(a)ibm.com> --- M src/FSAL/FSAL_GPFS/fsal_attrs.c 1 file changed, 14 insertions(+), 0 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/36/1239136/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1239136?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: Icf45b6544794a7f1f3b2994565cab78a552c1a58 Gerrit-Change-Number: 1239136 Gerrit-PatchSet: 1 Gerrit-Owner: Vidya.Thumukunta(a)ibm.com

1 0

[M] Change in ...nfs-ganesha[next]: recovery: add RecoverySkipIp to omit IP from tag
by Moritz Tanner (GerritHub) 15 Jun '26

15 Jun '26

Moritz Tanner has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1239050?usp=email ) Change subject: recovery: add RecoverySkipIp to omit IP from tag ...................................................................... recovery: add RecoverySkipIp to omit IP from tag In some HA configurations a client may be seen by the server under a different source IP after a failover (e.g. due to NAT or overlay networking). This causes grace-period reclaim to fail because the stored recovery tag embeds the client's source IP and no longer matches. Add an NFSv4 config boolean RecoverySkipIp (default false) that makes nfs4_create_clid_name() build the tag from only the opaque client owner, without the address prefix. The tag changes from "<addr>-(len:opaque)" to "(len:opaque)". Both add and check build the tag the same way, so check_clid() needs no change. Existing records written with the address prefix will not match after enabling the option; clients re-establish fresh state on the first restart. Change-Id: Ic33a23f4f4bb3d3a1dd82e1669ca47366290b213 Signed-off-by: Moritz Tanner <moritz.tanner(a)linbit.com> --- M src/SAL/nfs4_recovery.c M src/include/gsh_config.h M src/support/nfs_read_conf.c M src/test/CMakeLists.txt A src/test/test_recovery_tag.c 5 files changed, 118 insertions(+), 5 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/50/1239050/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1239050?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: Ic33a23f4f4bb3d3a1dd82e1669ca47366290b213 Gerrit-Change-Number: 1239050 Gerrit-PatchSet: 1 Gerrit-Owner: Moritz Tanner <moritz.tanner(a)linbit.com>

1 0

[M] Change in ...nfs-ganesha[next]: recovery: extract and test nfs4_recovery_build_tag()
by Moritz Tanner (GerritHub) 15 Jun '26

15 Jun '26

Moritz Tanner has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1239044?usp=email ) Change subject: recovery: extract and test nfs4_recovery_build_tag() ...................................................................... recovery: extract and test nfs4_recovery_build_tag() The tag-assembly logic (IP prefix + formatted opaque client owner) was duplicated between recovery_fs.c and recovery_rados_kv.c. Extract it into a shared nfs4_recovery_build_tag() in nfs4_recovery.c and have both backends call it. No change in behavior. Add src/test/test_recovery_tag.c covering the tag format for several opaque value and address shapes. Change-Id: I047ade3bc74cae6e3ab9443c5eeefa5e73de26d4 Signed-off-by: Moritz Tanner <moritz.tanner(a)linbit.com> --- M src/MainNFSD/libganesha_nfsd.ver M src/SAL/nfs4_recovery.c M src/SAL/recovery/recovery_fs.c M src/SAL/recovery/recovery_rados_kv.c M src/include/sal_functions.h M src/test/CMakeLists.txt A src/test/test_recovery_tag.c 7 files changed, 116 insertions(+), 67 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/44/1239044/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1239044?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: I047ade3bc74cae6e3ab9443c5eeefa5e73de26d4 Gerrit-Change-Number: 1239044 Gerrit-PatchSet: 1 Gerrit-Owner: Moritz Tanner <moritz.tanner(a)linbit.com>

1 0

[S] Change in ...nfs-ganesha[next]: fscrypt kmip module - build with obsolete openssl
by Name of user not set (GerritHub) 12 Jun '26

12 Jun '26

mwatts(a)redhat.com has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1238979?usp=email ) Change subject: fscrypt kmip module - build with obsolete openssl ...................................................................... fscrypt kmip module - build with obsolete openssl X509_add_cert() was added as of OpenSSL 3.0.0. You should not be using any version older than this. With this fix, this code will build on fedora 34 (openssl-devel-1.1.1n-1.fc34.x86_64), which does not provide X509_add_cert(). That does not mean this code works or should be used. Change-Id: Ibae50f13c042936cd31a05974fb877e04b0fc964 Signed-off-by: Marcus Watts <mwatts(a)redhat.com> --- M src/PLUGIN/kmip/kmip_fscrypt.c 1 file changed, 19 insertions(+), 0 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/79/1238979/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1238979?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: Ibae50f13c042936cd31a05974fb877e04b0fc964 Gerrit-Change-Number: 1238979 Gerrit-PatchSet: 1 Gerrit-Owner: mwatts(a)redhat.com

1 0

Announce Push of V9.16
by Frank Filz 12 Jun '26

12 Jun '26

Branch next Tag:V9.16 NOTE: This merge includes and ntirpc pullup - please update your submodule. Merge Highlights * Fix buffer overflow and missing null-termination in name2id * Optimisation to avoid same-client lock-path delegation recall in FSAL_UP * Add xattr support to FSAL-GPFS * Replace assert with a check for empty client record value. * Replace assert with a check for memory availability in readdir. * Fix: Improve type safety and resolve missing header dependency * Fix for NLM CLOSE_WAIT Socket Leak * Enable callback channel setup for RDMA (with ntirpc pullup) Signed-off-by: Frank S. Filz <ffilzlnx(a)mindspring.com> Contents: 6513f2631 Frank S. Filz V9.16 ebab1ec2e Rajesh Prasad Enable callback channel setup for RDMA 4a7cdbb27 Rajesh Prasad Fix for NLM CLOSE_WAIT Socket Leak fbff8385a Sreedhar Agraharam Fix: Improve type safety and resolve missing header dependency 20de4beb3 Ofir Vainshtein Replace assert with a check for memory availability in readdir. 654446681 Ofir Vainshtein Replace assert with a check for empty client record value. a89089c7f marc eshel Add xattr support to FSAL-GPFS 5c0c32e0d Suhas Athani Optimisation to avoid same-client lock-path delegation recall in FSAL_UP 6c076b4ea Lior Suliman Fix buffer overflow and missing null-termination in name2id

1 0

[XL] Change in ...nfs-ganesha[next]: Monitoring: Update in Grafana dashboard JSON usage file
by Name of user not set (GerritHub) 11 Jun '26

11 Jun '26

sragraha(a)redhat.com has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1238868?usp=email ) Change subject: Monitoring: Update in Grafana dashboard JSON usage file ...................................................................... Monitoring: Update in Grafana dashboard JSON usage file Change-Id: Ia67e22cf732e07c48718e08cf01809695af27a5e Signed-off-by: Sreedhar Agraharam <sragraha(a)redhat.com> --- M src/config_samples/grafana.dashboard.json 1 file changed, 2,116 insertions(+), 2,522 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/68/1238868/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1238868?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: Ia67e22cf732e07c48718e08cf01809695af27a5e Gerrit-Change-Number: 1238868 Gerrit-PatchSet: 1 Gerrit-Owner: sragraha(a)redhat.com

1 0

[L] Change in ...nfs-ganesha[next]: ganeshactl/log: add composite conditional logging management commands
by Nishant Puri (GerritHub) 10 Jun '26

10 Jun '26

Nishant Puri has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1238847?usp=email ) Change subject: ganeshactl/log: add composite conditional logging management commands ...................................................................... ganeshactl/log: add composite conditional logging management commands Add three new commands to ganesha_mgr for managing conditional logging configuration in a single operation: ganesha_mgr set log conditional_config [OPTIONS] Configure multiple aspects of conditional logging in one call. Supports repeatable --components/--level pairs to set different log levels per component group, --clients (IP/CIDR), --export-ids, and --policy (ANY|ALL|MATCH_ANY|MATCH_ALL). All flags are optional but at least one must be provided. Processing order is fixed regardless of flag order on the command line: component levels, then clients, then exports, then policy. Errors are collected and reported at the end (continue-and-collect semantics). ganesha_mgr show log conditional_config Display the full conditional logging configuration in a single view: client list, export list, match policy, and all component log levels. ganesha_mgr reset log conditional_config [OPTIONS] With no options, atomically resets all conditional logging state to defaults via a new ResetConditionalLogging D-Bus method: client and export lists are cleared, all component log levels are reset to NIV_FULL_DEBUG, and the match policy is reset to MATCH_ANY. With options (--clients, --export-ids, --components, --policy), selectively removes or resets only the specified items using the existing individual D-Bus methods. Add ResetConditionalLogging D-Bus method to log_conditional_interface to support the atomic full-reset path. Add validation to ChangeMatchPolicy: setting MATCH_ALL is rejected when either the client list or the export list is empty, since the AND condition can never be satisfied in that state and conditional logging would silently never fire. The rejection is reported back to the caller via the reply status and logged at EVENT level. All existing commands are preserved unchanged. Fix cond_log_props[] and HANDLE_COND_LOG_PROP to include COMPONENT_QOS, COMPONENT_RECOVERY, and COMPONENT_RDMA, which were missing from the conditional logging property table while present in the normal log component table. Change-Id: Ie8ccda20d85fa58dfe41ab835e8c66579cf71be5 Signed-off-by: Nishant Puri <npuri(a)redhat.com> --- M src/log/log_functions.c M src/scripts/ganeshactl/Ganesha/ganesha_mgr_utils.py M src/scripts/ganeshactl/ganesha_mgr.py 3 files changed, 484 insertions(+), 2 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/47/1238847/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1238847?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: Ie8ccda20d85fa58dfe41ab835e8c66579cf71be5 Gerrit-Change-Number: 1238847 Gerrit-PatchSet: 1 Gerrit-Owner: Nishant Puri <npuri(a)redhat.com>

1 0

[S] Change in ...nfs-ganesha[next]: Enable callback channel setup for RDMA
by Rajesh Prasad (GerritHub) 10 Jun '26

10 Jun '26

Rajesh Prasad has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1238808?usp=email ) Change subject: Enable callback channel setup for RDMA ...................................................................... Enable callback channel setup for RDMA Incoporated the review comments of PR - https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1222344 Addressed the conditional compilation issues for the RDMA. Tirpc PR - https://github.com/nfs-ganesha/ntirpc/pull/396 Change-Id: Iedb96b960b9bb0050f714637ef6e12ec6f2b965d Signed-off-by: Rajesh Prasad <raprasad(a)redhat.com> --- M src/MainNFSD/nfs_rpc_callback.c 1 file changed, 18 insertions(+), 9 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/08/1238808/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1238808?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: Iedb96b960b9bb0050f714637ef6e12ec6f2b965d Gerrit-Change-Number: 1238808 Gerrit-PatchSet: 1 Gerrit-Owner: Rajesh Prasad <raprasad(a)redhat.com>

1 0

[S] Change in ...nfs-ganesha[next]: Fix: Improve type safety and resolve missing header dependency
by Name of user not set (GerritHub) 09 Jun '26

09 Jun '26

sragraha(a)redhat.com has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1238798?usp=email ) Change subject: Fix: Improve type safety and resolve missing header dependency ...................................................................... Fix: Improve type safety and resolve missing header dependency Change-Id: I2fb00a675f44a2ef83ebd07ccb8bb5bf4ed87c65 Signed-off-by: Sreedhar Agraharam <sragraha(a)redhat.com> --- M src/include/common_utils.h M src/include/nfsv41.h M src/monitoring/prometheus_exposer.cc 3 files changed, 10 insertions(+), 9 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/98/1238798/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1238798?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: I2fb00a675f44a2ef83ebd07ccb8bb5bf4ed87c65 Gerrit-Change-Number: 1238798 Gerrit-PatchSet: 1 Gerrit-Owner: sragraha(a)redhat.com

1 0

[M] Change in ...nfs-ganesha[next]: Add xattr support to FSAL_GPFS
by Name of user not set (GerritHub) 09 Jun '26

09 Jun '26

eshel(a)us.ibm.com has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1238795?usp=email ) Change subject: Add xattr support to FSAL_GPFS ...................................................................... Add xattr support to FSAL_GPFS Change-Id: I827e9e4289c15576ff7c823516db311c9ab77a77 Signed-off-by: marc eshel <eshel(a)us.ibm.com> --- M src/FSAL/FSAL_GPFS/fsal_internal.h M src/FSAL/FSAL_GPFS/handle.c M src/FSAL/FSAL_GPFS/main.c 3 files changed, 27 insertions(+), 72 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/95/1238795/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1238795?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: I827e9e4289c15576ff7c823516db311c9ab77a77 Gerrit-Change-Number: 1238795 Gerrit-PatchSet: 1 Gerrit-Owner: eshel(a)us.ibm.com

1 0