Skip to content

Commit

Permalink
TESTS/TIER: bug-1303028-Rebalance-glusterd-rpc-connection-issue.t
Browse files Browse the repository at this point in the history
PROBLEM: spurious failure of the test.

CAUSE: the function "rebalance_run_time" calculates the total time
the tier has been running for. this being a test case, the run time
of tier can be 0 and when the function adds up zero it results in
zero. and thus it starts to fail.

FIX: Give it some time for the function to add up the values.

Signed-off-by: hari gowtham <hgowtham@redhat.com>

Change-Id: Ie270f3f3c8942081cca85dc49ef8fec76f3a261a
BUG: 1425743
Reviewed-on: https://review.gluster.org/16711
Smoke: Gluster Build System <jenkins@build.gluster.org>
Tested-by: hari gowtham <hari.gowtham005@gmail.com>
NetBSD-regression: NetBSD Build System <jenkins@build.gluster.org>
CentOS-regression: Gluster Build System <jenkins@build.gluster.org>
Reviewed-by: Shyamsundar Ranganathan <srangana@redhat.com>
  • Loading branch information
harigowtham authored and ShyamsundarR committed Feb 23, 2017
1 parent 583e6cf commit a88cb01
Showing 1 changed file with 2 additions and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -68,9 +68,10 @@ sleep 2;
# Wait for the bricks to come up, *then* the tier daemon.
EXPECT_WITHIN $PROCESS_UP_TIMEOUT 5 num_bricks_up
EXPECT_WITHIN $PROCESS_UP_TIMEOUT "0" tier_daemon_check;
sleep 1;
time1=$(rebalance_run_time $V0);
EXPECT "0" non_zero_check $time1;
sleep 4;
sleep 2;
time2=$(rebalance_run_time $V0);
EXPECT "0" non_zero_check $time2;
diff=`expr $time2 - $time1`
Expand Down

0 comments on commit a88cb01

Please sign in to comment.