d1681e
From 084b4754afedff7b3f96f17c6ca19b818b0d5767 Mon Sep 17 00:00:00 2001
d1681e
From: Atin Mukherjee <amukherj@redhat.com>
d1681e
Date: Tue, 2 Jan 2018 20:26:31 +0530
d1681e
Subject: [PATCH 136/139] glusterd: Nullify pmap entry for bricks belonging to
d1681e
 same port
d1681e
d1681e
Commit 30e0b86 tried to address all the stale port issues glusterd had
d1681e
in case of a brick is abruptly killed. For brick multiplexing case
d1681e
because of a bug the portmap entry was not getting removed. This patch
d1681e
addresses the same.
d1681e
d1681e
>upstream mainline patch : https://review.gluster.org/#/c/19119
d1681e
d1681e
Change-Id: Ib020b967a9b92f1abae9cab9492f0cacec59aaa1
d1681e
BUG: 1530325
d1681e
Signed-off-by: Atin Mukherjee <amukherj@redhat.com>
d1681e
Reviewed-on: https://code.engineering.redhat.com/gerrit/128884
d1681e
Tested-by: RHGS Build Bot <nigelb@redhat.com>
d1681e
---
d1681e
 xlators/mgmt/glusterd/src/glusterd-pmap.c | 2 +-
d1681e
 1 file changed, 1 insertion(+), 1 deletion(-)
d1681e
d1681e
diff --git a/xlators/mgmt/glusterd/src/glusterd-pmap.c b/xlators/mgmt/glusterd/src/glusterd-pmap.c
d1681e
index 4f045ab..c9e6c1b 100644
d1681e
--- a/xlators/mgmt/glusterd/src/glusterd-pmap.c
d1681e
+++ b/xlators/mgmt/glusterd/src/glusterd-pmap.c
d1681e
@@ -392,7 +392,7 @@ remove:
d1681e
          * there's no xprt either, then we have nothing left worth saving and
d1681e
          * can delete the entire entry.
d1681e
          */
d1681e
-        if (!pmap->ports[p].xprt) {
d1681e
+        if (brick_disconnect || !pmap->ports[p].xprt) {
d1681e
                 /* If the signout call is being triggered by brick disconnect
d1681e
                  * then clean up all the bricks (in case of brick mux)
d1681e
                  */
d1681e
-- 
d1681e
1.8.3.1
d1681e