summaryrefslogtreecommitdiffstats
path: root/tests
diff options
context:
space:
mode:
authorAtin Mukherjee <amukherj@redhat.com>2017-04-11 09:25:48 +0530
committerRaghavendra Talur <rtalur@redhat.com>2017-04-27 10:52:53 +0000
commit0a59320775ec44c472f9027b0838ad162e0e4f6c (patch)
tree973d2ce87a35443f260831641a5703ec29f72513 /tests
parentcb961e3beb70542d8aced3e33e5902fbd2ae69ae (diff)
tests: remove tests/bugs/core/bug-1421590-brick-mux-reuse-ports.t
bug-1421590-brick-mux-reuse-ports.t seems to be a bad test to me and here is my reasoning: This test tries to check if the ports are reused or not. When a volume is restarted, by the time glusterd tries to allocate a new port to the one of the brick processes of the volume there is no guarantee that the older port will be allocated given the kernel might take some extra time to free up the port between this time frame. From https://build.gluster.org/job/regression-test-burn-in/2932/console we can clearly see that post restart of the volume, glusterd allocated port 49153 & 49155 for brick1 & brick2 respectively but the test was expecting the ports to be matched with 49155 & 49156 which were allocated before the volume was restarted. >Reviewed-on: https://review.gluster.org/17033 >Smoke: Gluster Build System <jenkins@build.gluster.org> >NetBSD-regression: NetBSD Build System <jenkins@build.gluster.org> >CentOS-regression: Gluster Build System <jenkins@build.gluster.org> >Reviewed-by: Amar Tumballi <amarts@redhat.com> >(cherry picked from commit 1612355327fa5f86078b9dbcf7a38e4e0c63e205) Change-Id: Id887bf28445261d4de04fc7502e58057659c9512 BUG: 1445407 Signed-off-by: Atin Mukherjee <amukherj@redhat.com> Reviewed-on: https://review.gluster.org/17116 Smoke: Gluster Build System <jenkins@build.gluster.org> NetBSD-regression: NetBSD Build System <jenkins@build.gluster.org> CentOS-regression: Gluster Build System <jenkins@build.gluster.org> Reviewed-by: Raghavendra Talur <rtalur@redhat.com>
Diffstat (limited to 'tests')
-rw-r--r--tests/bugs/core/bug-1421590-brick-mux-reuse-ports.t60
1 files changed, 0 insertions, 60 deletions
diff --git a/tests/bugs/core/bug-1421590-brick-mux-reuse-ports.t b/tests/bugs/core/bug-1421590-brick-mux-reuse-ports.t
deleted file mode 100644
index a227f8275ed..00000000000
--- a/tests/bugs/core/bug-1421590-brick-mux-reuse-ports.t
+++ /dev/null
@@ -1,60 +0,0 @@
-#!/bin/bash
-
-. $(dirname $0)/../../include.rc
-. $(dirname $0)/../../traps.rc
-. $(dirname $0)/../../volume.rc
-
-function get_nth_brick_port_for_volume () {
- local VOL=$1
- local n=$2
-
- $CLI volume status $VOL --xml | sed -ne 's/.*<port>\([-0-9]*\)<\/port>/\1/p' \
- | head -n $n | tail -n 1
-}
-
-TEST glusterd
-
-TEST $CLI volume set all cluster.brick-multiplex on
-push_trapfunc "$CLI volume set all cluster.brick-multiplex off"
-push_trapfunc "cleanup"
-
-TEST $CLI volume create $V0 $H0:$B0/brick{0,1}
-TEST $CLI volume start $V0
-
-# We can't expect a valid port number instantly. We need to wait for the
-# bricks to finish coming up. In every other case we use EXPECT_WITHIN, but
-# this first time we need to wait more explicitly.
-sleep $PROCESS_UP_TIMEOUT
-
-port_brick0=$(get_nth_brick_port_for_volume $V0 1)
-
-# restart the volume
-TEST $CLI volume stop $V0
-TEST $CLI volume start $V0
-
-EXPECT_WITHIN $PROCESS_UP_TIMEOUT $port_brick0 get_nth_brick_port_for_volume $V0 1
-
-TEST $CLI volume stop $V0
-TEST $CLI volume set all cluster.brick-multiplex off
-
-TEST $CLI volume start $V0
-
-EXPECT_WITHIN $PROCESS_UP_TIMEOUT $port_brick0 get_nth_brick_port_for_volume $V0 1
-
-port_brick1=$(get_nth_brick_port_for_volume $V0 2)
-
-# restart the volume
-TEST $CLI volume stop $V0
-TEST $CLI volume start $V0
-
-EXPECT_WITHIN $PROCESS_UP_TIMEOUT $port_brick0 get_nth_brick_port_for_volume $V0 1
-EXPECT_WITHIN $PROCESS_UP_TIMEOUT $port_brick1 get_nth_brick_port_for_volume $V0 2
-
-TEST $CLI volume stop $V0
-
-TEST $CLI volume set all cluster.brick-multiplex on
-
-TEST $CLI volume start $V0
-
-EXPECT_WITHIN $PROCESS_UP_TIMEOUT $port_brick0 get_nth_brick_port_for_volume $V0 1
-