On 03/10/2023 19:57, fdmanana@xxxxxxxxxx wrote:
From: Filipe Manana <fdmanana@xxxxxxxx> After doing log replay, btrfs/192 is overwriting the $seqres.full file because it uses the plain ">" redirect operator, instead of an append ">>" redirect operator. As a consequence it is overriding the file and eliminating any previous output that may be useful to debug a test failure (such as the fsstress seed or mkfs results). So use >> instead of >. Signed-off-by: Filipe Manana <fdmanana@xxxxxxxx>
Reviewed-by: Anand Jain <anand.jain@xxxxxxxxxx>
--- tests/btrfs/192 | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tests/btrfs/192 b/tests/btrfs/192 index 80588a3c..00ea1478 100755 --- a/tests/btrfs/192 +++ b/tests/btrfs/192 @@ -121,7 +121,7 @@ log_writes_fast_replay_check() --replay $blkdev --check $check_point --fsck "$fsck_command" \ &> $tmp.full_fsck ret=$? - tail -n 150 $tmp.full_fsck > $seqres.full + tail -n 150 $tmp.full_fsck >> $seqres.full [ $ret -ne 0 ] && _fail "fsck failed during replay" }
