This patch rewrites the get_free_usb_port function:
 * Moving it out of make_create_command function.
 * Creating some helper function.
Signed-off-by: NQingtang Zhou <qzhou@redhat.com>

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

This reverts commit fadf99e6c0b1bdd86315040b3d3a35a804a550e8.

The freq option was removed in commit:
https://github.com/autotest/autotest/commit/93acc8819c8d912f2f3bf6f4d5d66bc2c99181a4
Thus revert this commit from virt-test.
Signed-off-by: NQingtang Zhou <qzhou@redhat.com>

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

The main monitor name is overriden to hmp1 and causes test failures...
Signed-off-by: NLukáš Doktor <ldoktor@redhat.com>

Use VM.get_monitory_by_type() function to get QMP monitor(s).
Signed-off-by: NLukáš Doktor <ldoktor@redhat.com>

This function can be used to get monitors of the given type.
Signed-off-by: NLukáš Doktor <ldoktor@redhat.com>

This patch add the lost APIs in utils_net, something wrong happended
when rebase the patch(commit 9efb355).
Signed-off-by: NYunping Zheng <yunzheng@redhat.com>

After the test during check_image the backup image was still being
overwritten.
Signed-off-by: NLukáš Doktor <ldoktor@redhat.com>

Signed-off-by: NLi Yang <liyang.fnst@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: NLi Yang <liyang.fnst@cn.fujitsu.com>

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

has_option(help_text, "use_nodefaults") will always return false because
qemu help_text just has 'nodefaults' line, instead of 'use_nodefaults'.
So -nodefaults will never be used as qemu command line.

This patch is to avoid this error situation
Signed-off-by: NMike Qiu <qiudayu@linux.vnet.ibm.com>

Now use test.bindir will not find the image file, just update it
to new interface data_dir.get_data_dir
Signed-off-by: NMike Qiu <qiudayu@linux.vnet.ibm.com>

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

This test checks groups when guest is started as
nobody [v2] using qemu -runas option to exec the qemu.
Signed-off-by: NYunping Zheng <yunzheng@redhat.com>

Add heavy disk write load test for windows guests. Using Crystal
Disk Mark [1] to generate the write load inside the guest. And set the
default cache to writeback in this test.

changes from v1:
  - Record steps in INFO logs
  - Close the session after test
  - Add command empty check

[1] Crystal Disk Mark is a BSD licensed program
    http://crystalmark.info/software/CrystalDiskMark/manual-en/License.html

    It can be downloaded here:
    http://crystalmark.info/software/CrystalDiskMark/index-e.html

    The file will be added to the winutils.iso
Signed-off-by: NYiqiao Pu <ypu@redhat.com>
Acked-by: NFeng Yang <fyang@redhat.com>

The previous patch introduced a bug in the qemu-ga.exe path.
This patch tries to fix this problem.
Signed-off-by: NQingtang Zhou <qzhou@redhat.com>

The previous patches for adding windows support of qemu-guest-agent
depend on a guest image in which user has installed necessary DLL
files.

This patch tries to install these DLLs by test script, so that to
we can test this feature with a fresh windows system.
Signed-off-by: NQingtang Zhou <qzhou@redhat.com>

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

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

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

* Fixed a docstring
* Improved error reporting for missing accessor generator parameters
* Avoid forcing subclass extended parameter value type to be str

* Add ability to compare LibvirtXMLBase derived instances
* Fixed indentation problems in libvirt_xml.devices.librarian
* Added ability to create devices from ElementTree elements
* Added simplistic input element XML validation
* Simplified Serial device class hierarchy
* Added new unittests for serial devices
* Added new unittests for vm_xml device enumeration / comparison
* Added stub modules for all devices classes
* Added address pseudo-device support
* Added unittests to verify no class steps on address-class properties
* Added unittest tearDown() to cleanup from any library module
  inter-test changes
* Added basic address pseudo-device unittest
* Added simplified address pseudo-device to take
  simple dictionary of attributes
* Fixed virsh_instance parameter-order issues
Signed-off-by: NChris Evich <cevich@redhat.com>

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>

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

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

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

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

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

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

In start()&create() of VM class, they will start
serial console automatic when vm is started.
It will prevent other test from using console.
So give a choice on starting serial console automatically.
Signed-off-by: NYu Mingfei <yumingfei@cn.fujitsu.com>

When network is transient, net_autostart() will raise.
The next step will try to get "transient" in result_object.
While error messages may not be in stderr but exist in result
object indeed:

 * Command:
 net-autostart default --disable
 Exit status: 1
 Duration: 0
 stdout:
 error: failed to unmark network default as autostarted
 error: Requested operation is not valid: cannot set autostart
        for transient network

Message "transient" exists, but stderr is None.
So search "ranient" in whole result object will be more suitable.
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>