The pool_autostart function was not being overriden in virsh instance,
but dummy function already written.  I just added it into setUp() now
unittest is working perfectly when run as non-root.
Signed-off-by: NChris Evich <cevich@redhat.com>

* Replace real virsh module with a bogus virsh module
* Rewrite relative virsh pool functions for unittest.
  With setting output to static strings.
Signed-off-by: NYu Mingfei <yumingfei@cn.fujitsu.com>

fix not all disk listed issue, get cdrom drive letter with
utils_misc.get_winutils_vol();
Signed-off-by: NXu Tian <xutian@redhat.com>

Our check_patch and run_unittests scripts both run all unittest files IN
PARALLEL, not one at a time. This leads to some really interesting and
difficult to debug problems, like one test deleting/modifying temp files
or state for another test.  Problems here are:

1) The VIRSH_EXEC logic isn't working right, and because the virsh
module is sharing memory, changing virsh.virsh_exec in your test is
interfering with other tests.

2) Many tests are calling self.sp.method() which execute real virsh
commands. This is fine, except that they don't all work w/o root access
and therefore are dependant on external state.
Signed-off-by: NChris Evich <cevich@redhat.com>

Mainly minor items that should have no effect on tests.  Some
more risky changes were made to set_agent_channel() and
get_iface_dev() methods.  Those changes were made because
it made line-wrapping easier (lines were too long) and to
make parameter passing more clear (redundant ** magic).
Signed-off-by: NChris Evich <cevich@redhat.com>

Migrate multiple vms with these features:
1.Orderly or simultaneous migration.
2.Simple live migration or combination options.
3.Different migration method.
Signed-off-by: NYu Mingfei <yumingfei@cn.fujitsu.com>

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

nic_extra_params were using an incorrect regexp to match additional
params. Use split instead similarily to blk_extra_params.
Signed-off-by: NLukáš Doktor <ldoktor@redhat.com>

blk_extra_params were accidentally left out while rewriting
images creation. Split them and assign them as normal params.

NOTE: This version would work only for params in the standard way
      %s=%s,... Currently I haven't found any other use so I hope
      this will last for ever.
Signed-off-by: NLukáš Doktor <ldoktor@redhat.com>

As an active NetworkManager may give us problems during
testing, stop the daemon before bonding. add a random wait
time before and after set_link.
Signed-off-by: NYunping Zheng <yunzheng@redhat.com>

Currently our script set bridge forward to 0, it cann't work
in some os. checking bridge doc 802.1D-1998 and latest
802.1D-2004, in these docs Bridege Forward Delay range time
is 4.0-30.0 seconds. so modify the value to 4
Signed-off-by: NYunping Zheng <yunzheng@redhat.com>

Signed-off-by: NXu Tian <xutian@redhat.com>

Add missed test scenario '1 0', AKA "indirect_desc on, event_idx off" test;
Signed-off-by: NXu Tian <xutian@redhat.com>

Update virsh vcpuinfo test.

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

* Specify string format arguments for logging functions.
* Remove unused variables.
* Replace invalid name "e" to "detail".
Signed-off-by: NYu Mingfei <yumingfei@cn.fujitsu.com>

Path /var/run/libvirt/lxc means there is a running lxc guest, but not the lxc
driver is available.
Signed-off-by: Nyangdongsheng <yangds.fnst@cn.fujitsu.com>

B/c the errors in __del__() will be ignored, then we want
user call conn_recover() manually our of __del__().

* rename conn_finish to conn_recover.
* add auto_recover into __init__.
Signed-off-by: Nyangdongsheng <yangds.fnst@cn.fujitsu.com>

* Add a unittest for tmp_dir to make sure it is unique for
  each instance.
* Add three unittests for functions in utils_conn which build
  CA key.
Signed-off-by: Nyangdongsheng <yangds.fnst@cn.fujitsu.com>

* Remove tmp_dir from param of __init__, and build tmp_dir
  using tempfile to assure it unique for each instance.
* Add more comment for the varaibles of connection class to
  make it more clear.
* Pick the CA key build method out from TLSConnection as a
  function in utils_conn.
Signed-off-by: Nyangdongsheng <yangds.fnst@cn.fujitsu.com>

Signed-off-by: Nyangdongsheng <yangds.fnst@cn.fujitsu.com>

Signed-off-by: Nyangdongsheng <yangds.fnst@cn.fujitsu.com>

Signed-off-by: Nyangdongsheng <yangds.fnst@cn.fujitsu.com>

virttest.virsh: Fix error of referenced before defined. 

A newer pylint version caught a problem in the __all__
dictionary for these 2 files, fix it.
Signed-off-by: NLucas Meneghel Rodrigues <lmr@redhat.com>

 tests.cfg: Set up the lowest_mem for unattended_install

1. Remove vcpucount_live() and modify virsh vcpupin test file
2. Remove a unused parameter 'vcpu' in virsh vcpupin test file
   which was found by run_pylint.py
3. Add return value for vcpucount()
Signed-off-by: NLi Yang <liyang.fnst@cn.fujitsu.com>

As we try to shrink memory size when host don't have enough huge pages.
unattended_install may fail because guest don't have enough RAM in Linux guests
when we start guest with huge pages. This is caused by anaconda memory size
check. For newest RHEL and fedora the lowest memory size is 512M. So set up the
it to 512M in cfg file.
Signed-off-by: NYiqiao Pu <ypu@redhat.com>

Signed-off-by: Nyangdongsheng <yangds.fnst@cn.fujitsu.com>

Signed-off-by: Nyangdongsheng <yangds.fnst@cn.fujitsu.com>

Signed-off-by: Nyangdongsheng <yangds.fnst@cn.fujitsu.com>

RHEL5 doesn't support both 'aio' and 'id' for drive
files. So check whether such options are supported
and don't add them in case they are not.
Signed-off-by: NLucas Meneghel Rodrigues <lmr@redhat.com>

Reduced the internal Cartesian variant set while retaining
testing of key values.  The result is much faster unit-testing
while loosing very little in depth.
Signed-off-by: NChris Evich <cevich@redhat.com>

virt: Rename virttest/nfs_unnittest.py to virttest/nfs_unittest.py

a += b equals to a = a + b.
So for self.virsh_exec += " -c xxx" in VirshSession.
It will cause problem of referenced before defined.
Now define self.virsh_exec before this line.
Signed-off-by: NYu Mingfei <yumingfei@cn.fujitsu.com>

Because virttest/nfs_unnittest.py is end with _unnittest.py, not
_unittest.py, tool/run_unittest.py cannot find this module.
Signed-off-by: NLi Yang <liyang.fnst@cn.fujitsu.com>

The correct command for old version pci-assign device should be
'-pcidevice' and not '-device pcidevice'.
Signed-off-by: NLukáš Doktor <ldoktor@redhat.com>

The bacground process was likely to be killed by SIGIO while setting
port to async mode. This was due of bad workflow, where first the
port was set to async mode, than the the new SIGIO handler was set.

With this version when we set async mode, we fist add the SIGIO
handler and then set the port to async mode.

When we are setting the port back, the order is the opposite, so
first the port is set to sync mode and then we remove the SIGIO
handler.

Additionally the workaround with test_delete_guest_script() was
removed as it's fixed by this patch.

NOTE: You have to remove the previous /tmp/virtio_console_guest.py*
      in order to let virtio_console test to copy this updated one!
Signed-off-by: NLukáš Doktor <ldoktor@redhat.com>

Test virsh_snapshot_disk: Fix indentation.
Signed-off-by: NRuda Moura <rmoura@redhat.com>