October 2018 - Devel - NFS Ganesha List Archives

Change in ffilz/nfs-ganesha[next]: Allow exports to be referral points in FSAL_VFS
by GerritHub 09 Oct '18

09 Oct '18

From Sriram Patil <sriramp(a)vmware.com>: Sriram Patil has uploaded this change for review. ( https://review.gerrithub.io/428513 Change subject: Allow exports to be referral points in FSAL_VFS ...................................................................... Allow exports to be referral points in FSAL_VFS With the current implementation, the export directories cannot act as referral point. Adding a referral point check in lookup_path code in FSAL_VFS to be able to satisfy referrals. Change-Id: Ibdfb52473056d3058b5798dd4e047606bd5172c6 Signed-off-by: Sriram Patil <sriramp(a)vmware.com> --- M src/FSAL/FSAL_VFS/handle.c 1 file changed, 73 insertions(+), 33 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/13/428513/1 -- To view, visit https://review.gerrithub.io/428513 To unsubscribe, or for help writing mail filters, visit https://review.gerrithub.io/settings Gerrit-Project: ffilz/nfs-ganesha Gerrit-Branch: next Gerrit-MessageType: newchange Gerrit-Change-Id: Ibdfb52473056d3058b5798dd4e047606bd5172c6 Gerrit-Change-Number: 428513 Gerrit-PatchSet: 1 Gerrit-Owner: Sriram Patil <sriramp(a)vmware.com>

1 0

Change in ffilz/nfs-ganesha[next]: config_samples: update the ceph.conf example config
by GerritHub 08 Oct '18

08 Oct '18

From Jeff Layton <jlayton(a)redhat.com>: Jeff Layton has uploaded this change for review. ( https://review.gerrithub.io/428392 Change subject: config_samples: update the ceph.conf example config ...................................................................... config_samples: update the ceph.conf example config Change-Id: If305a9c317f342fbf8531b05cf82cb4d6f0ce401 Signed-off-by: Jeff Layton <jlayton(a)redhat.com> --- M src/config_samples/ceph.conf 1 file changed, 17 insertions(+), 2 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/92/428392/1 -- To view, visit https://review.gerrithub.io/428392 To unsubscribe, or for help writing mail filters, visit https://review.gerrithub.io/settings Gerrit-Project: ffilz/nfs-ganesha Gerrit-Branch: next Gerrit-MessageType: newchange Gerrit-Change-Id: If305a9c317f342fbf8531b05cf82cb4d6f0ce401 Gerrit-Change-Number: 428392 Gerrit-PatchSet: 1 Gerrit-Owner: Jeff Layton <jlayton(a)redhat.com>

1 0

Change in ffilz/nfs-ganesha[next]: doc: clean up ganesha-rados-grace document
by GerritHub 08 Oct '18

08 Oct '18

From Jeff Layton <jlayton(a)redhat.com>: Jeff Layton has uploaded this change for review. ( https://review.gerrithub.io/428391 Change subject: doc: clean up ganesha-rados-grace document ...................................................................... doc: clean up ganesha-rados-grace document There have been some changes since this was merged. Also, flesh out the latter part of the document with more information about setting up and managing a cluster. Change-Id: I3f352bbd961a1a6374872289cde9dc00f5d305f9 Signed-off-by: Jeff Layton <jlayton(a)redhat.com> --- M src/doc/man/ganesha-rados-grace.rst 1 file changed, 33 insertions(+), 23 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/91/428391/1 -- To view, visit https://review.gerrithub.io/428391 To unsubscribe, or for help writing mail filters, visit https://review.gerrithub.io/settings Gerrit-Project: ffilz/nfs-ganesha Gerrit-Branch: next Gerrit-MessageType: newchange Gerrit-Change-Id: I3f352bbd961a1a6374872289cde9dc00f5d305f9 Gerrit-Change-Number: 428391 Gerrit-PatchSet: 1 Gerrit-Owner: Jeff Layton <jlayton(a)redhat.com>

1 0

Change in ffilz/nfs-ganesha[next]: RADOS_URLS: add a namespace parameter to the config block
by GerritHub 08 Oct '18

08 Oct '18

From Jeff Layton <jlayton(a)redhat.com>: Jeff Layton has uploaded this change for review. ( https://review.gerrithub.io/428390 Change subject: RADOS_URLS: add a namespace parameter to the config block ...................................................................... RADOS_URLS: add a namespace parameter to the config block ...and set it on the ioctx. Change-Id: I076f43ec3ced206b787273455fd2e4e566bc80a2 Signed-off-by: Jeff Layton <jlayton(a)redhat.com> --- M src/config_parsing/conf_url_rados.c 1 file changed, 6 insertions(+), 0 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/90/428390/1 -- To view, visit https://review.gerrithub.io/428390 To unsubscribe, or for help writing mail filters, visit https://review.gerrithub.io/settings Gerrit-Project: ffilz/nfs-ganesha Gerrit-Branch: next Gerrit-MessageType: newchange Gerrit-Change-Id: I076f43ec3ced206b787273455fd2e4e566bc80a2 Gerrit-Change-Number: 428390 Gerrit-PatchSet: 1 Gerrit-Owner: Jeff Layton <jlayton(a)redhat.com>

1 0

Change in ffilz/nfs-ganesha[next]: nfs_main: fix uninitialised log_path and dump_trace
by GerritHub 07 Oct '18

07 Oct '18

From Kinglong Mee <kinglongmee(a)gmail.com>: Kinglong Mee has uploaded this change for review. ( https://review.gerrithub.io/428316 Change subject: nfs_main: fix uninitialised log_path and dump_trace ...................................................................... nfs_main: fix uninitialised log_path and dump_trace ==13376== Conditional jump or move depends on uninitialised value(s) ==13376== at 0x52DE43: init_logging (log_functions.c:1152) ==13376== by 0x445E1F: nfs_prereq_init (nfs_init.c:295) ==13376== by 0x41F0BB: main (nfs_main.c:262) ==13376== ==13376== Conditional jump or move depends on uninitialised value(s) ==13376== at 0x445E24: nfs_prereq_init (nfs_init.c:296) ==13376== by 0x41F0BB: main (nfs_main.c:262) Change-Id: I29c12b140de94ec7731480b2e518ecce9b15f102 Signed-off-by: Kinglong Mee <kinglongmee(a)gmail.com> --- M src/MainNFSD/nfs_main.c 1 file changed, 2 insertions(+), 2 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/16/428316/1 -- To view, visit https://review.gerrithub.io/428316 To unsubscribe, or for help writing mail filters, visit https://review.gerrithub.io/settings Gerrit-Project: ffilz/nfs-ganesha Gerrit-Branch: next Gerrit-MessageType: newchange Gerrit-Change-Id: I29c12b140de94ec7731480b2e518ecce9b15f102 Gerrit-Change-Number: 428316 Gerrit-PatchSet: 1 Gerrit-Owner: Kinglong Mee <kinglongmee(a)gmail.com>

1 0

libcephfs / librados namespace support for nfs-ganesha
by Stefan Kooman 07 Oct '18

07 Oct '18

Dear devs (ceph-devel / nfs-ganesha-devel), Matt Benjamin asked me in this github issue [1] to send an email to the dev mailing lists to discuss a feature [1]. As this feature needs support from both "(lib)cephfs", librados as well as nfs-ganesha I'm sending it to both lists. TL;DR: Add support for nfs-ganesha to make use of 1) librados namespace, 2) (lib)cephfs namespace in order to avoid seperate pools for each nfs-ganesha instance. Motivation: As far as I'm aware of libcephfs / librados support "namespaces". Note: namespaces in this context are not "multiple filesystems" but rather an "object prefix" to separate different objects in cephfs / RADOS. One of the benefits of "namespaces" is that you can have multiple instances of ganesha using the same pool / filesystem without objects clashing. Currently you would need a separate pool for "RADOS_KV" and "RADOS_URLS" per nfs-ganesha instance. With namespace support in libcephfs you would be able to logically seperate the objects per namespace: one for each nfs-ganesha instance ... and restrict access to only this namespace instead of the whole pool: caps: [mds] allow rw path=/path/on/cephfs caps: [mon] allow r caps: [osd] allow rw pool=cephfs_data namespace=your-name-space-here As a (Ceph) storage admin you would be able to provide tenants the (cephx) capabilities to use nfs-ganesha, without the need to give them rw access to your cephfs_data pool(s). As it turns out the "librados" support is already present in nfs-ganesha 2.7 [2]. However, only when you are using the CEPH FSAL in "active/active" mode. It would be nice to "backport" / reuse this code to be able to use it with the "rados_ng" RADOS_KV recovery backend and / or RADOS_URLS. This would be useful for those that (still) want to use a "active/passive" setup (CTDB, pacemaker / corosync). The "libcephfs" namespace support is not yet implemented in nfs-ganesha AFAIK. This would provide the benefits as discussed in motivation. With adding support for "namespaces" it opens the way for nfs-ganesha multi-tenancy. Does this makes sense? Do you agree on the use case? Would this indeed need support from both ceph as well as nfs-ganesha developers? Thanks, Stefan Kooman [1]: https://github.com/nfs-ganesha/nfs-ganesha/issues/353 [2]: https://github.com/nfs-ganesha/nfs-ganesha/blob/next/src/doc/man/ganesha-ra… -- | BIT BV http://www.bit.nl/ Kamer van Koophandel 09090351 | GPG: 0xD14839C6 +31 318 648 688 / info(a)bit.nl

2 10

ceph_ll_fallocate. No ALLOCATE or DEALLOCATE support!
by Stefan Kooman 06 Oct '18

06 Oct '18

Hi list, When building nfs-ganesha 2.7 I see the following warning messages: -- Looking for ceph_ll_fallocate in cephfs - not found Cannot find ceph_ll_fallocate. No ALLOCATE or DEALLOCATE support! -- Looking for ceph_abort_conn in cephfs -- Looking for ceph_abort_conn in cephfs - not found Cannot find ceph_abort_conn. FSAL_CEPH will not leave session intact on clean shutdown. libcephfs-dev is installed (Mimic 13.2.2) ... What version of libcephfs-dev does nfs-ganesha-2.7 requires to build support for ceph_ll_fallocate? Thanks, Stefan -- | BIT BV http://www.bit.nl/ Kamer van Koophandel 09090351 | GPG: 0xD14839C6 +31 318 648 688 / info(a)bit.nl

2 2

Announce Push of V2.7.0.3
by Frank Filz 05 Oct '18

05 Oct '18

Branch next Tag:V2.7.0.3 NOTE: This merge includes a libntirpc pullup, please update your subodules Release Highlights * Pull up to ntirpc 1.7.1 * fixes for readdir chunking * FSAL_CEPH: use ceph_ll_sync_inode to handle COMMIT call when available * NFS4: don't set cid_server_owner more than once * two RECLAIM_COMPLETE fixes * Test status before freeing response for TEST_STATEID and XATTR funcs * nfs4_Compound.c: Document requirements for free_res funcs and sa_cachethis * 4 little fixes for fallocate and seek2 * Free group name string properly in proc_export() * Create 9P worker threads only if 9P is configured * 3 FSAL_PROXY fixes * namespace cleanup: some poorly named and/or non-static symbols Signed-off-by: Frank S. Filz <ffilzlnx(a)mindspring.com> Contents: 2803e13 Frank S. Filz V2.7.0.3 ca9551f Daniel Gryniewicz Pull up to ntirpc 1.7.1 4591a15 Kaleb S. KEITHLEY namespace cleanup: some poorly named and/or non-static symbols cabda8f Dominique Martinet fsal proxy: update config sample aadf499 Dominique Martinet fsal proxy: handle exporting remote / 0a52f7f Dominique Martinet fsal proxy: fix use-after-scope in create session 6d342c3 Malahal Naineni Create 9P worker threads only if 9P is configured 1b65dbb Malahal Naineni Free group name string properly in proc_export() 32b67bb Frank S. Filz MDCACHE: fallocate needs to invalidate attributes 10f1bdd Frank S. Filz FSAL_GLUSTER: for seek2, offset == filesize is "beyond the end of file" 0760db3 Frank S. Filz FSAL_GLUSTER: Allow seek2 on any open file not just open for read fd606f0 Frank S. Filz Fix ERR_FSAL_UNION_NOTSUPP to match V4.2 RFC 93501a5 Frank S. Filz nfs4_Compound.c: Document requirements for free_res funcs and sa_cachethis 38fda66 Frank S. Filz Test status before freeing response for TEST_STATEID and XATTR funcs 2fc1eeb Jeff Layton NFS: clean up and simplify RECLAIM_COMPLETE handling 5c7148a Jeff Layton SAL: decrement reclaim_completes counter when expiring client f6ee41e Jeff Layton NFS4: don't set cid_server_owner more than once 801f51f Jeff Layton FSAL_CEPH: use ceph_ll_sync_inode to handle COMMIT call when available ecd88d6 Daniel Gryniewicz MDCACHE - Fix readdir chunking when reloading chunks dfc921f Daniel Gryniewicz MDCACHE - Update tracepoints to print obj_handle f2ed41f Daniel Gryniewicz GTest - Add second readdir to readdir_correctness f4c3ac6 Daniel Gryniewicz readdir_correctness - Empty cache during run 8ec3e02 Daniel Gryniewicz MEM - Seek to dir location 483a62f Daniel Gryniewicz gtest - Add dirent name logging to readdir_correctness

1 0

Change in ffilz/nfs-ganesha[next]: nfs4_Compound.c: Document requirements for free_res funcs and sa_cach...
by GerritHub 05 Oct '18

05 Oct '18

From Frank Filz <ffilzlnx(a)mindspring.com>: Frank Filz has uploaded this change for review. ( https://review.gerrithub.io/428246 Change subject: nfs4_Compound.c: Document requirements for free_res funcs and sa_cachethis ...................................................................... nfs4_Compound.c: Document requirements for free_res funcs and sa_cachethis We got caught out by a free_res function assuming it would only be called with a valid dynamic data pointer. To hopefully prevent this in the future add some documentation of how the free_res function is used. Change-Id: I883d9c71ca98d68ae06c6070ee6e47d7d4f8ca03 Signed-off-by: Frank S. Filz <ffilzlnx(a)mindspring.com> --- M src/Protocols/NFS/nfs4_Compound.c 1 file changed, 56 insertions(+), 0 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/46/428246/1 -- To view, visit https://review.gerrithub.io/428246 To unsubscribe, or for help writing mail filters, visit https://review.gerrithub.io/settings Gerrit-Project: ffilz/nfs-ganesha Gerrit-Branch: next Gerrit-MessageType: newchange Gerrit-Change-Id: I883d9c71ca98d68ae06c6070ee6e47d7d4f8ca03 Gerrit-Change-Number: 428246 Gerrit-PatchSet: 1 Gerrit-Owner: Frank Filz <ffilzlnx(a)mindspring.com>

1 0

Change in ffilz/nfs-ganesha[next]: Pull up to ntirpc 1.7.1
by GerritHub 05 Oct '18

05 Oct '18

From Daniel Gryniewicz <dang(a)redhat.com>: Daniel Gryniewicz has uploaded this change for review. ( https://review.gerrithub.io/428232 Change subject: Pull up to ntirpc 1.7.1 ...................................................................... Pull up to ntirpc 1.7.1 Change-Id: I885f50818890660bf1cc074d15be02ec1bae73ce Signed-off-by: Daniel Gryniewicz <dang(a)redhat.com> --- M src/libntirpc 1 file changed, 1 insertion(+), 1 deletion(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/32/428232/1 -- To view, visit https://review.gerrithub.io/428232 To unsubscribe, or for help writing mail filters, visit https://review.gerrithub.io/settings Gerrit-Project: ffilz/nfs-ganesha Gerrit-Branch: next Gerrit-MessageType: newchange Gerrit-Change-Id: I885f50818890660bf1cc074d15be02ec1bae73ce Gerrit-Change-Number: 428232 Gerrit-PatchSet: 1 Gerrit-Owner: Daniel Gryniewicz <dang(a)redhat.com>

1 0