July 2026 - Devel - Nfs Ganesha List Archives

[M] Change in ...nfs-ganesha[next]: log: reset conditional logging state on config reload

by Nishant Puri (GerritHub)

Nishant Puri has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1241843?usp=email ) Change subject: log: reset conditional logging state on config reload ...................................................................... log: reset conditional logging state on config reload On SIGHUP/reload, global_client_ip_list and global_export_id_list were never cleared before re-parsing the LOG { Conditional {} } block. This caused two bugs: stale client/export entries persisted if removed from config, and reloading with unchanged entries triggered duplicate errors that aborted the entire reload. Extract reset_conditional_logging_state() to clear both global lists, reset all conditional component log levels to NIV_FULL_DEBUG, reset match policy to COND_LOG_MATCH_ANY, and set conditional_logging_configured to false. Call it at the start of read_log_config() so every reload starts from a clean slate and applies only what is present in the current config file. Simplify dbus_conditional_log_reset() to reuse the same helper instead of duplicating the logic. Also add missing CONF_INDEX_TOKEN entries for COMPONENT_RDMA and COMPONENT_GRPC in conditional_params[] to prevent them from being incorrectly forced to NIV_NULL when not explicitly configured. Change-Id: I4b1c2ca0cc3cefca556839ead49f03d234725efd Signed-off-by: Nishant Puri <npuri(a)redhat.com> --- M src/log/log_functions.c 1 file changed, 64 insertions(+), 36 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/43/1241843/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1241843?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: I4b1c2ca0cc3cefca556839ead49f03d234725efd Gerrit-Change-Number: 1241843 Gerrit-PatchSet: 1 Gerrit-Owner: Nishant Puri <npuri(a)redhat.com>

4 weeks, 1 day

1
0
0 / 0

[L] Change in ...nfs-ganesha[next]: gRPC: Add gRPC client I/O show stats APIs for NFS v3/v4.x

by Suhas Athani (GerritHub)

Suhas Athani has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1241403?usp=email ) Change subject: gRPC: Add gRPC client I/O show stats APIs for NFS v3/v4.x ...................................................................... gRPC: Add gRPC client I/O show stats APIs for NFS v3/v4.x - Add a new cltmgrService.proto for client manager stats RPCs mirroring D-Bus cltmgr_show_v3_io, cltmgr_show_v40_io, cltmgr_show_v41_io, and cltmgr_show_v42_io. - Add shared protobuf messages for client IP requests, status, timestamps, and read/write I/O stats. - Implement gRPC handlers that reuse the existing client manager and server stats data paths while keeping C++ gRPC code isolated from D-Bus/private C headers. New RPCs: cltmgrService.ClientStats/GetNFSv3IO cltmgrService.ClientStats/GetNFSv40IO cltmgrService.ClientStats/GetNFSv41IO cltmgrService.ClientStats/GetNFSv42IO - Each RPC accepts a client IP address and returns status, timestamp, read stats, and write stats. - Error handling matches D-Bus behavior via response status fields, while transport-level gRPC status remains OK. Change-Id: Iaddba09713befc6069a7b4b55d71a31f6d704f0d Signed-off-by: Suhas Athani <Suhas.Athani(a)ibm.com> --- M src/grpc_server/GrpcServer.cc M src/grpc_server/GrpcServer.h M src/grpc_server/nfsService.h M src/grpc_server/nfsServiceServer.cc A src/grpc_server/proto/cltmgrService.proto M src/grpc_server/proto/nfsProtoUtil.proto A src/include/server_stats_grpc.h M src/support/client_mgr.c M src/support/server_stats.c 9 files changed, 652 insertions(+), 0 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/03/1241403/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1241403?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: Iaddba09713befc6069a7b4b55d71a31f6d704f0d Gerrit-Change-Number: 1241403 Gerrit-PatchSet: 1 Gerrit-Owner: Suhas Athani <Suhas.Athani(a)ibm.com>

1 month

1
0
0 / 0

[S] Change in ...nfs-ganesha[next]: FSAL/CEPH: Release invalidated dentry inode

by ShengLi (GerritHub)

