December 2021 - Devel - NFS Ganesha List Archives

Announce Push of V4-rc7
by Frank Filz 17 Dec '21

17 Dec '21

Branch next Tag:V4-rc7 NOTE: This merge includes an ntirpc pull up. Please update your submodule NOTE: This is the final merge before V4.0 will be tagged Ganesha V4.0 is in sight! Merge Highlights * ganesha_mgr: Fix some "IndexError: list index out of range" error. * Fixed NFSv4 lock reclaim issue in failover scenario * Make client/export related dbus command more clarify * Adding xattr support to FSAL_RGW * MDCACHE - Proper chunk refcounting when skipping * Pullup ntirpc 4.0 * Bump FSAL_MAJOR_VERSION to 11 Signed-off-by: Frank S. Filz <ffilzlnx(a)mindspring.com> Contents: f8566a7dc Frank S. Filz V4-rc7 ca8a8fb0a Frank S. Filz Bump FSAL_MAJOR_VERSION to 11 dd080623e Daniel Gryniewicz Pullup ntirpc 4.0 dd5c9e65d Daniel Gryniewicz MDCACHE - Proper chunk refcounting when skipping 6074f1967 Arjun Sharma Adding xattr support to FSAL_RGW c0c104750 Vicente Cheng Make client/export related dbus command more clarify 4cfc59d0e Prabhu Murugesan Fixed NFSv4 lock reclaim issue in failover scenario 9c43e9bd5 Xi Jinyu ganesha_mgr: Fix some "IndexError: list index out of range" error.

1 0

Change in ...nfs-ganesha[next]: Pullup ntirpc 4.0
by Daniel Gryniewicz (GerritHub) 17 Dec '21

17 Dec '21

