With the py3k changes some returns are not strings, but decoded
unicodes. Currently this mainly breaks monitor returns, where human
monitor gets recognized as qmp one. Let's use "six.string_types" that
provides the right string types for given python version.
Signed-off-by: NLukáš Doktor <ldoktor@redhat.com>

Signed-off-by: NHaotong Chen <hachen@redhat.com>

Signed-off-by: NHaotong Chen <hachen@redhat.com>

Signed-off-by: NHaotong Chen <hachen@redhat.com>

1 Add jumbo setting for macvtap devices
2 Use second ethernet for login guest since host could not
communicate guest through first one which is attached from macvtap
3 fix no attribute 'stop_tcpdump' error
Signed-off-by: NWenli Quan <wquan@redhat.com>

instead of website
Signed-off-by: NWenli Quan <wquan@redhat.com>

1. When netdst is switch, set specified mtu on host/client/guest
(linux/windows) before netperf start tests.
2. The end of jumbo test, set back to default mtu
3. Update all error to error_context
Signed-off-by: NWenli Quan <wquan@redhat.com>

Ksm uses a lot of processing power, we need to disable it. it takes
30% cpu usage (16cpus) in my tests.
Signed-off-by: NWenli Quan <wquan@redhat.com>

1.Remove irq_injs and rpkt_per_irq since it's not supported any
more in kvm state

2.Since powerpc doest not support io_exits in kvm state, uses exits
for all platform.

3.Fail to control last numa node by numactl tool in some ppc machine,
so switch to node 0 instead.
Signed-off-by: NWenli Quan <wquan@redhat.com>

Found and fixed by inspektor 0.4.5.
Signed-off-by: NLucas Meneghel Rodrigues <lookkas@gmail.com>

Ideally, guest needs to boot up with two nics, one is for connection,
the other is for testing. but in mostly guest, only one ONBOOT=yes
in configuration ifcfg-ethx, need to force to restart_network on vm.
Signed-off-by: NWenli Quan <wquan@redhat.com>

Rhel 7.4 uses systemctl to control services instead of service command
Signed-off-by: NWenli Quan <wquan@redhat.com>

Signed-off-by: NWenli Quan <wquan@redhat.com>

Signed-off-by: NMaosheng Zhang <mazhang@redhat.com>

Instead of get_download_dir(), which is really used to hold download
definition files, AKA as assets, the ".ini" files that define locations
and hashes such as the JeOS image files.
Signed-off-by: NCleber Rosa <crosa@redhat.com>

ID: 1315548 1315167
Signed-off-by: NMaosheng Zhang <mazhang@redhat.com>

Signed-off-by: NMaosheng Zhang <mazhang@redhat.com>

Clean up imports across the board in tp-qemu. The criteria
followed was:

1) Remove unnecessary imports
2) Turn multi imports from a single entry point to one
   import per line. This increases both patch and
   source code readability.
3) Separate blocks of imports with a line:
   * Python standard library imports
   * Autotest imports
   * virttest imports
   * Internal tp-qemu imports
4) Remove try/except blocks refering autotest libs and
   stick with the virttest.staging copy

This paves the way for the future removal of autotest
references, replacing those references with avocado ones.
Signed-off-by: NLucas Meneghel Rodrigues <lmr@redhat.com>

The new pep8 adds W503 line break before %s check. This is mass style
update of autotest framework without any actual changes.
Signed-off-by: NLukáš Doktor <ldoktor@redhat.com>

Add _netserver_started to judge if need setup netserver during test.
Signed-off-by: NYiqiao Pu <ypu@redhat.com>

Now the guest statys collect functions are implemented and tested with
RHEL and Fedora guests. It may cause problem for other guests. So set
it default to no, only enable it for RHEL and Fedora guests.
Signed-off-by: NYiqiao Pu <ypu@redhat.com>

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

Sometime the output from server include some unexpect letters include
some numbers. So the old method will failed to get the correct cpu
number.
Signed-off-by: NYiqiao Pu <ypu@redhat.com>

Signed-off-by: NAmos Kong <akong@redhat.com>

Currently we still wait client thread to stop after effective test
finishes, it wastes of time. This patch directly killes the threads,
then client_thread.join() will return immediately.
Suggested-by: NWenli Quan <wquan@redhat.com>
Signed-off-by: NAmos Kong <akong@redhat.com>

We do rounding in regression.py, so here we adjust the default
fbase to 5
Signed-off-by: NAmos Kong <akong@redhat.com>

User wants to collect sysinfo of host/guest, but different users
might have different requests, and many kinds of system might be
used. So I would like to use custom-built scripts to collect the
information that will make users happy.

User needs to add shell scripts to autotest/client/tests/virt/shared/scripts/,
and set the script path in config file.

eg:
 #log_hostinfo_script = scripts/rh_perf_log_hostinfo_script.sh
 #log_guestinfo_script = scripts/rh_perf_log_guestinfo_script.sh
 #log_guestinfo_exec = bash
 #log_guestinfo_path = /tmp/log_guestinfo.sh

Then the script output will be saved to $test.resultsdir/sysinfo,
analyzer can use 'diff' to compare the system information when
regression occurs.

I will send a pull request to add to example scripts to virt-tests repo.
Signed-off-by: NAmos Kong <akong@redhat.com>

  ethtool -L $ethname combined $N

  todo: enable multiqueue for Windows guest
Signed-off-by: NAmos Kong <akong@redhat.com>

When we enable multiple queue, there are multiple virtio
input/output interrupts in guest.

This patch counts interrupt for each queue, and output
a summary in result.

This patch also moves the volatile items to the end of
record list. Then the avg_update in perf.cfg should be
updated.
Signed-off-by: NAmos Kong <akong@redhat.com>

commit a84a267d fixed one typo, another typo also exists. It caused that
'rpkt_per_irq' and 'tpkt_per_exit' were lost in the result.
Signed-off-by: NAmos Kong <akong@redhat.com>

commit [1] introduced a regression, it wrongly update the server_ip
to ip address of control connection.

We have two nics, the first (virtio-net) is data connection, the second
nic (non virtio-net) is control connection.

[1] https://github.com/kongove/virt-test/commit/63ada005Signed-off-by: NAmos Kong <akong@redhat.com>

Signed-off-by: NAmos Kong <akong@redhat.com>

Some times when boot guest with two nics, the second nic can not
get ip address.
The wait_for_get_address will try to renew guest ip and try to get
guest ip via serial console. it is more safe than get_address
Signed-off-by: NYunping Zheng <yunzheng@redhat.com>

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

The largest timeout_netperf_start should be int(l) * 0.5,
if netperf threads start all at once, it will still need
int(l) * 0.5 time waste. Therefore, the netperf_start_timeout
overhead is not needed.
Signed-off-by: NWenli Quan <wquan@redhat.com>

During the review of the repository split, it was pointed
out that 'agnostic' was a rather strange word, so we'll go
with 'generic'. I opted for not rebasing the PR, since I
had many problems when trying to do so.
Signed-off-by: NLucas Meneghel Rodrigues <lmr@redhat.com>

This will make it easier when we're splitting these
tests into their own repositories.
Signed-off-by: NLucas Meneghel Rodrigues <lmr@redhat.com>

There's no need to have the name of the "run_<name>" method (main
test entry point) depend on the test name. So, for simplicity sake,
let's just name the test entry point "run".

I have decided to not fallback gracefully to the previous naming
convention as I believe this can lead to confusion in the future.
Signed-off-by: NCleber Rosa <crosa@redhat.com>

There is a background tcpdump thread that records DHCP leases
in the env["address_cache"] dictionary. Sometimes, this thread
might write to this dictionary while the environment object is
being saved, triggering a RuntimeError in virt-test.

So the solution involves:

1) Creating a save lock, (a threading reentrant lock object),
that is acquired by the env.save() method when it wants to
save env.data to a pickle file, and for methods that alter
env.data.

2) Move the tcpdump thread to a env object attribute, rather
than one of the keys of env.data, given that the lock object
can't be pickled.

With the implementation of env.save_lock, we can also fix
the problem identified by the locking unittest. The base of
the solution was Pavel's idea, including a good deal of the
implementation, so kudos to him.
Signed-off-by: NPavel Hrdina <phrdina@redhat.com>
Signed-off-by: NLucas Meneghel Rodrigues <lmr@redhat.com>

We compute rpkt_per_exit by tx_pkts, and compute tpkt_per_exit
by rx_pkts. It's wrong.
Signed-off-by: NAmos Kong <akong@redhat.com>