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

No one noticed this old module still there, until pylint did.
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>

Sometimes, people might quote the test type in their
config files, making the test generator to create 2
different entries for the same test. So, use the cartesian
parser itself to parse the test type value and get it,
rather than the ugly custom code (made by yours truly).

CC: Chris Evich <cevich@redhat.com>
Signed-off-by: NLucas Meneghel Rodrigues <lmr@redhat.com>

Simple validation for vm.verify_kernel_crash()
Signed-off-by: NChris Evich <cevich@redhat.com>

* Made post-process use the vm instance LOGIN_WAIT_TIMEOUT attribute
* Removed redefinition of inherited qemu_vm.VM attributes, update comment.
Signed-off-by: NChris Evich <cevich@redhat.com>

Before it was only ever setup during create().  As soon as
a vm object was started from scratch, it never got created.
Also removed libvirt_vm.VM.only_pty to always use virsh console
which should be more reliable and still support all types.  Added
a fixme in create() -> add_serial() to investigate older libvirt
additional serial addition at install.
Signed-off-by: NChris Evich <cevich@redhat.com>

Signed-off-by: NYu Mingfei <yumingfei@cn.fujitsu.com>

Signed-off-by: NYu Mingfei <yumingfei@cn.fujitsu.com>

Signed-off-by: NYu Mingfei <yumingfei@cn.fujitsu.com>

We shouldn't take the virsh bin for granted when running this
unittest. Use a trick similar to the one used in other test
and skip the test that depends on it directly.
Signed-off-by: NLucas Meneghel Rodrigues <lmr@redhat.com>

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

Currently we added a comma for netdev_extra_params in framework,
if we want to add multiple parameters in more than one times,
we should not use comma prefix in first parameter string.

eg. add multiple parameters more than one times:
  netdev_extra_params = "vhost=on"
  netdev_extra_params += ",sndbuf=512"

This patch moved the comma out of framework, then we can
set the parameter conveniently. We don't care about if it's the
first assignment, just using comma prefix for all params strings.
Signed-off-by: NAmos Kong <akong@redhat.com>

Since this prevents running things with the test runner
with other guests.
Signed-off-by: NLucas Meneghel Rodrigues <lmr@redhat.com>

Try to use ip link in case ifconfig didn't work. Looking
on /sys would be the last resource.
Signed-off-by: NMike Qiu <qiudayu@linux.vnet.ibm.com>
Signed-off-by: NLucas Meneghel Rodrigues <lmr@redhat.com>

Sometimes, /sys/class/net may contain files (such as bonding_masters,
if the bonding module is loaded). So, let's handle the ShellCmdErrors
that might arise when trying to list the mac address of the interfaces.

Also, if no suitable interfaces were found, just return None.
Signed-off-by: NLucas Meneghel Rodrigues <lmr@redhat.com>

Signed-off-by: NQingtang Zhou <qzhou@redhat.com>

Signed-off-by: NQingtang Zhou <qzhou@redhat.com>

The original VMCreateError exception arguments are incorrect.
And actually these 2 places should raise a VMConfigMissingError.
Signed-off-by: NQingtang Zhou <qzhou@redhat.com>

qemu/unittests symlink is required to run the 'unittest', so makes
sense to .gitignore it.
Signed-off-by: NCole Robinson <crobinso@redhat.com>

This allows the basic unittest config to be shared between

./run --type qemu --test unittest        and
./run --config qemu/cfg/unittests.cfg

While the latter may be the preferred invocation, I don't see any
reason to arbitrarily restrict things, especially when 'unittest'
shows up the --list-tests output.
Signed-off-by: NCole Robinson <crobinso@redhat.com>

Acked-by: NLukáš Doktor <ldoktor@redhat.com>
Signed-off-by: NCole Robinson <crobinso@redhat.com>

Since qemu_vm.py does it unconditionally. Adapt the code to handle
empty extra_params.
Signed-off-by: NCole Robinson <crobinso@redhat.com>

quote the unittest name to make it more obvious when reading logs.
Acked-by: NLukáš Doktor <ldoktor@redhat.com>
Signed-off-by: NCole Robinson <crobinso@redhat.com>

This logs the command as a single blog with properly escaped
line breaks, which allows copy+pasting the command. Helps for
manually reproducing a test.
Signed-off-by: NCole Robinson <crobinso@redhat.com>
Acked-by: NLukáš Doktor <ldoktor@redhat.com>

It's an older guest, but frequently useful for tests.
Signed-off-by: NLucas Meneghel Rodrigues <lmr@redhat.com>

Introduce support to Debian 6.0.7 x86_64. I happened to
be working on a related subject, so I took the chance and
added support to it. It comes with a download asset file,
which means you can conveniently download it.
Signed-off-by: NLucas Meneghel Rodrigues <lmr@redhat.com>

In order to install Ubuntu, we've been relying on cpio
directly. However, it needs root access for its operations.

So, in order to avoid failures when running as nonroot,
use fakeroot instead. This introduces a new dependency,
but hopefully it is not a huge one. And moreover, this
should only affect people looking to install Ubuntu or
Debian.
Signed-off-by: NLucas Meneghel Rodrigues <lmr@redhat.com>

Change the default hostname and domain names to something
more palatable.
Signed-off-by: NLucas Meneghel Rodrigues <lmr@redhat.com>

Signed-off-by: NYu Mingfei <yumingfei@cn.fujitsu.com>

Signed-off-by: NYu Mingfei <yumingfei@cn.fujitsu.com>

Signed-off-by: NYu Mingfei <yumingfei@cn.fujitsu.com>

Signed-off-by: NYu Mingfei <yumingfei@cn.fujitsu.com>

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

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

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

When detecting appropriate qemu names by default to run the
test suite, right now x86_64 is assumed. So, in order to
support other host architectures, such as ppc64, guess the
appropriate qemu name based on the host architecture.
Signed-off-by: NMike Qiu <qiudayu@linux.vnet.ibm.com>

Currently, when creating config files, autotest tries to find out
the config files that need to be overridden in testdir/cfg/*.cfg.

But it tries to pop the element for the list during traversing
itself, this is very dangerous, and it might cause an error when
running get_started.py, because the list wasn't updated yet, but
the transverse index wasn't updated.
Signed-off-by: NMike Qiu <qiudayu@linux.vnet.ibm.com>

1) Migration to read-only space.
2) Migration to low space destination.

Both migration uses exec protocol for migration to file.

qemu-kvm is started like Expect class not only like Tail.
Signed-off-by: NJiří Župka <jzupka@redhat.com>