ShengLi has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1241257?usp=email ) Change subject: FSAL/CEPH: Release invalidated dentry inode ...................................................................... FSAL/CEPH: Release invalidated dentry inode Change-Id: I68d7847a87b363c7b4332ed853bfd23fd7963879 Signed-off-by: Sheng Li <50203418+LiShengG(a)users.noreply.github.com> --- M src/FSAL/FSAL_CEPH/main.c 1 file changed, 20 insertions(+), 0 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/57/1241257/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1241257?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: I68d7847a87b363c7b4332ed853bfd23fd7963879 Gerrit-Change-Number: 1241257 Gerrit-PatchSet: 1 Gerrit-Owner: ShengLi <50203418+LiShengG(a)users.noreply.github.com>

1 month

1
0
0 / 0

[XS] Change in ...nfs-ganesha[next]: packaging: fix ganesha-top install on el10

by Name of user not set (GerritHub)

mwatts(a)redhat.com has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1241295?usp=email ) Change subject: packaging: fix ganesha-top install on el10 ...................................................................... packaging: fix ganesha-top install on el10 This commit works around antique python install logic for ganesha-top. "mkdir -p build/lib" invites the python installer to recursively search for all .py files, which results in really weird behavior when it finds build/lib. We don't need this, so get rid of it. The goal is that the wheel that comes out of this should not have setup.py or build/lib, or any variation thereof. package_dir isn't needed here, so get rid of it. Change-Id: Ibb118d6c6666efe5d4909d7cb14225d7a0ddc60d Signed-off-by: Marcus Watts <mwatts(a)redhat.com> --- M src/scripts/ganesha-top/CMakeLists.txt M src/scripts/ganesha-top/setup.py.in 2 files changed, 0 insertions(+), 2 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/95/1241295/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1241295?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: Ibb118d6c6666efe5d4909d7cb14225d7a0ddc60d Gerrit-Change-Number: 1241295 Gerrit-PatchSet: 1 Gerrit-Owner: mwatts(a)redhat.com

1 month

1
0
0 / 0

[S] Change in ...nfs-ganesha[next]: packaging: install gpfs-epoch in el10

by Name of user not set (GerritHub)

mwatts(a)redhat.com has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1241294?usp=email ) Change subject: packaging: install gpfs-epoch in el10 ...................................................................... packaging: install gpfs-epoch in el10 Ganesha wants to put gpfs-epoch, a python script, into /usr/libexec/ganesha (or some logical variation thereof). In el10, we have python 3.12, which removes support for antique install logic. The replacement (python3 -m installer) lacks support for setting bindir and always puts scripts into /usr/bin. Rather than fighting with the python installer, this commit just installs the script directly. Change-Id: I37fca5330908b10440042d7e35eb1d088f3ab513 Signed-off-by: Marcus Watts <mwatts(a)redhat.com> --- M src/scripts/gpfs-epoch/CMakeLists.txt 1 file changed, 6 insertions(+), 9 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/94/1241294/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1241294?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: I37fca5330908b10440042d7e35eb1d088f3ab513 Gerrit-Change-Number: 1241294 Gerrit-PatchSet: 1 Gerrit-Owner: mwatts(a)redhat.com

1 month

1
0
0 / 0

[XS] Change in ...nfs-ganesha[next]: gcc 14: fix uninitialized variables

by Name of user not set (GerritHub)

mwatts(a)redhat.com has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1241293?usp=email ) Change subject: gcc 14: fix uninitialized variables ...................................................................... gcc 14: fix uninitialized variables Gcc 14 in el10 likes to warn about things that it thinks might be uninitialized. Ganesha builds with warnings as errors. Result: unhappiness. Change-Id: I51260792b34f136dbed9d5056e45e45e8e1ae33d Signed-off-by: Marcus Watts <mwatts(a)redhat.com> --- M src/FSAL/FSAL_CEPH/main.c M src/FSAL/Stackable_FSALs/FSAL_MDCACHE/mdcache_helpers.c M src/tools/ganesha-rados-grace.c 3 files changed, 3 insertions(+), 3 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/93/1241293/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1241293?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: I51260792b34f136dbed9d5056e45e45e8e1ae33d Gerrit-Change-Number: 1241293 Gerrit-PatchSet: 1 Gerrit-Owner: mwatts(a)redhat.com

