diff options
author | Avra Sengupta <asengupt@redhat.com> | 2014-05-06 04:19:48 +0000 |
---|---|---|
committer | Kaushal M <kaushal@redhat.com> | 2014-05-08 03:44:41 -0700 |
commit | e80daecc3dd3023f010c785ff0e238fc92214477 (patch) | |
tree | fd6f8a4f11a40cdcca4018e6013beda6a84d4bd8 /tests | |
parent | b47b46d98a5e1d3e6e9eb9bde4f81b745f2ac09d (diff) |
glusterd: Use a calloc-ed copy of txn_id for glusterd_do_replace_brick
As glusterd_do_replace_brick() is spawned through gf_timer_call_after(),
by the time it's called the event is freed, and the txn_id is lost.
Hence using a calloc-ed copy, which will be freed as a part of rb_ctx
dict.
Change-Id: I3e309fe1a7ba96ad1d1ce01f4d2aa18178f59244
BUG: 1095097
Signed-off-by: Avra Sengupta <asengupt@redhat.com>
Reviewed-on: http://review.gluster.org/7686
Reviewed-by: Krishnan Parthasarathi <kparthas@redhat.com>
Tested-by: Gluster Build System <jenkins@build.gluster.com>
Reviewed-by: Kaushal M <kaushal@redhat.com>
Diffstat (limited to 'tests')
-rwxr-xr-x | tests/bugs/bug-1095097.t | 21 |
1 files changed, 21 insertions, 0 deletions
diff --git a/tests/bugs/bug-1095097.t b/tests/bugs/bug-1095097.t new file mode 100755 index 00000000000..9f9db7a3df5 --- /dev/null +++ b/tests/bugs/bug-1095097.t @@ -0,0 +1,21 @@ +#!/bin/bash + +. $(dirname $0)/../include.rc + +cleanup; +TEST glusterd; +TEST pidof glusterd; +TEST $CLI volume info; + +TEST $CLI volume create $V0 $H0:$B0/brick1 $H0:$B1/brick1; +EXPECT 'Created' volinfo_field $V0 'Status'; + +TEST $CLI volume start $V0; +EXPECT 'Started' volinfo_field $V0 'Status'; + +TEST $CLI volume profile $V0 start +TEST $CLI volume profile $V0 info +TEST $CLI volume replace-brick $V0 $H0:$B0/brick1 $H0:$B0/brick2 start +TEST $CLI volume replace-brick $V0 $H0:$B0/brick1 $H0:$B0/brick2 status + +cleanup; |