public inbox for fstests@vger.kernel.org
 help / color / mirror / Atom feed
* [PATCH] test-appliance: revert logging change on startup
@ 2017-05-24  4:17 Eric Biggers
  0 siblings, 0 replies; only message in thread
From: Eric Biggers @ 2017-05-24  4:17 UTC (permalink / raw)
  To: Theodore Ts'o; +Cc: fstests, Eric Biggers

From: Eric Biggers <ebiggers@google.com>

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@google.com>
---
 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


^ permalink raw reply related	[flat|nested] only message in thread

only message in thread, other threads:[~2017-05-24  4:18 UTC | newest]

Thread overview: (only message) (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2017-05-24  4:17 [PATCH] test-appliance: revert logging change on startup Eric Biggers

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox