summaryrefslogtreecommitdiffstats
path: root/xlators/mgmt
diff options
context:
space:
mode:
authorRaghavendra Bhat <raghavendrabhat@gluster.com>2010-10-26 05:20:34 +0000
committerAnand V. Avati <avati@dev.gluster.com>2010-10-26 23:56:24 -0700
commita7d41a64fb11451e622db19813738c3ee3942c92 (patch)
tree70df39892972fa6929dd2fe9e06713d066c83f1a /xlators/mgmt
parenta53d14d5f5be431279c5e56963b495bf164cf453 (diff)
report the error if a volume not in the started state is tried to be stopped
Signed-off-by: Raghavendra Bhat <raghavendrabhat@gluster.com> Signed-off-by: Anand V. Avati <avati@dev.gluster.com> BUG: 2010 () URL: http://bugs.gluster.com/cgi-bin/bugzilla3/show_bug.cgi?id=2010
Diffstat (limited to 'xlators/mgmt')
-rw-r--r--xlators/mgmt/glusterd/src/glusterd-op-sm.c12
1 files changed, 10 insertions, 2 deletions
diff --git a/xlators/mgmt/glusterd/src/glusterd-op-sm.c b/xlators/mgmt/glusterd/src/glusterd-op-sm.c
index e57acf1d2..6341fd0f3 100644
--- a/xlators/mgmt/glusterd/src/glusterd-op-sm.c
+++ b/xlators/mgmt/glusterd/src/glusterd-op-sm.c
@@ -559,7 +559,7 @@ out:
}
static int
-glusterd_op_stage_stop_volume (gd1_mgmt_stage_op_req *req)
+glusterd_op_stage_stop_volume (gd1_mgmt_stage_op_req *req, char **op_errstr)
{
int ret = -1;
dict_t *dict = NULL;
@@ -567,6 +567,7 @@ glusterd_op_stage_stop_volume (gd1_mgmt_stage_op_req *req)
int flags = 0;
gf_boolean_t exists = _gf_false;
glusterd_volinfo_t *volinfo = NULL;
+ char msg[2048] = {0,};
dict = dict_new ();
if (!dict)
@@ -595,8 +596,11 @@ glusterd_op_stage_stop_volume (gd1_mgmt_stage_op_req *req)
ret = glusterd_is_volume_started (volinfo);
if (ret) {
+ snprintf (msg, sizeof(msg), "Volume %s "
+ "is not in the started state", volname);
gf_log ("", GF_LOG_ERROR, "Volume %s "
"has not been started", volname);
+ *op_errstr = gf_strdup (msg);
goto out;
}
}
@@ -4408,6 +4412,10 @@ glusterd_op_send_cli_response (int32_t op, int32_t op_ret,
rsp.op_ret = op_ret;
rsp.op_errno = op_errno;
rsp.volname = "";
+ if (op_errstr)
+ rsp.op_errstr = op_errstr;
+ else
+ rsp.op_errstr = "";
cli_rsp = &rsp;
sfunc = gf_xdr_serialize_cli_stop_vol_rsp;
break;
@@ -4795,7 +4803,7 @@ glusterd_op_stage_validate (gd1_mgmt_stage_op_req *req, char **op_errstr,
break;
case GD_OP_STOP_VOLUME:
- ret = glusterd_op_stage_stop_volume (req);
+ ret = glusterd_op_stage_stop_volume (req, op_errstr);
break;
case GD_OP_DELETE_VOLUME: