When restoring the configurations of cpu affinity, some threads
would be no long existed, so need to ignore the errors emitted
by this operation.
Signed-off-by: NXu Han <xuhan@redhat.com>

Signed-off-by: NXu Han <xuhan@redhat.com>

Signed-off-by: NXu Han <xuhan@redhat.com>

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

'virttest.aexpect' was refactored and turned to its own project.
Let's update all the references to simply 'aexpect'
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>

Make virt-test to have a docstring style mostly compliant with
the sphinx's [1] format, reStructured text. This means that
we have to trade all '@' tags to be ':', mostly.
Signed-off-by: NLucas Meneghel Rodrigues <lmr@redhat.com>

With the help of the PEP8 script, a semi-automated clean
of the code base was made, and effectively changing some
of the coding style rules of our code base:

 * No more 2 lines spacing among functions
 * No more combined module imports

In the end, it was going to be too difficult to take
into consideration E501 (line length > 80) errors and
W601 (.has_key() is deprecated), so these were ignored,
at least for now.
Signed-off-by: NLucas Meneghel Rodrigues <lmr@redhat.com>

Add two parameters to let test case timedift.with_load
can be configured. When need run long time testing, can
reset the value "test_duration" & "interval_gettime".
Signed-off-by: NGolita Yue <gyue@redhat.com>

changes from v1:
 - add boot options for remove.

changes from v2:
 - also fix guest kernel cmdline in other timedrift testcases.

changes from v3:
 - add a judgement for boot options, don't run this step if there
   is nothing to be add or remove
Signed-off-by: NQingtang Zhou <qzhou@redhat.com>
Signed-off-by: NAmos Kong <akong@redhat.com>
Signed-off-by: NYiqiao Pu <ypu@redhat.com>
Acked-by: NFeng Yang <fyang@redhat.com>
Acked-by: Suqin Huang<shuang@redhat.com>

This patch benefits from the better error message when key is missing
in config params. Instead of custom handling of missing param it
uses params[] directly.

I modified those params, where I was 100% certain that they are
required and doesn't have default value:
 * val = params.get(..) followed by if not val: raise...
 * int(params.get(...))
 * params.get(...).split()
 * commands which require string (eg. session.cmd(params.get(...)))

I didn't test all the tests before and after in order to verify
possible typos but I used pylint before and after, watching for
new errors (code rate before and after).
Signed-off-by: NLukáš Doktor <ldoktor@redhat.com>

Assign the host load cmd to a variable, so we can get
its PID without resorting to using a index list -1 to
retrieve it later. This avoids a IndexError on Windows
guests.
Signed-off-by: NLucas Meneghel Rodrigues <lmr@redhat.com>

In order to be consistent with previous changes.
Signed-off-by: NLucas Meneghel Rodrigues <lmr@redhat.com>

A frequent request we've got from users is that the name
'kvm' makes assumptions that are not true for a lot of
the use cases of the test suite. Then let's rename
the subtest 'kvm' to 'qemu', making it more accurate.
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>