Daniel Gryniewicz has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/529636 ) Change subject: Pullup ntirpc 4.0 ...................................................................... Pullup ntirpc 4.0 Change-Id: Ice51409f4c4434cf9285014bf5de3c0cab5566fb Signed-off-by: Daniel Gryniewicz <dang(a)redhat.com> --- M src/libntirpc 1 file changed, 11 insertions(+), 1 deletion(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/36/529636/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/529636 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: Ice51409f4c4434cf9285014bf5de3c0cab5566fb Gerrit-Change-Number: 529636 Gerrit-PatchSet: 1 Gerrit-Owner: Daniel Gryniewicz <dang(a)redhat.com> Gerrit-MessageType: newchange

1 0

Change in ...nfs-ganesha[next]: Bump FSAL_MAJOR_VERSION to 11
by Frank Filz (GerritHub) 16 Dec '21

16 Dec '21

Frank Filz has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/529594 ) Change subject: Bump FSAL_MAJOR_VERSION to 11 ...................................................................... Bump FSAL_MAJOR_VERSION to 11 Change-Id: I30781f66f132e6788f334e9a3174450da8a04892 Signed-off-by: Frank S. Filz <ffilzlnx(a)mindspring.com> --- M src/include/fsal_api.h 1 file changed, 11 insertions(+), 1 deletion(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/94/529594/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/529594 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: I30781f66f132e6788f334e9a3174450da8a04892 Gerrit-Change-Number: 529594 Gerrit-PatchSet: 1 Gerrit-Owner: Frank Filz <ffilzlnx(a)mindspring.com> Gerrit-MessageType: newchange

1 0

Final call for V4.0
by Frank Filz 16 Dec '21

16 Dec '21

Tomorrow I will be doing the final rc merge for V4.0 and then on Monday I expect to tag V4.0. I am waiting on some testing for the RGW xattrs patch, but the code looks OK, so I should be able to merge unless testing shows an issue (in which case, the patch will need to be fixed up for merge by my end of day Monday or at worst, Tuesday morning. After the V4.0 tag, I will refrain from merging features for the first several weeks of 2022 so we can do a V4.1 before forking. Thanks Frank Filz

1 0

Change in ...nfs-ganesha[next]: ganesha_mgr: Fix some "IndexError: list index out of range" error.
by xijinyu (GerritHub) 14 Dec '21

14 Dec '21

xijinyu has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/529469 ) Change subject: ganesha_mgr: Fix some "IndexError: list index out of range" error. ...................................................................... ganesha_mgr: Fix some "IndexError: list index out of range" error. [root@localhost ~]# ganesha_mgr getall Traceback (most recent call last): File "/usr/bin/ganesha_mgr", line 497, in <module> if sys.argv[2] == "logs": IndexError: list index out of range [root@localhost ~]# ganesha_mgr get Traceback (most recent call last): File "/usr/bin/ganesha_mgr", line 485, in <module> if sys.argv[2] == "log": IndexError: list index out of range [root@localhost ~]# ganesha_mgr display Traceback (most recent call last): File "/usr/bin/ganesha_mgr", line 417, in <module> if sys.argv[2] == "export": IndexError: list index out of range ...... Signed-off-by: Xi Jinyu <xijinyu0811(a)163.com> Change-Id: Ib410e526b69247dd99c98665d63378f3c3b1b3ae --- M src/scripts/ganeshactl/ganesha_mgr.py 1 file changed, 56 insertions(+), 21 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/69/529469/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/529469 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: Ib410e526b69247dd99c98665d63378f3c3b1b3ae Gerrit-Change-Number: 529469 Gerrit-PatchSet: 1 Gerrit-Owner: xijinyu <xijinyu0811(a)163.com> Gerrit-MessageType: newchange

1 0

Change in ...nfs-ganesha[next]: MDCACHE - Proper chunk refcounting when skipping
by Daniel Gryniewicz (GerritHub) 14 Dec '21

14 Dec '21

Daniel Gryniewicz has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/529440 ) Change subject: MDCACHE - Proper chunk refcounting when skipping ...................................................................... MDCACHE - Proper chunk refcounting when skipping Change-Id: I90cbd055880e36f79b05ee4bfaa6af9e6866238f Signed-off-by: Daniel Gryniewicz <dang(a)redhat.com> --- M src/FSAL/Stackable_FSALs/FSAL_MDCACHE/mdcache_helpers.c 1 file changed, 22 insertions(+), 10 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/40/529440/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/529440 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: I90cbd055880e36f79b05ee4bfaa6af9e6866238f Gerrit-Change-Number: 529440 Gerrit-PatchSet: 1 Gerrit-Owner: Daniel Gryniewicz <dang(a)redhat.com> Gerrit-MessageType: newchange

1 0

With V4-rc6 out and the year almost over we are almost there!
by Frank Filz 10 Dec '21

10 Dec '21

I just pushed V4-rc6. Next week is the last full week before the end of year holiday time and I expect to be the last merge before I tag 4.0 (which may happen separately on Monday the 20th). With the final push, I encourage anyone who has a github issue that has been marked "Needs Verification" to verify the issue with V4-rc6. Any additional testing folks are able to do would be much appreciated. Any patches that have been submitted to gerrithub but not yet merged will need to be wrapped up next week. Following the V4.0 tag, I will follow our standard practice where I do not fork off a v4-stable branch for a few weeks which gives the opportunity for a V4.1 update to include some bug fixes before I start merging new feature work for the V5 development stream. Thanks everyone. This has been a long release, but we have made some significant code improvements and fixed a lot of bugs and we do have some features of note: More information on the wiki: https://github.com/nfs-ganesha/nfs-ganesha/wiki/ReleaseNotes_4 POSIX ACL support for FSAL_CEPH Enhancements to Export Update (DBUS) and Reload (SIGHUP) protocol update (add/remove v3/v4 to export) Pseudo Path can be changed when reloading exports Refresh file system table (allows adding exports for new file systems) FSAL_PROXY_V3 FSAL_CEPH enhancement to release inodes quicker NFSACL POSIX ACL side-band protocol for NFSv3 Option to increase aggressiveness of MDCACHE releasing entries Can actually disable RQUOTA and NFSv3 when building Improvements to building on MacOS and other compilers FSAL_KVSFS Match xattr support up with RFC 8276 Add xattr to FSAL_CEPH Support BTRFS subvolumes as if they are separate file systems Said page still needs some updates so if anyone sees anything that needs updating, let us know. Thanks Frank Filz

