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

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>

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>

Previously, in VirshClosure.__call__(), weakref to VirshPersistent will update the VP.
It will change or modify the slots in VirshPersistent instance, then make the dargs
to next __call__().
Signed-off-by: Nyangdongsheng <yangds.fnst@cn.fujitsu.com>

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

To avoid circle reference, we use a class variable named closure_args
to replace VirshPersistent object for VirshClosure.dict-like-instance.
But we need to override the all item setting method in Virsh, such as
__setitem__, dict_set, super_set. It is not a good design.

Then, we use weakref to make reference from VirshClosure to VirshPersistent
weak, so that the __del__ method in VirshPersistent will work properly.
Signed-off-by: NChris Evich <cevich@redhat.com>
Signed-off-by: Nyangdongsheng <yangds.fnst@cn.fujitsu.com>

Although self identical to self.data, prefer to use self.data
like other function did.
Signed-off-by: NLi Yang <liyang.fnst@cn.fujitsu.com>

We found following error in python 2.4.
Traceback (most recent call last):
  File "/usr/code/autotest-devel/client/shared/test.py", line 426, in _exec
    _call_test_function(self.execute, *p_args, **p_dargs)
  File "/usr/code/autotest-devel/client/shared/test.py", line 853, in _call_test_function
    raise error.UnhandledTestFail(e)
UnhandledTestFail: Unhandled AttributeError: 'function' object has no attribute '__globals__'    [context: (run_unattended_install)]
Traceback (most recent call last):
  File "/usr/code/autotest-devel/client/shared/test.py", line 846, in _call_test_function
    return func(*args, **dargs)
  File "/usr/code/autotest-devel/client/shared/test.py", line 299, in execute
    postprocess_profiled_run, args, dargs)
  File "/usr/code/autotest-devel/client/shared/test.py", line 215, in _call_run_once
    self.run_once_profiling(postprocess_profiled_run,
  File "/usr/code/autotest-devel/client/shared/test.py", line 322, in run_once_profiling
    self.run_once(*args, **dargs)
  File "/usr/code/autotest-devel/client/tests/virt/virt.py", line 133, in run_once
    run_func(self, params, env)
  File "/usr/code/autotest-devel/client/shared/error.py", line 138, in new_fn
    return fn(*args, **kwargs)
  File "/usr/code/autotest-devel/client/tests/virt/tests/unattended_install.py", line 1003, in run_unattended_install
    dst, mount_point, image_name)
  File "/usr/code/autotest-devel/client/tests/virt/virttest/funcatexit.py", line 50, in register
    if func.func_name not in func.__globals__:
AttributeError: 'function' object has no attribute '__globals__'

In python 2.7:
>>> fuc.__globals__
{'__builtins__': <module '__builtin__' (built-in)>, '__name__': '__main__', 'fuc': <function fuc at 0x7f17b90778c0>, '__doc__': None, '__package__': None}
>>> fuc.func_name
'fuc'
>>> fuc.func_globals
{'__builtins__': <module '__builtin__' (built-in)>, '__name__': '__main__', 'fuc': <function fuc at 0x7f17b90778c0>, '__doc__': None, '__package__': None}
Signed-off-by: NFeng Yang <fyang@redhat.com>

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

Previously this tool was dependant on a specific directory layout
and the layout was fixed.  Now that project tree has been split
across multiple repositories, there is no guaranteed match.  This
patch tries to locate the nearest LICENSE file while traversing
up the directory tree.  It takes the lower-case version of the
first word in this file as the project name.  As of this commit
all LICENSE files have appropriate content for this scheme to
function properly.  It assumes any future project tree splits
will cause LICENSE files to be updated to match new/changed names.
Signed-off-by: NChris Evich <cevich@redhat.com>

Signed-off-by: NFeng Yang <fyang@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: NCong Li <coli@redhat.com>

Signed-off-by: NCong Li <coli@redhat.com>

The sandbox service class reuses many of the sandbox-command based
classes along with a service manager.  Like it's relatives, it is
fully pickleable so instances can span across tests.
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>

fix a typo introduced by me in 651,
the "image_format" param should still be qcow2 for qcow2v3.
I was renaming the qcow3 to qcow2_v3 in 630, but I mistakenly set the branch to master when
opening the pullreq, then reopen it on 651 to branch next, but that time I mistakenly modified the 'image_format'
Signed-off-by: NXiaoqing Wei <xwei@redhat.com>

and modify the qemu_storage.py to expand to support check qed qcow2v3
and increase the verbose to let people know the format in debug file.
Signed-off-by: NXiaoqing Wei <xwei@redhat.com>

Added missing param'tmp_dir' for Windows guests for the
'mirroring.continuous_backup' test case.
Signed-off-by: NXu Tian <xutian@redhat.com>

Fix the 'arguments too long' error when running tests like
'virtio_scsi_variants.multi_scsiid_lun'.  Now, we will save
the command into a tempfile when the command's length is bigger
than 255 characters, and make the shell to source the tempfile,
causing it to effectively execute the command line, now
without the 255 char restriction.
Signed-off-by: NXu Tian <xutian@redhat.com>

As /etc/mtab will always use the real patch, so get the real path for
mount_point and src. Otherwise it will make mistake when we try to
find the mounted file system from mtab.
Signed-off-by: NYiqiao Pu <ypu@redhat.com>

Boot up a guest and malloc memory in host to force it to move
among the nodes of host.
Signed-off-by: NYiqiao Pu <ypu@redhat.com>

Boot up a guest have the same number of node as host. Each node in guest
only have 1 cpu. Try to malloc inside the guest with taksset. The memory
allocated in host should also in the same node as the cpu thread.
Signed-off-by: NYiqiao Pu <ypu@redhat.com>

Get host numa topology. Start and bind guest to the cpu in each node
one by one. Then check the memory status for qemu process.
The qemu process should mainly used the memory from the node that it
is binded to.
Signed-off-by: NYiqiao Pu <ypu@redhat.com>

Add the command line to simulate a multi node NUMA system in guest.
Signed-off-by: NYiqiao Pu <ypu@redhat.com>

Add a class NumaInfo in utils_misc and some related functions to
get the host numa structure and information.

Also add a function in utils_test to get qemu process's memory
and cpu use status in each node.
Signed-off-by: NYiqiao Pu <ypu@redhat.com>