Check for I/O Errors before the test end and raise Warning in such case.
Signed-off-by: NLukáš Doktor <ldoktor@redhat.com>

Log the hotplug device's name for easier log analyzing.
Signed-off-by: NLukáš Doktor <ldoktor@redhat.com>

Stop stresser should destroy the VM ungracefully in case login fails.
Also check the vm.verify_alive before using monitor on "hot" places.
Signed-off-by: NLukáš Doktor <ldoktor@redhat.com>

Parallel mode of this test doesn't keep order of inserted devices, which
leads to devices ending up in different buses. Setting strict_mode = True
always enforces bus and addr.
Signed-off-by: NLukáš Doktor <ldoktor@redhat.com>

Set absolute path for disk images using data_dir.get_data_dir().
Signed-off-by: NLukáš Doktor <ldoktor@redhat.com>

This patch adds parallel variant, which hotplugs multiple random disks
over multiple monitors simultaneously.

As qdev is not thread safe it was necessarily to put the locking here.
Signed-off-by: NLukáš Doktor <ldoktor@redhat.com>

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

This version first initiate all hotplug/unplugs and then verifies the
results, which puts more stress on qemu.
Signed-off-by: NLukáš Doktor <ldoktor@redhat.com>

With incorrect configuration it might happen that no disks can be added.
Throw exception instead of simple warning and clean test run.
Signed-off-by: NLukáš Doktor <ldoktor@redhat.com>

The qdev.images_define_by_variables needs to be in the try/except
block in order to work as expected (remove format and try another round)
Signed-off-by: NLukáš Doktor <ldoktor@redhat.com>

Add context for easier debugging.
Signed-off-by: NLukáš Doktor <ldoktor@redhat.com>

This patch changes the handling of unplug. The simplu_unplug has sometimes
problem detecting correctly the unplug status, as it might take seconds
before the disk is removed (monitor returns output, but qemu still
waits for OS repsonse before it removes the device from qtree).

This custom handler waits up to 1s for device to be removed and even
when it's not proved to be removed it removes it from representation.
We can afford this as all devices are again checked later in the test
and the VM state is for the whole period set to dirty.
Signed-off-by: NLukáš Doktor <ldoktor@redhat.com>

DeviceInsertError is currently located in qemu_devices.utils.
Signed-off-by: NLukáš Doktor <ldoktor@redhat.com>

qemu_devices doesn't support force nor unplug for some time,
this patch fixes this test.
Signed-off-by: NLukáš Doktor <ldoktor@redhat.com>

qemu_devices is becoming way too long. This patch splits it into
logical modules: qdevices, qbuses and qcontainer.

I perserved the qemu_devices_unittest in the same form for now,
but it can be split in the future. It already is sepparated into
three classes.
Signed-off-by: NLukáš Doktor <ldoktor@redhat.com>

There's no need to have the name of the "run_<name>" method (main
test entry point) depend on the test name. So, for simplicity sake,
let's just name the test entry point "run".

I have decided to not fallback gracefully to the previous naming
convention as I believe this can lead to confusion in the future.
Signed-off-by: NCleber Rosa <crosa@redhat.com>

The default scsi hba is lsi53c895a, but in ppc64, it is spapr-vscsi.

This patch to enable spapr-vscsi in ppc64 platform.
Signed-off-by: NMike Qiu <qiudayu@linux.vnet.ibm.com>

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

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

This tests the disk hotplug/unplug functionality.
 1) prepares multiple disks to be hotplugged
 2) hotplugs them
 3) verifies that they are in qtree/guest system/...
 4) unplugs them
 5) verifies they are not in qtree/guest system/...
 6) repeats $repeat_times
 *) During the whole test stress_cmd might be executed
Signed-off-by: NLukáš Doktor <ldoktor@redhat.com>