|
 |
14f8ab |
From 172f32058b1a7d2e42f373490853aef5dd72f02f Mon Sep 17 00:00:00 2001
|
|
|
14f8ab |
From: Jiffin Tony Thottan <jthottan@redhat.com>
|
|
|
14f8ab |
Date: Wed, 22 Feb 2017 14:20:41 +0530
|
|
|
14f8ab |
Subject: [PATCH 058/124] glusterd/ganesha : During volume delete remove the
|
|
|
14f8ab |
ganesha export configuration file
|
|
|
14f8ab |
|
|
|
14f8ab |
Label: DOWNSTREAM ONLY
|
|
|
14f8ab |
|
|
|
14f8ab |
Change-Id: I0363e7f4d7cefd3f1b3c4f91e495767ec52e230e
|
|
|
14f8ab |
Signed-off-by: Jiffin Tony Thottan <jthottan@redhat.com>
|
|
|
14f8ab |
Reviewed-on: https://review.gluster.org/16707
|
|
|
14f8ab |
Smoke: Gluster Build System <jenkins@build.gluster.org>
|
|
|
14f8ab |
NetBSD-regression: NetBSD Build System <jenkins@build.gluster.org>
|
|
|
14f8ab |
CentOS-regression: Gluster Build System <jenkins@build.gluster.org>
|
|
|
14f8ab |
Reviewed-by: soumya k <skoduri@redhat.com>
|
|
|
14f8ab |
Reviewed-by: Kaleb KEITHLEY <kkeithle@redhat.com>
|
|
|
14f8ab |
Signed-off-by: Jiffin Tony Thottan <jthottan@redhat.com>
|
|
|
14f8ab |
Reviewed-on: https://code.engineering.redhat.com/gerrit/167143
|
|
|
14f8ab |
Reviewed-by: Soumya Koduri <skoduri@redhat.com>
|
|
|
14f8ab |
Tested-by: RHGS Build Bot <nigelb@redhat.com>
|
|
|
14f8ab |
Reviewed-by: Sunil Kumar Heggodu Gopala Acharya <sheggodu@redhat.com>
|
|
|
14f8ab |
---
|
|
|
14f8ab |
xlators/mgmt/glusterd/src/glusterd-ganesha.c | 2 +-
|
|
|
14f8ab |
xlators/mgmt/glusterd/src/glusterd-volume-ops.c | 9 +++++++++
|
|
|
14f8ab |
2 files changed, 10 insertions(+), 1 deletion(-)
|
|
|
14f8ab |
|
|
|
14f8ab |
diff --git a/xlators/mgmt/glusterd/src/glusterd-ganesha.c b/xlators/mgmt/glusterd/src/glusterd-ganesha.c
|
|
|
14f8ab |
index 81f794d..6d72fda 100644
|
|
|
14f8ab |
--- a/xlators/mgmt/glusterd/src/glusterd-ganesha.c
|
|
|
14f8ab |
+++ b/xlators/mgmt/glusterd/src/glusterd-ganesha.c
|
|
|
14f8ab |
@@ -445,7 +445,7 @@ manage_export_config(char *volname, char *value, char **op_errstr)
|
|
|
14f8ab |
CONFDIR, value, volname, NULL);
|
|
|
14f8ab |
ret = runner_run(&runner);
|
|
|
14f8ab |
|
|
|
14f8ab |
- if (ret)
|
|
|
14f8ab |
+ if (ret && !(*op_errstr))
|
|
|
14f8ab |
gf_asprintf(op_errstr,
|
|
|
14f8ab |
"Failed to create"
|
|
|
14f8ab |
" NFS-Ganesha export config file.");
|
|
|
14f8ab |
diff --git a/xlators/mgmt/glusterd/src/glusterd-volume-ops.c b/xlators/mgmt/glusterd/src/glusterd-volume-ops.c
|
|
|
14f8ab |
index a0417ca..81c668c 100644
|
|
|
14f8ab |
--- a/xlators/mgmt/glusterd/src/glusterd-volume-ops.c
|
|
|
14f8ab |
+++ b/xlators/mgmt/glusterd/src/glusterd-volume-ops.c
|
|
|
14f8ab |
@@ -2936,6 +2936,15 @@ glusterd_op_delete_volume(dict_t *dict)
|
|
|
14f8ab |
goto out;
|
|
|
14f8ab |
}
|
|
|
14f8ab |
|
|
|
14f8ab |
+ if (glusterd_check_ganesha_export(volinfo)) {
|
|
|
14f8ab |
+ ret = manage_export_config(volname, "off", NULL);
|
|
|
14f8ab |
+ if (ret)
|
|
|
14f8ab |
+ gf_msg(this->name, GF_LOG_WARNING, 0, 0,
|
|
|
14f8ab |
+ "Could not delete ganesha export conf file "
|
|
|
14f8ab |
+ "for %s",
|
|
|
14f8ab |
+ volname);
|
|
|
14f8ab |
+ }
|
|
|
14f8ab |
+
|
|
|
14f8ab |
ret = glusterd_delete_volume(volinfo);
|
|
|
14f8ab |
out:
|
|
|
14f8ab |
gf_msg_debug(this->name, 0, "returning %d", ret);
|
|
|
14f8ab |
--
|
|
|
14f8ab |
1.8.3.1
|
|
|
14f8ab |
|