From 6155743181a4f4cd2b0c789d90739ae12013431b Mon Sep 17 00:00:00 2001 From: hari gowtham Date: Thu, 18 May 2017 14:15:39 +0530 Subject: Tier: removing port allocated for tier Problem: Tier has a port which it doesn't use. Fix: Remove the port getting allocated for tier. >Change-Id: If0fe393fc335d9f622a063787e0a3c6db9b7a50c >BUG: 1452006 >Signed-off-by: hari gowtham >Reviewed-on: https://review.gluster.org/17328 >Tested-by: hari gowtham >NetBSD-regression: NetBSD Build System >CentOS-regression: Gluster Build System >Smoke: Gluster Build System >Reviewed-by: Shyamsundar Ranganathan Change-Id: If0fe393fc335d9f622a063787e0a3c6db9b7a50c BUG: 1457289 Signed-off-by: hari gowtham Reviewed-on: https://review.gluster.org/17428 Smoke: Gluster Build System Tested-by: hari gowtham CentOS-regression: Gluster Build System NetBSD-regression: NetBSD Build System Reviewed-by: Shyamsundar Ranganathan --- xlators/mgmt/glusterd/src/glusterd-tier.c | 6 ------ xlators/mgmt/glusterd/src/glusterd-tierd-svc.c | 28 -------------------------- xlators/mgmt/glusterd/src/glusterd-tierd-svc.h | 1 - 3 files changed, 35 deletions(-) (limited to 'xlators') diff --git a/xlators/mgmt/glusterd/src/glusterd-tier.c b/xlators/mgmt/glusterd/src/glusterd-tier.c index 5f9a08f1909..59a4bc1b3e0 100644 --- a/xlators/mgmt/glusterd/src/glusterd-tier.c +++ b/xlators/mgmt/glusterd/src/glusterd-tier.c @@ -1177,12 +1177,6 @@ glusterd_add_tierd_to_dict (glusterd_volinfo_t *volinfo, if (ret) goto out; - memset (key, 0, sizeof (key)); - snprintf (key, sizeof (key), "%s.port", base_key); - ret = dict_set_int32 (dict, key, volinfo->tierd.port); - if (ret) - goto out; - glusterd_svc_build_tierd_pidfile (volinfo, pidfile, sizeof (pidfile)); brick_online = gf_is_service_running (pidfile, &pid); diff --git a/xlators/mgmt/glusterd/src/glusterd-tierd-svc.c b/xlators/mgmt/glusterd/src/glusterd-tierd-svc.c index 638d4941521..6f53947e540 100644 --- a/xlators/mgmt/glusterd/src/glusterd-tierd-svc.c +++ b/xlators/mgmt/glusterd/src/glusterd-tierd-svc.c @@ -226,7 +226,6 @@ glusterd_tierdsvc_manager (glusterd_svc_t *svc, void *data, int flags) volinfo->volname); goto out; } - volinfo->tierd.port = 0; } out: @@ -244,7 +243,6 @@ glusterd_tierdsvc_start (glusterd_svc_t *svc, int flags) glusterd_conf_t *priv = NULL; xlator_t *this = NULL; char valgrind_logfile[PATH_MAX] = {0}; - int tierd_port = 0; char msg[1024] = {0,}; char tierd_id[PATH_MAX] = {0,}; glusterd_volinfo_t *volinfo = NULL; @@ -345,32 +343,6 @@ glusterd_tierdsvc_start (glusterd_svc_t *svc, int flags) if (volinfo->memory_accounting) runner_add_arg (&runner, "--mem-accounting"); - /* Do a pmap registry remove on the older connected port */ - if (volinfo->tierd.port) { - ret = pmap_registry_remove (this, volinfo->tierd.port, - tierd_id, GF_PMAP_PORT_BRICKSERVER, - NULL); - if (ret) { - snprintf (msg, sizeof (msg), "Failed to remove pmap " - "registry for older signin"); - goto out; - } - } - - - - tierd_port = pmap_registry_alloc (this); - if (!tierd_port) { - snprintf (msg, sizeof (msg), "Could not allocate port " - "for tierd service for volume %s", - volinfo->volname); - runner_log (&runner, this->name, GF_LOG_DEBUG, msg); - ret = -1; - goto out; - } - - volinfo->tierd.port = tierd_port; - snprintf (msg, sizeof (msg), "Starting the tierd service for volume %s", volinfo->volname); runner_log (&runner, this->name, GF_LOG_DEBUG, msg); diff --git a/xlators/mgmt/glusterd/src/glusterd-tierd-svc.h b/xlators/mgmt/glusterd/src/glusterd-tierd-svc.h index dfc63d25eb2..2ff840d0f25 100644 --- a/xlators/mgmt/glusterd/src/glusterd-tierd-svc.h +++ b/xlators/mgmt/glusterd/src/glusterd-tierd-svc.h @@ -18,7 +18,6 @@ typedef struct glusterd_tierdsvc_ glusterd_tierdsvc_t; struct glusterd_tierdsvc_ { glusterd_svc_t svc; - int port; gf_store_handle_t *handle; }; -- cgit