Sometimes the virsh_domjobabort test failed with:

error: Requested operation is not valid: no job is active on the domain

because the job finished earlier than the 'domjobabort' command was
issued.  This patch makes all the relevant jobs go out to a named pipe
which doesn't read the whole domain in order to stop the job until the
abort request.
Signed-off-by: NMartin Kletzander <mkletzan@redhat.com>

This patch modify the method of et netperf, make the script support
UDP_RR test, also modify the temporary dir.
Signed-off-by: NYunping Zheng <yunzheng@redhat.com>

Get netperf src package from web;
Make the script compatible with netperf 2.6.0;
Modify the method of get winutils vol;
Modify the method of start netserver in windows guest
Using wait_for  to wait until all net perf clients start to work;
Reorganize the configure file.
Signed-off-by: NYunping Zheng <yunzheng@redhat.com>

This patch make the systemtap_tracing test more robust.
Signed-off-by: NYunping Zheng <yunzheng@redhat.com>

If the command rpm --version fails, it means this command
is not present there. Fix the test that checks this condition.
Signed-off-by: NLucas Meneghel Rodrigues <lmr@redhat.com>

On newer libvirt there is a fix for a bug, that xml file is updated
before guest is resumed. The correct way is that the xml file is
updated after the guest is resumed.
Signed-off-by: NPavel Hrdina <phrdina@redhat.com>

Executing the 'rpm -qa | grep redhat-release' on non RHEL system or
non RPM system will cause the tests to fail. Adding few more conditions
resolves this issue.
Signed-off-by: NPavel Hrdina <phrdina@redhat.com>

It's obvious that if you are running the attach/detach config test
that you will not see the change in the xml returned by dumpxml of
a live guest.

This patch also fixes whitespace issues and typo in error message.
Signed-off-by: NPavel Hrdina <phrdina@redhat.com>

We have to attach two disks for the detach_twice test in order to run the test.
Signed-off-by: NPavel Hrdina <phrdina@redhat.com>

To reduce restart count and time needed to do it I've just move this code after
disks preparation. For disk preparation you need to have the guest offline and
it would be pointless to start the guest, modprobe the acpiphp and than shutdonw
the guest again to attach/detach required disks.
Signed-off-by: NPavel Hrdina <phrdina@redhat.com>

We want to attach the disk with --config parameter, because we have to be sure,
that the disk will be attached using all versions of libvirt. But to use this
parameter we have to at first shutdown the guest because this chagne will affect
next boot of the guest.
Signed-off-by: NPavel Hrdina <phrdina@redhat.com>

On some systems the cpu and cpuacct controllers are mounted together
like cpu,cpuacct and we should count with it.
Signed-off-by: NPavel Hrdina <phrdina@redhat.com>

We have to strip the slash from cgroup path because the "join" function
takes the last element starting with slash as the root path.
Signed-off-by: NPavel Hrdina <phrdina@redhat.com>

Signed-off-by: NLucas Meneghel Rodrigues <lmr@redhat.com>

Signed-off-by: NLucas Meneghel Rodrigues <lmr@redhat.com>

Signed-off-by: NLucas Meneghel Rodrigues <lmr@redhat.com>

In order to check if the guest_worker's script hadn't changed
check the file size instead just the file presense.
Signed-off-by: NLukáš Doktor <ldoktor@redhat.com>

Before we can cleanup after migrate test the guest worker has to
reconnect to the current VM. The session itself works fine
as the VM was migrated, but the VM object is different and reports
non-living VM, which results in impropper cleanup.
Signed-off-by: NLukáš Doktor <ldoktor@redhat.com>

Current kernels report Error 16 Device or resource busy: ... instead
of Error 24 Access is denied. Accept all kind of failures as success
in this particular test.
Signed-off-by: NLukáš Doktor <ldoktor@redhat.com>

Currently qemu dies when we try to remove and plug the port couple of
times without delay.

This modification allows to execute x (virtio_console_micro_repeates)
interuptions, where we don't check anything, only do them as fast as
possible. After that we execute normal loop where the transfer is
checked after each interruption.

As the port might not be openable for some time, the guest_worker
supports multiple attemps to open the port. Don't forget to remove
the old /tmp/virtio_console_guest.py* files in order to use the new one.
Signed-off-by: NLukáš Doktor <ldoktor@redhat.com>

No actual change, only put the interrupted_loopback initialization
before the thread initialization (in case of failure it's quicker)
and fix the 80 characters line width.
Signed-off-by: NLukáš Doktor <ldoktor@redhat.com>

Some tests were voulnurable to infinite loop due of not finished
background thread. Use funcatexit to register function, which sets
the exit_event and causes the background threads to finish.
Signed-off-by: NLukáš Doktor <ldoktor@redhat.com>

Temove the infinite loop from send and receiver threads.
Signed-off-by: NLukáš Doktor <ldoktor@redhat.com>

Tweak the config to maximize interrupted_loopback test
* short interruption is 0s
* short test_time is 0.1s
* tweak the acceptable_loss
* increase the number of acceptable test_time repetition
Signed-off-by: NLukáš Doktor <ldoktor@redhat.com>

virtio_console is now able to handle the reconnection without any sleep
thus we can use busy wait and use the console immediatelly when it's
created.
Signed-off-by: NLukáš Doktor <ldoktor@redhat.com>

Usually the index of lost characters is reloaded while reconnection.
This patch creates a function, which can be called from outside of this
thread in case we want to reload this index elswhere. Additionally booth
(debug and normal) threads use the same code now.
Signed-off-by: NLukáš Doktor <ldoktor@redhat.com>

err value was overwritten by thread return number which resulted in
incorrectly reported failed tests as pass.
Signed-off-by: NLukáš Doktor <ldoktor@redhat.com>

Signed-off-by: NLucas Meneghel Rodrigues <lmr@redhat.com>

Signed-off-by: NLucas Meneghel Rodrigues <lmr@redhat.com>

Signed-off-by: NLucas Meneghel Rodrigues <lmr@redhat.com>

Signed-off-by: NLucas Meneghel Rodrigues <lmr@redhat.com>

Signed-off-by: NLucas Meneghel Rodrigues <lmr@redhat.com>

Signed-off-by: NLucas Meneghel Rodrigues <lmr@redhat.com>

Signed-off-by: NLucas Meneghel Rodrigues <lmr@redhat.com>

Signed-off-by: NLucas Meneghel Rodrigues <lmr@redhat.com>

That function was supposed to return a CmdResult object,
but it wasn't due to a rebase mistake.
Signed-off-by: NLucas Meneghel Rodrigues <lmr@redhat.com>

Signed-off-by: NLucas Meneghel Rodrigues <lmr@redhat.com>

This is a legacy of the days where the kvm kernel modules would
be installed either by compiling the source code or via "kmod"
like packages, and the loading and unloading of kernel modules
would depend on that.

Nowadays, we can assume that the kvm modules always come from
the kernel packages.

One last piece of refiniment here would be to remove the module
loading/unloading feature from the base installer code.
Signed-off-by: NCleber Rosa <crosa@redhat.com>

Rather than just print the top SHA1, it is frequently
useful for quick reference to print the top commit
message.
Signed-off-by: NLucas Meneghel Rodrigues <lmr@redhat.com>

Signed-off-by: NCleber Rosa <crosa@redhat.com>