VM paused after install finished because of option '-no-shutdown', and
env.get_vm() got a paused VM, so case will fail at wait_for_login() when
need_install is 'yes';
Signed-off-by: NXu Tian <xutian@redhat.com>

virttest.qemu_vm: Check migration_exec_cmd_src before accessing it

Add NetworkManager and enable servie for RHEL.5 guest

Signed-off-by: NXiaoqing Wei <xwei@redhat.com>

In order to build git repos in the guest, ship this simple script.
It'll be used to build SPICE components, though it could be used
for other purposes.
Signed-off-by: NSwapna Krishnan <skrishna@redhat.com>

virttest.qemu_vm : add function to get vhost threads

This patch add a function to get vhost threads
Signed-off-by: NYunping Zheng <yunzheng@redhat.com>

Conflicts:
	virttest/qemu_vm.py

update autoyast.xml to enable serial console in SUSE guest;
Signed-off-by: NXu Tian <xutian@redhat.com>

Give a default value "" to reboot_after_config. Otherwise it will raise
an TypeError(argument of type 'NoneType' is not iterable) when it is None.
Signed-off-by: NYiqiao Pu <ypu@redhat.com>

Signed-off-by: NQingtang Zhou <zhouqt@gmail.com>

libvirt: Virsh: session manager fixing.

Introduce a class variable named COUNTERS in VirshPersistent. COUNTERS is a
dict whose key is a_id and value is counter. Two method counter_increase and
counter_decrease introduced in VirshPersistent at the same time. B/c VirshSession is
not auto closed, then we need to manage the VirshSession in VirshPersistent.
Signed-off-by: Nyangdongsheng <yangds.fnst@cn.fujitsu.com>

Add a param named auto_close into __init__ method of VirshSession. The default
value of this param is False, and it will not close the ShellSession when we derefer it.
So, I add this param to allow user pass a auto_close=True to use it more convinient.
Signed-off-by: Nyangdongsheng <yangds.fnst@cn.fujitsu.com>

Fix an error of exit. There is no method named exit in Virsh class,
So super().__exit__ in VirshPersistent will down.
Signed-off-by: Nyangdongsheng <yangds.fnst@cn.fujitsu.com>

In current code, VirshPersistent.__del__() will never be called in
`del virshPersistent_instance`
Because there are some other references to this object.
They are VirshClosure object. The VirshClosure.dict_like_instance is a
reference to VirshPersistent object. So, we can't use the del feature
of VirshPersistent at all.
So every test using VirshPersistent object will remain a virsh process
after exiting.

This patch fix this problem.
 * Change the dict_like_instance in VirshClosure from the reference of
Virsh object to another object named closure_args stored in VirshPersistent.
And we update the closure_args in setting slots of VirshPersistent to keep
the closure_args and VirshPersistent in-sync.
Signed-off-by: Nyangdongsheng <yangds.fnst@cn.fujitsu.com>

* test_del_VirshPersistent to make sure __del__ method works well.
* test_VirshSession to make sure VirshSession works well without virshPersistent.
* test_VirshPersistent to make sure COUNTERS in VirshPersistent works well.
Signed-off-by: Nyangdongsheng <yangds.fnst@cn.fujitsu.com>

 tests.cfg.netperf.cfg : Make test support different types of network apapters

Signed-off-by: NEduardo Habkost <ehabkost@redhat.com>

The pclmulqdq bug is fixed on qemu.git on commit
56383703c060777fd01aaf8d63d5f46d660e9fb9, so we can stop ignoring the
bug and report as an actual failure.
Signed-off-by: NEduardo Habkost <ehabkost@redhat.com>

Remove the limit of virtio_net, make the test support different types of
network adapters like e1000 and rtl8139
Signed-off-by: NYunping Zheng <yunzheng@redhat.com>

The output of ifconfig is different between different os,
so some time the regex can not get the nics number in the guest.
This patch try to fix this issue
Signed-off-by: NYunping Zheng <yunzheng@redhat.com>

The later spelling is more clear, and used in more places
inside the code. This fixes a hidden problem in the command
creation code.
Signed-off-by: NLucas Meneghel Rodrigues <lmr@redhat.com>

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

parameter 'md5sum' in test configuration file will conflict with
cdrom iso md5sum parameter, and it will caused md5sum mismatch error
when re-create(env_process.preprocess_vm) VM in test script, so rename
it;
Signed-off-by: NXu Tian <xutian@redhat.com>

Replace the method which used only disk names to match autotest
image format with virttest.qemu_qtree one. It additionally verifies
all kind of other params of "info qtree" and "info block".
Signed-off-by: NLukáš Doktor <ldoktor@redhat.com>

some fix for block stream tests

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

add_drive and add_cdrom are obsoleted by
virttest.qemu_devices.images_define_by_variables()
Signed-off-by: NLukáš Doktor <ldoktor@redhat.com>

q35 doesn't have ide bus. People should use ahci instead.
Signed-off-by: NLukáš Doktor <ldoktor@redhat.com>

Libvirt doesn't use the full cmdline, but only -device $driver,$params
format. This patch makes autotest follow this syntax too.
Signed-off-by: NLukáš Doktor <ldoktor@redhat.com>

When we force insert device into bus where it doesn't belongs to
we should probably set bus and addr as it's not the default behavior.
Signed-off-by: NLukáš Doktor <ldoktor@redhat.com>

Error reason was shown only in force insertion mode.
Signed-off-by: NLukáš Doktor <ldoktor@redhat.com>

Add support for QDrive, which supports drive_add (QHPDrive) or
__com.redhat_drive_add (QRHDrive).
Signed-off-by: NLukáš Doktor <ldoktor@redhat.com>

This patch adds the possibility to safely hotplug/unplug devices from
running VM.

Additionally it adds another VM state and VM param. When the test
requires clean VM, you might specify "allow_hotplugged_vm = no" and
when VM used hotplug it will be recreated.

For hotplugging you might specify monitor, which should be used and
whether to verify results. This is not supported by all devices, but
when there is a method to automatically verify hotplug results, it'll
do it for you and cleans the qdev state for you. Otherwise it's upto the
user to do it by qdev.hotplug_verified().
Signed-off-by: NLukáš Doktor <ldoktor@redhat.com>

To simplify and unify usage of QBaseDevice.child_bus I added
add/rm_child_bus functions.
Signed-off-by: NLukáš Doktor <ldoktor@redhat.com>

Fix issues of DevContainer.remove() and add support for
non-recursive removal.
Signed-off-by: NLukáš Doktor <ldoktor@redhat.com>

Currently during vm_preprocess params of unused running VMs are updated,
but VM is not recreated (it's not used, so there is no need for it).
If we create VM in 1st test, in 2nd test we use different params but set
start_vm = no, params are updated so when we use the same params from
2nd test in 3rd test, preprocess would match the VM from 1st test as
the params were updated. Anyway as the VM is not recreated, the running
VM doesn't represents the params from 2nd test.

With my modification the preprocess would always handle VM properly. The
only problem might be, when someone uses vm.create() directly without
preprocess in the 2nd test. Anyway it's a bad practise and I haven't found
it in upstream tests. We always use preprocess to prepare the VM.
Signed-off-by: NLukáš Doktor <ldoktor@redhat.com>

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

Original autotest ignored some params.
Signed-off-by: NLukáš Doktor <ldoktor@redhat.com>

This patch changes the way of handling images, cdroms and floppies
params. Instead of custom handling in make_create_command it uses
wrappers in self.devices. It keeps track of all image buses according
to autotest.

It shouldn't change the way how autotest used to create disks. Anyway
few params which were omited for some devices were added. Also all
buses are tracked so in case there is no free bus it should create one.

This feature works 100% with strict_mode. For non-strict_mode there
might be few differences as qemu doesn't differentiate between ide and
ahci buses. Handling of this would require deeper modifications to
the way of images handling and will be discussed and resolved later.

Images should be added with the correct version of device thus we should
be able to unplug/hotplug the device in test.
Signed-off-by: NLukáš Doktor <ldoktor@redhat.com>