diff options
author | Shehjar Tikoo <shehjart@gluster.com> | 2011-02-04 03:14:37 +0000 |
---|---|---|
committer | Anand V. Avati <avati@dev.gluster.com> | 2011-02-04 11:21:34 -0800 |
commit | ed743d89affe8be2d9d7da87479a498af40e12b0 (patch) | |
tree | c34fc6adeabafc359dd14f1708d6a0ddb8e4baaf /xlators | |
parent | 44e68cbf833cc176d589780d47ffdbe5b4c3ab7c (diff) |
mount3: Do not return error when mount list is empty
An exports list can be empty when no subvolumes have come up.
No point returning error and confusing the user.
Signed-off-by: Shehjar Tikoo <shehjart@gluster.com>
Signed-off-by: Anand V. Avati <avati@dev.gluster.com>
BUG: 2345 (Taking distribute brick down returns 10006 error on mount command)
URL: http://bugs.gluster.com/cgi-bin/bugzilla3/show_bug.cgi?id=2345
Diffstat (limited to 'xlators')
-rw-r--r-- | xlators/nfs/server/src/mount3.c | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/xlators/nfs/server/src/mount3.c b/xlators/nfs/server/src/mount3.c index 53282d9a191..295b4e830d8 100644 --- a/xlators/nfs/server/src/mount3.c +++ b/xlators/nfs/server/src/mount3.c @@ -1364,11 +1364,15 @@ mnt3svc_export (rpcsvc_request_t *req) /* Using the children translator names, build the export list */ elist = mnt3_xlchildren_to_exports (nfs_rpcsvc_request_service (req), ms); + /* Do not return error when exports list is empty. An exports list can + * be empty when no subvolumes have come up. No point returning error + * and confusing the user. if (!elist) { gf_log (GF_MNT, GF_LOG_ERROR, "Failed to build exports list"); nfs_rpcsvc_request_seterr (req, SYSTEM_ERR); goto err; } + */ /* Note how the serializer is passed to the generic reply function. */ mnt3svc_submit_reply (req, &elist, |