July 2020 - Devel - NFS Ganesha List Archives

PROXY fails to renew session under heavy load
by Alok Sinha 19 Jul '20

19 Jul '20

Version : 2.8.3 FSAL: PROXY NFS version tried : 4.1 and 4.2 Setup: Running iozone from 40 hosts Proxy works fine under low load ( 10 threads of iozone). As loaded increases, proxy server fails with only message after few seconds: Failed to renew session Any idea? I am planning to use a proxy in production. Can proxy take the load? I see that the connection between proxy server and nfs server is just one which will have limited RPC slots ( 16 by default ) per host. Can 16 RPC take a reasonable amount of load for production use? -alok

1 0

Change in ...nfs-ganesha[next]: [GPFS] Fix gpfs-epoch.py script for python3
by Malahal (GerritHub) 19 Jul '20

19 Jul '20

Malahal has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/498287 ) Change subject: [GPFS] Fix gpfs-epoch.py script for python3 ...................................................................... [GPFS] Fix gpfs-epoch.py script for python3 Can't use binary string with re.search. Add decode() to the binary string. This gives text string in python3 (unicode string in python2) and it will work with both python2 and python3. Change-Id: I49cd124b19881afa9ca0af0b916eca2bba31fd82 Signed-off-by: Malahal Naineni <malahal(a)us.ibm.com> --- M src/scripts/gpfs-epoch/gpfs-epoch.py 1 file changed, 1 insertion(+), 1 deletion(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/87/498287/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/498287 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: I49cd124b19881afa9ca0af0b916eca2bba31fd82 Gerrit-Change-Number: 498287 Gerrit-PatchSet: 1 Gerrit-Owner: Malahal <malahal(a)gmail.com> Gerrit-MessageType: newchange

1 0

Announce Push of V4-dev.26
by Frank Filz 17 Jul '20

17 Jul '20

Branch next Tag:V4-dev.26 Merge Highlights * NFSv4.1 CLOSE/LOCK race fix * Fix mdcache_test_attrs_trust to check for required attrs * Fix fsal_common_is_referral() * ProxyV3: Fix reuse of idle-closed connections. * selinux: Lock test is failing on v3 mount in Enforcing mode * CMAKE - stop using deperecated python detection Signed-off-by: Frank S. Filz <ffilzlnx(a)mindspring.com> Contents: 1cace94 Frank S. Filz V4-dev.26 f906429 Daniel Gryniewicz CMAKE - stop using deperecated python detection 89a66be Kaleb S. KEITHLEY selinux: Lock test is failing on v3 mount in Enforcing mode fcdbf25 Solomon Boulos ProxyV3: Fix reuse of idle-closed connections. bcbd9a7 Malahal Naineni Fix fsal_common_is_referral() 15a70af Malahal Naineni Fix mdcache_test_attrs_trust to check for required attrs 4c7ef03 Malahal Naineni NFSv4.1 CLOSE/LOCK race fix

1 0

Change in ...nfs-ganesha[next]: move run dir to /run
by Dominique Martinet (GerritHub) 17 Jul '20

17 Jul '20

Dominique Martinet has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/498224 ) Change subject: move run dir to /run ...................................................................... move run dir to /run /run has been preferred over /var/run for a while. The main motivation for this change is that /run is not dependent on SYSCONFDIR, and under some distributions (e.g. nixos) led to improbable run dir. Also, for systemd distributions, /run/ganesha has been created by systemd for a while (via the RunDirectory= setting), so embrace the dynamic directory creation for all service files Change-Id: I83b3871d5dd1984534487e6e237dbe8ae0eefe00 Signed-off-by: Dominique Martinet <dominique.martinet(a)cea.fr> --- M src/CMakeLists.txt M src/doc/man/ganesha-core-config.rst M src/include/config-h.in.cmake M src/libntirpc M src/nfs-ganesha.spec-in.cmake M src/scripts/init.d/nfs-ganesha.debian8 M src/scripts/init.d/nfs-ganesha.el6 M src/scripts/init.d/nfs-ganesha.gpfs M src/selinux/ganesha.fc 9 files changed, 18 insertions(+), 16 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/24/498224/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/498224 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: I83b3871d5dd1984534487e6e237dbe8ae0eefe00 Gerrit-Change-Number: 498224 Gerrit-PatchSet: 1 Gerrit-Owner: Dominique Martinet <asmadeus(a)codewreck.org> Gerrit-MessageType: newchange

1 0

Change in ...nfs-ganesha[next]: Fix fsal_common_is_referral()
by Malahal (GerritHub) 16 Jul '20

16 Jul '20

Malahal has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/498188 ) Change subject: Fix fsal_common_is_referral() ...................................................................... Fix fsal_common_is_referral() We probably don't have a case where only one of TYPE or MODE attributes is valid, so this isn't really a bug fix. Just a good practice to have right conditional expression! Change-Id: Iee4453e17a153fd35017e287d27ceaa9ed75bf56 Signed-off-by: Malahal Naineni <malahal(a)us.ibm.com> --- M src/FSAL/commonlib.c 1 file changed, 4 insertions(+), 2 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/88/498188/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/498188 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: Iee4453e17a153fd35017e287d27ceaa9ed75bf56 Gerrit-Change-Number: 498188 Gerrit-PatchSet: 1 Gerrit-Owner: Malahal <malahal(a)gmail.com> Gerrit-MessageType: newchange

1 0

Change in ...nfs-ganesha[next]: Fix mdcache_test_attrs_trust to check for required attrs
by Malahal (GerritHub) 16 Jul '20

