October 2019 - Devel - NFS Ganesha List Archives

Change in ...nfs-ganesha[next]: Fix ganesha_mgr show_version failed.
by xijinyu (GerritHub) 23 Oct '19

23 Oct '19

xijinyu has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/472234 ) Change subject: Fix ganesha_mgr show_version failed. ...................................................................... Fix ganesha_mgr show_version failed. For example: [root@localhost ]# ganesha_mgr show_version NFS-Ganesha Release = V2.9-dev.11 ganesha compiled on Oct 23 2019 at 08:31:13 Release comment = GANESHA file server is 64 bits compliant and supports NFS v3,4.0,4.1 (pNFS) and 9P Git HEAD = NOT-GIT Git Describe = /root/rpmbuild/BUILD/nfs-ganesha-2.9-dev.11 Signed-off-by: Xi Jinyu <xijinyu(a)cmss.chinamobile.com> Change-Id: I1243552d49b12aad01d9626ef0931e9445a471e6 --- M src/scripts/ganeshactl/ganesha_mgr.py 1 file changed, 4 insertions(+), 0 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/34/472234/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/472234 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: I1243552d49b12aad01d9626ef0931e9445a471e6 Gerrit-Change-Number: 472234 Gerrit-PatchSet: 1 Gerrit-Owner: xijinyu <xijinyu(a)cmss.chinamobile.com> Gerrit-MessageType: newchange

1 0

Change in ...nfs-ganesha[next]: Utilize svc referece to nfs reqdata for callback within nfs v4, inste...
by Nathaniel Roiger (GerritHub) 21 Oct '19

21 Oct '19

Nathaniel Roiger has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/471921 ) Change subject: Utilize svc referece to nfs reqdata for callback within nfs v4, instead of that of the xp_u1. ...................................................................... Utilize svc referece to nfs reqdata for callback within nfs v4, instead of that of the xp_u1. Change-Id: Ibf2f2f2adc0d54e117c911c6c65e58f215d791e8 Signed-off-by: TweakySolution <nroiger(a)gmail.com> --- M src/Protocols/NFS/nfs4_Compound.c 1 file changed, 1 insertion(+), 3 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/21/471921/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/471921 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: Ibf2f2f2adc0d54e117c911c6c65e58f215d791e8 Gerrit-Change-Number: 471921 Gerrit-PatchSet: 1 Gerrit-Owner: Nathaniel Roiger <nate_roiger(a)atavium.com> Gerrit-MessageType: newchange

1 0

Change in ...nfs-ganesha[next]: RADOS KV - Don't deref size, it may be NULL
by Daniel Gryniewicz (GerritHub) 21 Oct '19

21 Oct '19

Daniel Gryniewicz has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/471899 ) Change subject: RADOS KV - Don't deref size, it may be NULL ...................................................................... RADOS KV - Don't deref size, it may be NULL In rados_kv_create_val(), size may be NULL. Don't assume it exists, only use it if it's passed. Change-Id: I6d42411241a8156cbf04cd6a6e9834b663474829 Signed-off-by: Daniel Gryniewicz <dang(a)redhat.com> --- M src/SAL/recovery/recovery_rados_kv.c 1 file changed, 6 insertions(+), 2 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/99/471899/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/471899 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: I6d42411241a8156cbf04cd6a6e9834b663474829 Gerrit-Change-Number: 471899 Gerrit-PatchSet: 1 Gerrit-Owner: Daniel Gryniewicz <dang(a)redhat.com> Gerrit-MessageType: newchange

1 0

Announce Push of V3.0-rc3
by Frank Filz 18 Oct '19

18 Oct '19

Branch next Tag:V3.0-rc3 Release Highlights * 2 GPFS fixes * Fix C++ build * Fix tabs in python file Signed-off-by: Frank S. Filz <ffilzlnx(a)mindspring.com> Contents: b358209 Frank S. Filz V3.0-rc3 660404e Daniel Gryniewicz Fix tabs in python file ebbb633 Daniel Gryniewicz Fix C++ build 26b07c2 Madhu Thorat [GPFS] Close parent fd when the parent's and the child's file systems differ e0863ee Madhu Thorat [GPFS]: Use fsal_internal_close() to close FDs for symbolic links

1 0

Change in ...nfs-ganesha[next]: build: remove librados dependency on ganesha.nfsd
by Kaleb KEITHLEY (GerritHub) 17 Oct '19

17 Oct '19

Kaleb KEITHLEY has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/471667 ) Change subject: build: remove librados dependency on ganesha.nfsd ...................................................................... build: remove librados dependency on ganesha.nfsd When using ganesha for non-ceph backing stores, it should not require that librados be installed. Signed-off-by: Kaleb S. KEITHLEY <kkeithle(a)redhat.com> Change-Id: I0067229f4ae93752584ceb7fab4411e02ee4d4f9 --- M src/CMakeLists.txt M src/MainNFSD/CMakeLists.txt M src/MainNFSD/libganesha_nfsd.ver M src/MainNFSD/nfs_admin_thread.c M src/MainNFSD/nfs_init.c M src/SAL/CMakeLists.txt M src/SAL/nfs4_recovery.c M src/config_parsing/CMakeLists.txt M src/config_parsing/conf_url.c M src/include/conf_url.h M src/include/conf_url_rados.h M src/include/sal_functions.h M src/nfs-ganesha.spec-in.cmake M src/support/CMakeLists.txt 14 files changed, 194 insertions(+), 73 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/67/471667/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/471667 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: I0067229f4ae93752584ceb7fab4411e02ee4d4f9 Gerrit-Change-Number: 471667 Gerrit-PatchSet: 1 Gerrit-Owner: Kaleb KEITHLEY <kaleb(a)redhat.com> Gerrit-MessageType: newchange

1 0

Change in ...nfs-ganesha[next]: Use same strategy for nfs4
by Nathaniel Roiger (GerritHub) 16 Oct '19

16 Oct '19

Nathaniel Roiger has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/471555 ) Change subject: Use same strategy for nfs4 ...................................................................... Use same strategy for nfs4 Change-Id: Ide764c90dc4e084cdbd602060a4220a4ad954622 Signed-off-by: TweakySolution <nroiger(a)gmail.com> --- M src/Protocols/NFS/nfs4_Compound.c 1 file changed, 1 insertion(+), 3 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/55/471555/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/471555 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: Ide764c90dc4e084cdbd602060a4220a4ad954622 Gerrit-Change-Number: 471555 Gerrit-PatchSet: 1 Gerrit-Owner: Nathaniel Roiger <nate_roiger(a)atavium.com> Gerrit-MessageType: newchange

1 0

Change in ...nfs-ganesha[next]: Fix tabs in python file
by Daniel Gryniewicz (GerritHub) 16 Oct '19

16 Oct '19

Daniel Gryniewicz has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/471546 ) Change subject: Fix tabs in python file ...................................................................... Fix tabs in python file Change-Id: Iba61d0e415c82df0118232011b1e301ef49a4100 Signed-off-by: Daniel Gryniewicz <dang(a)redhat.com> --- M src/scripts/ganeshactl/Ganesha/glib_dbus_stats.py 1 file changed, 2 insertions(+), 2 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/46/471546/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/471546 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: Iba61d0e415c82df0118232011b1e301ef49a4100 Gerrit-Change-Number: 471546 Gerrit-PatchSet: 1 Gerrit-Owner: Daniel Gryniewicz <dang(a)redhat.com> Gerrit-MessageType: newchange

1 0

Change in ...nfs-ganesha[next]: Fix C++ build
by Daniel Gryniewicz (GerritHub) 16 Oct '19

16 Oct '19

Daniel Gryniewicz has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/471532 ) Change subject: Fix C++ build ...................................................................... Fix C++ build C++ automatically typedefs all structs, so you can't have a struct with the same name as a typedef that has a different type. Fix this by renameing struct fattr3 to struct fattr3_wire. Change-Id: I5dc18d5046e2d6597ad38bdac69aedd3fd7637bd Signed-off-by: Daniel Gryniewicz <dang(a)redhat.com> --- M src/Protocols/NFS/nfs3_readdirplus.c M src/include/nfs23.h 2 files changed, 3 insertions(+), 3 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/32/471532/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/471532 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: I5dc18d5046e2d6597ad38bdac69aedd3fd7637bd Gerrit-Change-Number: 471532 Gerrit-PatchSet: 1 Gerrit-Owner: Daniel Gryniewicz <dang(a)redhat.com> Gerrit-MessageType: newchange

1 0

Change in ...nfs-ganesha[next]: nfs v3 async read/write callbacks reference obsolete transport pointe...
by Nathaniel Roiger (GerritHub) 16 Oct '19

16 Oct '19

Nathaniel Roiger has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/471492 ) Change subject: nfs v3 async read/write callbacks reference obsolete transport pointer. reqdata should be initialized from the scv request. ...................................................................... nfs v3 async read/write callbacks reference obsolete transport pointer. reqdata should be initialized from the scv request. Change-Id: Iee81ee3d9c65175a7bc9283b561729da8d379a9a Signed-off-by: TweakySolution <nroiger(a)gmail.com> --- M src/Protocols/NFS/nfs3_read.c M src/Protocols/NFS/nfs3_write.c 2 files changed, 2 insertions(+), 4 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/92/471492/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/471492 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: Iee81ee3d9c65175a7bc9283b561729da8d379a9a Gerrit-Change-Number: 471492 Gerrit-PatchSet: 1 Gerrit-Owner: Nathaniel Roiger <nate_roiger(a)atavium.com> Gerrit-MessageType: newchange

1 0

Change in ...nfs-ganesha[next]: GPFSFSAL_lookup: Close 'parent_fd' immediately after getting file handle
by Madhu Thorat (GerritHub) 15 Oct '19

15 Oct '19

Madhu Thorat has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/471389 ) Change subject: GPFSFSAL_lookup: Close 'parent_fd' immediately after getting file handle ...................................................................... GPFSFSAL_lookup: Close 'parent_fd' immediately after getting file handle In GPFSFSAL_lookup() we don't use 'parent_fd' after calling fsal_internal_get_handle_at(..) to get file handle 'fh'. Thus we can close 'parent_fd' immediately after getting 'fh'. Made the change for the same, and removed call to fsal_internal_close(parent_fd, NULL, 0) at other places in the same function. Change-Id: Id477814dac223297a8251749217a72b0210a2214 Signed-off-by: Madhu Thorat <madhu.punjabi(a)in.ibm.com> --- M src/FSAL/FSAL_GPFS/fsal_lookup.c 1 file changed, 7 insertions(+), 5 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/89/471389/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/471389 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: Id477814dac223297a8251749217a72b0210a2214 Gerrit-Change-Number: 471389 Gerrit-PatchSet: 1 Gerrit-Owner: Madhu Thorat <madhu.punjabi(a)in.ibm.com> Gerrit-MessageType: newchange

1 0