Change in ffilz/nfs-ganesha[next]: Allow exports to be referral points in FSAL_VFS
by GerritHub
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>
6 years, 2 months
Change in ffilz/nfs-ganesha[next]: config_samples: update the ceph.conf example config
by GerritHub
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>
6 years, 2 months
Change in ffilz/nfs-ganesha[next]: doc: clean up ganesha-rados-grace document
by GerritHub
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>
6 years, 2 months
Change in ffilz/nfs-ganesha[next]: RADOS_URLS: add a namespace parameter to the config block
by GerritHub
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>
6 years, 2 months
Change in ffilz/nfs-ganesha[next]: nfs_main: fix uninitialised log_path and dump_trace
by GerritHub
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>
6 years, 2 months
libcephfs / librados namespace support for nfs-ganesha
by Stefan Kooman
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-...
--
| BIT BV http://www.bit.nl/ Kamer van Koophandel 09090351
| GPG: 0xD14839C6 +31 318 648 688 / info(a)bit.nl
6 years, 2 months
ceph_ll_fallocate. No ALLOCATE or DEALLOCATE support!
by Stefan Kooman
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
6 years, 2 months
Announce Push of V2.7.0.3
by Frank Filz
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
6 years, 2 months
Change in ffilz/nfs-ganesha[next]: nfs4_Compound.c: Document requirements for free_res funcs and sa_cach...
by GerritHub
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>
6 years, 2 months
Change in ffilz/nfs-ganesha[next]: Pull up to ntirpc 1.7.1
by GerritHub
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>
6 years, 2 months