16 Jul '20

Malahal has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/498187 ) Change subject: Fix mdcache_test_attrs_trust to check for required attrs ...................................................................... Fix mdcache_test_attrs_trust to check for required attrs pynfs st_read.testLargeCount test (RD4) creates a file with 10MB size with OPEN request and reads it. When pynfs reads the file, nfs4_read_cb() sets EOF bit as the file size attribute happens to be zero in the attribute cache then. This happens after commit 72d4e632d3c81ff1db0988459bd5e45be7b18949. The pynfs test open request compound has OPEN and GETFH in addition to others, and nfs4_op_getfh() is using is_referral() to cache all attributes. Commit 72d4e632d3c81ff1db0988459bd5e45be7b18949 made the mdcache_is_referral() ignore the passed in attrs leading to attribute cache not having all attributes in the cache. mdcache_test_attrs_trust() currently checks if we can trust attributes but doesn't check if all the requested attributes are valid. Change-Id: Idbdb5a22809cd62b991fbe3aab81e62cfef58c48 Signed-off-by: Malahal Naineni <malahal(a)us.ibm.com> --- M src/FSAL/Stackable_FSALs/FSAL_MDCACHE/mdcache_int.h 1 file changed, 3 insertions(+), 0 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/87/498187/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/498187 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: Idbdb5a22809cd62b991fbe3aab81e62cfef58c48 Gerrit-Change-Number: 498187 Gerrit-PatchSet: 1 Gerrit-Owner: Malahal <malahal(a)gmail.com> Gerrit-MessageType: newchange

1 0

Change in ...nfs-ganesha[next]: Fix PNFS flags in EXCHANGE_ID
by Sriram Patil (GerritHub) 16 Jul '20

16 Jul '20

Sriram Patil has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/498184 ) Change subject: Fix PNFS flags in EXCHANGE_ID ...................................................................... Fix PNFS flags in EXCHANGE_ID NFSv4.1 RFC provides the list of combinations that are acceptable in case of PNFS. However, all these restrictions are only applied to server, the client is allowed to send any combination when requesting. This change fixes the EXCHANGE_ID code to make sure that the server always returns acceptable PNFS flags even if client has request invalid combination. Change-Id: Iba05fb9729beda8bc30c957f3a3f153a37c8554c Signed-off-by: Sriram Patil <sriramp(a)vmware.com> --- M src/Protocols/NFS/nfs4_op_exchange_id.c 1 file changed, 23 insertions(+), 5 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/84/498184/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/498184 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: Iba05fb9729beda8bc30c957f3a3f153a37c8554c Gerrit-Change-Number: 498184 Gerrit-PatchSet: 1 Gerrit-Owner: Sriram Patil <sriramp(a)vmware.com> Gerrit-MessageType: newchange

1 0

Down to less than 100 open github issues!
by Frank Filz 16 Jul '20

16 Jul '20

I took another run at closing github issues. As the backlog is tracking newer and newer, I would encourage anyone who has worked an issue to check if it maybe has been resolved. If a patch has been merged upstream I think we can consider the issue resolved unless a backport seems necessary, and if so, and the backport has not been done, indicate so in the issue (and then let's get the issue up on the Kanban project board so we can more easily track it.) Frank

2 3

Change in ...nfs-ganesha[next]: selinux: Lock test is failing on v3 mount in Enforcing mode
by Kaleb KEITHLEY (GerritHub) 15 Jul '20

15 Jul '20

Kaleb KEITHLEY has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/498107 ) Change subject: selinux: Lock test is failing on v3 mount in Enforcing mode ...................................................................... selinux: Lock test is failing on v3 mount in Enforcing mode https://bugzilla.redhat.com/show_bug.cgi?id=1857170 More selinux whack-a-mole Change-Id: I7c33f3b91465905daf6605b3726369874b3849e4 Signed-off-by: Kaleb S. KEITHLEY <kkeithle(a)redhat.com> --- M src/selinux/ganesha.te 1 file changed, 7 insertions(+), 0 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/07/498107/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/498107 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: I7c33f3b91465905daf6605b3726369874b3849e4 Gerrit-Change-Number: 498107 Gerrit-PatchSet: 1 Gerrit-Owner: Kaleb KEITHLEY <kaleb(a)redhat.com> Gerrit-MessageType: newchange

1 0

Change in ...nfs-ganesha[next]: CMAKE - stop using deperecated python detection
by Daniel Gryniewicz (GerritHub) 14 Jul '20

14 Jul '20

Daniel Gryniewicz has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/498039 ) Change subject: CMAKE - stop using deperecated python detection ...................................................................... CMAKE - stop using deperecated python detection PythonInterp is deprecated, we're supposed to use Python now. Switch to the supported detection. Change-Id: I8feaa07ce427743dc291242bc3512f8243d56468 Signed-off-by: Daniel Gryniewicz <dang(a)redhat.com> --- M src/CMakeLists.txt M src/scripts/CMakeLists.txt M src/scripts/ganeshactl/CMakeLists.txt M src/scripts/gpfs-epoch/CMakeLists.txt 4 files changed, 13 insertions(+), 14 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/39/498039/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/498039 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: I8feaa07ce427743dc291242bc3512f8243d56468 Gerrit-Change-Number: 498039 Gerrit-PatchSet: 1 Gerrit-Owner: Daniel Gryniewicz <dang(a)redhat.com> Gerrit-MessageType: newchange

1 0