Don't wait until the test timeout, but fail the test when thread
or VM fails.
Signed-off-by: NLukáš Doktor <ldoktor@redhat.com>

When VM dies it closes connection to sockets. We might (and often)
stuck in readout for ever.

Additionally get rid of in-test readouts which are handeled by
guest_worker's safe_exit_loopback_threads()
Signed-off-by: NLukáš Doktor <ldoktor@redhat.com>

Signed-off-by: NLucas Meneghel Rodrigues <lmr@redhat.com>

Signed-off-by: NLucas Meneghel Rodrigues <lmr@redhat.com>

Signed-off-by: NLi Yang <liyang.fnst@cn.fujitsu.com>

Signed-off-by: NLi Yang <liyang.fnst@cn.fujitsu.com>

* Bug introduced in 8f464022f845e958c4777f943384179b66428eec
* Fix 'name' -> 'xml_path'
Signed-off-by: NChris Evich <cevich@redhat.com>

ksmtuned modifies KSM setting which results in test failure.
Signed-off-by: NLukáš Doktor <ldoktor@redhat.com>

No code changes, only variable renames, etc.
Signed-off-by: NLukáš Doktor <ldoktor@redhat.com>

No test change, only additional comments.
Signed-off-by: NLukáš Doktor <ldoktor@redhat.com>

This patch removes debug print which somehow made it to upstream, plus
two long line fixes. There are no other (fixable) pylint errors now.
Signed-off-by: NLukáš Doktor <ldoktor@redhat.com>

Signed-off-by: NChris Evich <cevich@redhat.com>

Signed-off-by: NChris Evich <cevich@redhat.com>

Fix cgroup testing by adding the cgroup library from
current autotest upstream (0.14 rpms won't have it).
Signed-off-by: NLucas Meneghel Rodrigues <lmr@redhat.com>

Signed-off-by: NLucas Meneghel Rodrigues <lmr@redhat.com>

It is more helpful to have a slightly larger and more
explanatory variable name than hlp. This way, we make
the internal implementation the same as the libvirt vm.

Please bear with me, I'll try to not do mass changes
in this code (at least as much as possible).
Signed-off-by: NLucas Meneghel Rodrigues <lmr@redhat.com>

And also, fix users of this id param in the qmp_basic tests.
Signed-off-by: NLucas Meneghel Rodrigues <lmr@redhat.com>

Signed-off-by: NLucas Meneghel Rodrigues <lmr@redhat.com>

Signed-off-by: NLucas Meneghel Rodrigues <lmr@redhat.com>

Signed-off-by: NLucas Meneghel Rodrigues <lmr@redhat.com>

Signed-off-by: NLucas Meneghel Rodrigues <lmr@redhat.com>

Signed-off-by: NLucas Meneghel Rodrigues <lmr@redhat.com>

Signed-off-by: NLucas Meneghel Rodrigues <lmr@redhat.com>

Signed-off-by: NLucas Meneghel Rodrigues <lmr@redhat.com>

Signed-off-by: NLucas Meneghel Rodrigues <lmr@redhat.com>

Signed-off-by: NLucas Meneghel Rodrigues <lmr@redhat.com>

Signed-off-by: NLucas Meneghel Rodrigues <lmr@redhat.com>

Signed-off-by: NLucas Meneghel Rodrigues <lmr@redhat.com>

Signed-off-by: NLucas Meneghel Rodrigues <lmr@redhat.com>

Signed-off-by: NLucas Meneghel Rodrigues <lmr@redhat.com>

Signed-off-by: NLucas Meneghel Rodrigues <lmr@redhat.com>

Signed-off-by: NLucas Meneghel Rodrigues <lmr@redhat.com>

Signed-off-by: NLucas Meneghel Rodrigues <lmr@redhat.com>

Signed-off-by: NLucas Meneghel Rodrigues <lmr@redhat.com>

It is pretty hard to anticipate what may happen when using
a custom config file rather than the more controlled environment
of the virt test. So, every time we are using custom configs,
let's print the shortname rather than the highly neat (and
modified) trimmed full name.
Signed-off-by: NLucas Meneghel Rodrigues <lmr@redhat.com>

Do basic sanity checking of the qemu option passed,
if it's not a real file, throw a RuntimeError.
Signed-off-by: NLucas Meneghel Rodrigues <lmr@redhat.com>

This change is similar to 81ff35cbc64d973964c8f84efc0dfe3c7c06ceb7,
only slightly nicer, since we search for qemu-img and
qemu-io in more paths, and also, if they are not actually
found, an error will be raised.
Signed-off-by: NLucas Meneghel Rodrigues <lmr@redhat.com>

Signed-off-by: NLucas Meneghel Rodrigues <lmr@redhat.com>

We can strip a lot of unneeded [1] info from
test names displayed in the test runner.

Unfortunately, there are lots and lots of corner
cases when trying to figure out automagically
what to strip from the test name and what to not
strip, so instead of doing it choose an appropriate
index, appropriate for the way the test runner works
(this might print names not entirely correct for
people using custom cartesian files).

Using a part of the full name instead of the
short name also means we can be sure that just
copying and pasting the name from the test list
in the test list will just work, always.

So now a test run will give the much nicer output:

$ ./run -t kvm
SETUP: PASS (1.40 s)
DATA DIR: /home/lmr/Code/virt-test.git/shared/data
DEBUG LOG:
/home/lmr/Code/virt-test.git/logs/run-2012-11-29-15.15.00/debug.log
TESTS: 13
migrate.default.tcp: PASS (51.86 s)
migrate.default.unix: PASS (27.91 s)
migrate.default.exec: PASS (26.07 s)

And the test log dirs are also coherent with this
printed name.

[1] Some might argue the information is important, but
here we will sacrifice some of the immediately obvious
info for shorter and easier in the eyes names
Signed-off-by: NLucas Meneghel Rodrigues <lmr@redhat.com>