]> www.infradead.org Git - users/dwmw2/qemu.git/commitdiff
tests: Fix ubuntu.i386 image initialization
authorFam Zheng <famz@redhat.com>
Thu, 22 Mar 2018 03:47:53 +0000 (11:47 +0800)
committerFam Zheng <famz@redhat.com>
Mon, 9 Apr 2018 07:13:02 +0000 (15:13 +0800)
The apt-get commands we run through ssh expect certain features of the
tty, and refuses to work if /dev/null is used. It is ugly, but easy to
satisfy.

Actually, there is no reason to hide the output. It just makes things
harder to diagnose. We can always redirect in the Makefile, so don't do
it conditionally here.

Reported-by: Eric Blake <eblake@redhat.com>
Signed-off-by: Fam Zheng <famz@redhat.com>
Message-Id: <20180322034753.6301-1-famz@redhat.com>
Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Tested-by: Eric Blake <eblake@redhat.com>
Signed-off-by: Fam Zheng <famz@redhat.com>
tests/vm/basevm.py

index 686d88decf370ce223aa4a9383d47309ff99397a..3a2d508c354a6d540bca72a4e3f48d118e098fb9 100755 (executable)
@@ -107,10 +107,7 @@ class BaseVM(object):
         assert not isinstance(cmd, str)
         ssh_cmd += ["%s@127.0.0.1" % user] + list(cmd)
         logging.debug("ssh_cmd: %s", " ".join(ssh_cmd))
-        r = subprocess.call(ssh_cmd,
-                            stdin=sys.stdin if interactive else self._devnull,
-                            stdout=sys.stdout if interactive else self._stdout,
-                            stderr=sys.stderr if interactive else self._stderr)
+        r = subprocess.call(ssh_cmd)
         if check and r != 0:
             raise Exception("SSH command failed: %s" % cmd)
         return r