[PATCH] test-appliance: revert logging change on startup

[Date Prev] [Date Next] [Thread Prev] [Thread Next] [Date Index] [Thread Index]



From: Eric Biggers <ebiggers@xxxxxxxxxx>

Commit 458ce3391003 ("Add support to optionally build gce-xfstests for
Debian stretch") made the output of kvm-xfstests.boot get redirected to
a file /var/log/kvm-xfstests-boot.log.  This actually isn't desirable,
because the output is supposed to be shown on the console.  This was
especially apparent for kvm-xfstests, since it hid the output of
runtests.sh.  But it was also applicable to gce-xfstests, since
'gce-xfstests console' can be used to show the console output.

We could send the output to both places using 'tee', but this seems
useless since the output can just be retrieved from the console instead
of from the log file.  Also, the 'set -vx' output is too verbose for the
console, and I don't think there would be a way to send it just to the
log file.

So just revert the change to kvm-xfstests.boot.

Signed-off-by: Eric Biggers <ebiggers@xxxxxxxxxx>
---
 kvm-xfstests/test-appliance/files/root/kvm-xfstests.boot | 7 -------
 1 file changed, 7 deletions(-)

diff --git a/kvm-xfstests/test-appliance/files/root/kvm-xfstests.boot b/kvm-xfstests/test-appliance/files/root/kvm-xfstests.boot
index 4b687d0..26ef2fd 100755
--- a/kvm-xfstests/test-appliance/files/root/kvm-xfstests.boot
+++ b/kvm-xfstests/test-appliance/files/root/kvm-xfstests.boot
@@ -11,13 +11,6 @@ else
 fi
 }
 
-date >> /var/log/kvm-xfstests-boot.log
-exec 1<&-
-exec 2<&-
-exec 1>> /var/log/kvm-xfstests-boot.log
-exec 2>&1
-set -vx
-
 . /root/test-config
 
 FSTESTCFG=$(parse fstestcfg | sed -e 's/,/ /g')
-- 
2.13.0

--
To unsubscribe from this list: send the line "unsubscribe fstests" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Linux Filesystems Development]     [Linux NFS]     [Linux NILFS]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux