From 66224718ab57238e16b30d8b461c380d64df237c Mon Sep 17 00:00:00 2001 From: Amar Tumballi Date: Thu, 25 Jun 2009 16:11:44 +0000 Subject: version check between client/server made more specific to protocol only. This patch is a step towards giving compatibility between the versions of GlusterFS. Now onwards, the protocol-version won't depend on release versions. In general, multiple glusterfs versions can have common protocol-version. Signed-off-by: Anand V. Avati --- xlators/protocol/client/src/client-protocol.c | 10 +++++----- xlators/protocol/server/src/server-protocol.c | 8 ++++---- 2 files changed, 9 insertions(+), 9 deletions(-) (limited to 'xlators') diff --git a/xlators/protocol/client/src/client-protocol.c b/xlators/protocol/client/src/client-protocol.c index 261027648b8..096713687b6 100644 --- a/xlators/protocol/client/src/client-protocol.c +++ b/xlators/protocol/client/src/client-protocol.c @@ -6125,11 +6125,11 @@ protocol_client_handshake (xlator_t *this, transport_t *trans) char *process_uuid_xl; options = this->options; - ret = dict_set_str (options, "version", PACKAGE_VERSION); + ret = dict_set_str (options, "protocol-version", GF_PROTOCOL_VERSION); if (ret < 0) { gf_log (this->name, GF_LOG_DEBUG, - "failed to set version(%s) in options dictionary", - PACKAGE_VERSION); + "failed to set protocol version(%s) in handshake msg", + GF_PROTOCOL_VERSION); } asprintf (&process_uuid_xl, "%s-%s", this->ctx->process_uuid, @@ -6138,8 +6138,8 @@ protocol_client_handshake (xlator_t *this, transport_t *trans) process_uuid_xl); if (ret < 0) { gf_log (this->name, GF_LOG_DEBUG, - "failed to set process-uuid(%s) in options dictionary", - PACKAGE_VERSION); + "failed to set process-uuid(%s) in handshake msg", + process_uuid_xl); } if (this->ctx->cmd_args.volfile_server) { diff --git a/xlators/protocol/server/src/server-protocol.c b/xlators/protocol/server/src/server-protocol.c index 6faf0a5171b..accd8358f3f 100644 --- a/xlators/protocol/server/src/server-protocol.c +++ b/xlators/protocol/server/src/server-protocol.c @@ -6955,7 +6955,7 @@ mop_setvolume (call_frame_t *frame, xlator_t *bound_xl, if (trans->xl_private != conn) trans->xl_private = conn; - ret = dict_get_str (params, "version", &version); + ret = dict_get_str (params, "protocol-version", &version); if (ret < 0) { ret = dict_set_str (reply, "ERROR", "No version number specified"); @@ -6968,11 +6968,11 @@ mop_setvolume (call_frame_t *frame, xlator_t *bound_xl, goto fail; } - ret = strcmp (version, PACKAGE_VERSION); + ret = strcmp (version, GF_PROTOCOL_VERSION); if (ret != 0) { asprintf (&msg, - "Version mismatch: client(%s) Vs server (%s)", - version, PACKAGE_VERSION); + "protocol version mismatch: client(%s) - server(%s)", + version, GF_PROTOCOL_VERSION); ret = dict_set_dynstr (reply, "ERROR", msg); if (ret < 0) gf_log (trans->xl->name, GF_LOG_DEBUG, -- cgit