diff options
| author | ShyamsundarR <srangana@redhat.com> | 2018-08-11 13:21:36 -0400 |
|---|---|---|
| committer | Atin Mukherjee <amukherj@redhat.com> | 2018-08-13 03:00:48 +0000 |
| commit | 48b93c292c0069da9ac2fe77e66d08a1cdeacfdc (patch) | |
| tree | f2567d48fe45bde861db28328740d2948fbda53f /tests/basic/multiplex.t | |
| parent | 488dcc17b76b1b46c13ccd269b7a092c014a4d94 (diff) | |
| download | glusterfs-48b93c292c0069da9ac2fe77e66d08a1cdeacfdc.tar.gz glusterfs-48b93c292c0069da9ac2fe77e66d08a1cdeacfdc.tar.xz glusterfs-48b93c292c0069da9ac2fe77e66d08a1cdeacfdc.zip | |
tests: Fix cleanup routine for some mux tests
Some of the mux tests, set a trap to catch test exit and
call cleanup. This will cause cleanup to be invoked twice
in case the test times out, or even otherwise, as include.rc
also sets a trap to cleanup on exit (TERM and others).
This leads to the tarballs generated on failures for these
tests to be empty and does not aid debugging.
This patch corrects this pattern across the tests to the
more standard cleanup at the end.
Fixes: bz#1615037
Change-Id: Ib83aeb09fac2aa591b390b9fb9e1f605bfef9a8b
Signed-off-by: ShyamsundarR <srangana@redhat.com>
Diffstat (limited to 'tests/basic/multiplex.t')
| -rw-r--r-- | tests/basic/multiplex.t | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/tests/basic/multiplex.t b/tests/basic/multiplex.t index 2f0f462f14..2f558a6824 100644 --- a/tests/basic/multiplex.t +++ b/tests/basic/multiplex.t @@ -21,8 +21,6 @@ cleanup 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 set $V0 features.trash enable @@ -77,3 +75,4 @@ TEST glusterd EXPECT_WITHIN $PROCESS_DOWN_TIMEOUT 1 count_brick_pids EXPECT_WITHIN $PROCESS_DOWN_TIMEOUT 1 count_brick_processes +cleanup;
\ No newline at end of file |