virt-libvirt: Fix typo in virsh schedinfo(qemu) test

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

Following lead of similar tests that use the EXAMPLE.COM
checking, use TestNAError rather than TestError to result
in SKIP instead of ERROR for test
Signed-off-by: NJohn Ferlan <jferlan@redhat.com>

os.path.join("/cgroup/cpu", "/libvirt/qemu", "vm1") will
return "/libvirt/qemu/vm1", not "/cgroup/cpu/libvirt/qemu/vm1",
so I modify 'libvirt_cgroup_path' default value from
"/libvirt/qemu" to "libvirt/qemu"
Signed-off-by: NLi Yang <liyang.fnst@cn.fujitsu.com>

Signed-off-by: NSatheesh Rajendran <sathnaga@linux.vnet.ibm.com>

There are four actions in the case, host to guest, guest to host
and repeat in another thread, for action, the time out is 1000,
but the whole case's timeout value also 1000, which is unreasonable
and cause failure sometimes.

This patch is to solve this bug.
Signed-off-by: NMike Qiu <qiudayu@linux.vnet.ibm.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: NYu Mingfei <yumingfei@cn.fujitsu.com>

* Target_dev should be same as lvm pool's name.
* Replace test.tmpdir to data_dir to avoid too long
  path error.
* Add logic volume checking for creating lvm pool.
* Print more debug info for test.
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>

From time to time, people are forgetting the little
PEP8 details, and we have to fix them.
Signed-off-by: NLucas Meneghel Rodrigues <lmr@redhat.com>

Existing code didn't check for 'in shutdown' condition which led to
odd timing issues in various tests where is_dead() was returning true
during the "Trying to shutdown VM with shell command" path followed
by code attempting a startup which would fail because you cannot start
an already started guest.

Rather than add 'in shutdown' to the list of things to check - I flipped
the check to look for dead states and return true (per the comments).  I
also note that 'no state' was returning False previously which could
have caused false positives.

Since the list of checked states could grow over time, I added a check
for all currently known states and just print a message if a state
is not known.
Signed-off-by: NLucas Meneghel Rodrigues <lmr@redhat.com>

since qemu_iotests merged into tests/qemu_iotests, add param
qemu_iotests_dir to set iotests root dir;
And updated default git repo to qemu-kvm new repo;
Signed-off-by: NXu Tian <xutian@redhat.com>

This commit sets the Qemu sandboxing to 'on' by default. It allows the
user to run all tests using this feature to catch missing system calls
and providing more feedback to the general security of Qemu.
Signed-off-by: NEduardo Otubo <otubo@linux.vnet.ibm.com>

Not first index is 0, so update bootindex to 0, if we want to boot from it
firstly.
Signed-off-by: NFeng Yang <fyang@redhat.com>

In older code we did not update params for vm before creating it.
So old VM is reused. So we will meet following error:
15:42:02 ERROR| Exception escaping from test:
Traceback (most recent call last):
  File "/usr/code/autotest/client/shared/test.py", line 426, in _exec
    _call_test_function(self.execute, *p_args, **p_dargs)
  File "/usr/code/autotest/client/shared/test.py", line 853, in _call_test_function
    raise error.UnhandledTestFail(e)
UnhandledTestFail: Unhandled VMImageMissingError: CD image file not found: '/tmp/test.iso'
[context: (run_boot_from_device) --> creating 'virt-tests-vm1']
Traceback (most recent call last):
  File "/usr/code/autotest/client/shared/test.py", line 846, in _call_test_function
    return func(*args, **dargs)
  File "/usr/code/autotest/client/shared/test.py", line 299, in execute
    postprocess_profiled_run, args, dargs)
  File "/usr/code/autotest/client/shared/test.py", line 216, in _call_run_once
    *args, **dargs)
  File "/usr/code/autotest/client/shared/test.py", line 322, in run_once_profiling
    self.run_once(*args, **dargs)
  File "/usr/code/autotest/client/tests/virt/virt.py", line 139, in run_once
    run_func(self, params, env)
  File "/usr/code/autotest/client/shared/error.py", line 138, in new_fn
    return fn(*args, **kwargs)
  File "/usr/code/autotest/client/tests/virt/qemu/tests/boot_from_device.py", line 112, in run_boot_from_device
    vm.create()
  File "/usr/code/autotest/client/shared/error.py", line 138, in new_fn
    return fn(*args, **kwargs)
  File "/usr/code/autotest/client/tests/virt/virttest/qemu_vm.py", line 1769, in create
    raise virt_vm.VMImageMissingError(iso)
VMImageMissingError: CD image file not found: '/tmp/test.iso'

This patch could fix this.
Signed-off-by: NFeng Yang <fyang@redhat.com>

It makes up for a quicker default run, good for presentation
purposes.
Signed-off-by: NLucas Meneghel Rodrigues <lmr@redhat.com>

* Update try...except...finally.. syntax to old-style.
* Conformed parameter keys to match rest of test
* Fix logging parameter format bug
Signed-off-by: NChris Evich <cevich@redhat.com>

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

Signed-off-by: NAlex Jia <ajia@redhat.com>

Signed-off-by: NAlex Jia <ajia@redhat.com>

Signed-off-by: NAlex Jia <ajia@redhat.com>

Signed-off-by: NAlex Jia <ajia@redhat.com>

Signed-off-by: NAlex Jia <ajia@redhat.com>

cpulist is a list of physical CPU numbers. Its syntax is a comma
separated list and a special markup using '-' and '^' (ex. '0-4',
'0-3,^2') can also be allowed. The '-' denotes the range and the
'^' denotes exclusive.  The expression is sequentially evaluated,
so "0-15,^8" is identical to "9-14,0-7,15" but not identical to
"^8,0-15".
Signed-off-by: NAlex Jia <ajia@redhat.com>

Signed-off-by: NAlex Jia <ajia@redhat.com>

Signed-off-by: NAlex Jia <ajia@redhat.com>

Signed-off-by: NTom Zhou <zhouqt@gmail.com>

Some particular Linux distrio, such as debian, uses a
shell program called dash. Unfortunately, the dash
program can't work with "dash -c source foobar" command.

Hence let's use '/bin/bash' explicitly, instead of the
ambiguous '/bin/sh' symbol link.
Signed-off-by: NTom Zhou <zhouqt@gmail.com>

This type of installation is commonly used, remove the '@'
before it.

Also update the comment for it.
Signed-off-by: NTom Zhou <zhouqt@gmail.com>

virt-libvirt: Add floppy test for virsh change-media

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

When a guest is running, insert/update a file without
source path with "--config", then shutdown/destroy
guest, at this time, the guest cannot be started.
Signed-off-by: NLi Yang <liyang.fnst@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: NYu Mingfei <yumingfei@cn.fujitsu.com>

virt.tests: autotest.cpuflags test fix

Add control file to shared.control and update compress command to
compress virt dir to source tarball because autotest.cpuflags test
requried it;
Update compress result command to ignore core dump and crash file to
avoid compress or scp failed;
Signed-off-by: NXu Tian <xutian@redhat.com>