1 0

Announce Push of V4-rc6
by Frank Filz 10 Dec '21

10 Dec '21

Branch next Tag:V4-rc6 NOTE: This merge includes an ntirpc pull up. Please update your submodule Ganesha V4.0 is in sight! Merge Highlights * On resume_op_context call SetClientIP * DRC: Better handle duplicate requests while a request is in progress * selinux: non-fatal yum install error of nfs-ganesha * Remove 3rd parameter from xdrproc_t calls Signed-off-by: Frank S. Filz <ffilzlnx(a)mindspring.com> Contents: 872aab911 Frank S. Filz V4-rc6 ea695cc84 Frank S. Filz Remove 3rd parameter from xdrproc_t calls dae2da63d Kaleb S. KEITHLEY selinux: non-fatal yum install error of nfs-ganesha 9316124dd Frank S. Filz DRC: Better handle duplicate requests while a request is in progress 885db6e52 Frank S. Filz On resume_op_context call SetClientIP

1 0

Change in ...nfs-ganesha[next]: Remove 3rd parameter from xdrproc_t calls
by Frank Filz (GerritHub) 10 Dec '21

10 Dec '21

Frank Filz has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/529211 ) Change subject: Remove 3rd parameter from xdrproc_t calls ...................................................................... Remove 3rd parameter from xdrproc_t calls Includes pullup of libntirpc 4.0rc3 Change-Id: I4735ea590b3cbee40cc4d9b0471638e6475a21fe Signed-off-by: Frank S. Filz <ffilzlnx(a)mindspring.com> --- M src/FSAL/FSAL_PROXY_V3/rpc.c M src/libntirpc 2 files changed, 2 insertions(+), 2 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/11/529211/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/529211 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: I4735ea590b3cbee40cc4d9b0471638e6475a21fe Gerrit-Change-Number: 529211 Gerrit-PatchSet: 1 Gerrit-Owner: Frank Filz <ffilzlnx(a)mindspring.com> Gerrit-MessageType: newchange

1 0

Re: Commit duplicating include line and failing make process
by Frank Filz 09 Dec '21

09 Dec '21

I'm not sure about all of that, maybe someone else has an idea. Frank From: Yoann.VALERI(a)cea.fr [mailto:Yoann.VALERI@cea.fr] Sent: Wednesday, December 8, 2021 7:23 AM To: ffilzlnx(a)mindspring.com Cc: Philippe.DENIEL(a)CEA.FR Subject: Commit duplicating include line and failing make process Hello, I am Yoann Valeri, working at CEA with Philippe Deniel on NFS-Ganesha and LibKVSNS. I have been working on NFS-Ganesha for a few months without problem, but recently, after a rebase on next, the "make" process became broken because the function "bswap_16" is not defined in multiple files (the platform I'm using is a CentOS Linux release 8.2.2004 (Core)): * nfs-ganesha/src/FSAL/FSAL_GPFS/fsal_ds.c * nfs-ganesha/src/FSAL/FSAL_GPFS/export.c * nfs-ganesha/src/FSAL/FSAL_MEM/mem_export.c * nfs-ganesha/src/FSAL/FSAL_PSEUDO/export.c After some digging, I found out the source of the problem for the last two files: in commit " 18e0dcf784ebb92fd5b04d9e31955e18cd11bad0", the two includes of "byteswap.h" were changed for "misc/portable.h", and this function is not found anymore. Moreover, in commit "5a72ceda2688297bbc82de436eaebf7d6ab0bf32", this same include "misc/portable.h" was added to remove an "#ifdef", resulting in this file being included twice, but the function "bswap_16" is unknown. By adding back the "byteswap" include, everything works fine on the other hand. However, the first two files also require this include, but they were not changed recently, at least from what I can see, hence I don't really understand why these two became problematic. The duplicated include call is an error, but do you have any idea why the "byteswap" removal for "misc/portable" breaks the building process, and why the other two files also became problems? Thanks for your help, Best regards, Yoann Valeri.

2 1