1 month

1
0
0 / 0

[XS] Change in ...nfs-ganesha[next]: packaging: buildrequires python3-setuptools

by Name of user not set (GerritHub)

mwatts(a)redhat.com has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1241292?usp=email ) Change subject: packaging: buildrequires python3-setuptools ...................................................................... packaging: buildrequires python3-setuptools Install python3-setuptools on el 9, 10. Change-Id: Ie1e7f06400b17f2696c9c1085f39ca090f27245c Signed-off-by: Marcus Watts <mwatts(a)redhat.com> --- M src/nfs-ganesha.spec-in.cmake 1 file changed, 3 insertions(+), 0 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/92/1241292/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1241292?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: Ie1e7f06400b17f2696c9c1085f39ca090f27245c Gerrit-Change-Number: 1241292 Gerrit-PatchSet: 1 Gerrit-Owner: mwatts(a)redhat.com

1 month

1
0
0 / 0

[XS] Change in ...nfs-ganesha[next]: packaging: must install /Ganesha/ bits for el9,10

by Name of user not set (GerritHub)

mwatts(a)redhat.com has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1241291?usp=email ) Change subject: packaging: must install /Ganesha/ bits for el9,10 ...................................................................... packaging: must install /Ganesha/ bits for el9,10 They get installed. And I believe at least some of the bits are needed by some of the nfs ganesha utilities. Change-Id: I51710ee178ecd423cc378652b8cd0b3cff330085 Signed-off-by: Marcus Watts <mwatts(a)redhat.com> --- M src/nfs-ganesha.spec-in.cmake 1 file changed, 4 insertions(+), 0 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/91/1241291/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1241291?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: I51710ee178ecd423cc378652b8cd0b3cff330085 Gerrit-Change-Number: 1241291 Gerrit-PatchSet: 1 Gerrit-Owner: mwatts(a)redhat.com

1 month

1
0
0 / 0

[XS] Change in ...nfs-ganesha[next]: packaging : do not remove __pycache__

by Name of user not set (GerritHub)

mwatts(a)redhat.com has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1241290?usp=email ) Change subject: packaging : do not remove __pycache__ ...................................................................... packaging : do not remove __pycache__ Removing __pycache__ at buld time won't work. It will be built by an automatic rule at %install time which happens after %build time. Secondly, removing __pycache__ is Bad. What will happen then is: the first time root runs across this python module, a .pyc file will be made. Since that .pyc file isn't being managed, it will interfere with later attempts to remove or update the package. Change-Id: I119ebe699a92d6970964ad7859fe9400e24a32a5 Signed-off-by: Marcus Watts <mwatts(a)redhat.com> --- M src/nfs-ganesha.spec-in.cmake 1 file changed, 0 insertions(+), 2 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/90/1241290/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1241290?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: I119ebe699a92d6970964ad7859fe9400e24a32a5 Gerrit-Change-Number: 1241290 Gerrit-PatchSet: 1 Gerrit-Owner: mwatts(a)redhat.com

1 month

1
0
0 / 0

[XS] Change in ...nfs-ganesha[next]: packaging : don't install bits not built

by Name of user not set (GerritHub)

mwatts(a)redhat.com has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1241289?usp=email ) Change subject: packaging : don't install bits not built ...................................................................... packaging : don't install bits not built If using the system libntirpc, the file libntirpcmonitoring.so will not be built and cannot be installed. Change-Id: I943ed9958bb1326acc21f3354f8d8ced297ded0a Signed-off-by: Marcus Watts <mwatts(a)redhat.com> --- M src/nfs-ganesha.spec-in.cmake 1 file changed, 2 insertions(+), 0 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/89/1241289/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1241289?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: I943ed9958bb1326acc21f3354f8d8ced297ded0a Gerrit-Change-Number: 1241289 Gerrit-PatchSet: 1 Gerrit-Owner: mwatts(a)redhat.com

1 month

1
0
0 / 0

2026

2025

2024

2023

2022

2021

2020

2019

2018

Devel July 2026