summaryrefslogtreecommitdiffstats
path: root/xlators/mgmt
diff options
context:
space:
mode:
authorAtin Mukherjee <amukherj@redhat.com>2015-08-05 16:02:49 +0530
committerAtin Mukherjee <amukherj@redhat.com>2016-01-27 19:50:45 -0800
commitfd16186b403e38763b1d0b39c8c7ce14ffc508ba (patch)
treec98731ff76bf65ca42d45eb96319d2ed23e1d526 /xlators/mgmt
parenteca923349b4cb397b19520e4e391e42b8d6532c2 (diff)
glusterd: improve error logs for unsupported clients
Change-Id: I90be46a7b25cc3efd0e61c9a145852e9027f5f64 BUG: 1302205 Signed-off-by: Atin Mukherjee <amukherj@redhat.com> Reviewed-on: http://review.gluster.org/11831 Reviewed-by: Pranith Kumar Karampuri <pkarampu@redhat.com> Tested-by: Pranith Kumar Karampuri <pkarampu@redhat.com> Smoke: Gluster Build System <jenkins@build.gluster.com> CentOS-regression: Gluster Build System <jenkins@build.gluster.com> Reviewed-by: Prashanth Pai <ppai@redhat.com> Reviewed-by: Kaushal M <kaushal@redhat.com> NetBSD-regression: NetBSD Build System <jenkins@build.gluster.org>
Diffstat (limited to 'xlators/mgmt')
-rw-r--r--xlators/mgmt/glusterd/src/glusterd-utils.c22
1 files changed, 15 insertions, 7 deletions
diff --git a/xlators/mgmt/glusterd/src/glusterd-utils.c b/xlators/mgmt/glusterd/src/glusterd-utils.c
index f129b1cb04f..5f3bd76039d 100644
--- a/xlators/mgmt/glusterd/src/glusterd-utils.c
+++ b/xlators/mgmt/glusterd/src/glusterd-utils.c
@@ -10808,14 +10808,22 @@ glusterd_check_client_op_version_support (char *volname, uint32_t op_version,
if (ret) {
gf_msg (this->name, GF_LOG_ERROR, 0,
GD_MSG_UNSUPPORTED_VERSION,
- "One or more clients "
- "don't support the required op-version");
+ "Client %s is running with min_op_version as %d and "
+ "max_op_version as %d and don't support the required "
+ "op-version %d", xprt->peerinfo.identifier,
+ xprt->peerinfo.min_op_version,
+ xprt->peerinfo.max_op_version, op_version);
if (op_errstr)
- ret = gf_asprintf (op_errstr, "One or more connected "
- "clients cannot support the feature "
- "being set. These clients need to be"
- " upgraded or disconnected before "
- "running this command again");
+ ret = gf_asprintf (op_errstr, "One of the client %s is "
+ "running with op-version %d and "
+ "doesn't support the required "
+ "op-version %d. This client needs to"
+ " be upgraded or disconnected "
+ "before running this command again",
+ xprt->peerinfo.identifier,
+ xprt->peerinfo.max_op_version,
+ op_version);
+
return -1;
}
return 0;