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>

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>

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>