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

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

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

Signed-off-by: NLukáš Doktor <ldoktor@redhat.com>

* https://github.com/avocado-framework/avocado:
  SPEC file: visual aid for conditionals by means of indentation

Added methods to get total,online cpus count.
Signed-off-by: NSatheesh Rajendran <sathnaga@linux.vnet.ibm.com>

Let's modify the `make_broken` function to optionally accept
`description` which is used to report additional comments after the test
uid, instead of doing this manually per each call.

This also replaces the same parameter for the dummy `ignore_broken`
function which still contained other optional parameter not used by the
parallel `make_broken` function.
Signed-off-by: NLukáš Doktor <ldoktor@redhat.com>

Signed-off-by: NLukáš Doktor <ldoktor@redhat.com>

* https://github.com/avocado-framework/avocado:
  Lower stevedore version requirement

Signed-off-by: NLukáš Doktor <ldoktor@redhat.com>

* https://github.com/avocado-framework/avocado:
  avocado/utils/cpu.py: more compact implementation of get_cpu_arch()
  avocado/utils/cpu.py: small optimization on _get_cpu_info()

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

The current requirement set for the stevedore library doing build/use
of Avocado doesn't actually reflect the minimum version needed.

The proof is that Avocado on EL7, when installed from packages, will
run just fine with 0.14, the version that ships on that distro.

Without this change, package builds on EL7 attempt to download a newer
(Python) module:

    Processing dependencies for avocado-framework==50.0
    Searching for stevedore>=1.8.0
    Reading https://pypi.python.org/simple/stevedore/
    Best match: stevedore 1.21.0
    Downloading  https://pypi.python.org/[...]/stevedore-1.21.0.tar.gz[...]

With this change, no download is attempted:

    Processing dependencies for avocado-framework==50.0
    Searching for stevedore==0.14
    Best match: stevedore 0.14
    Adding stevedore 0.14 to easy-install.pth file
    Using /usr/lib/python2.7/site-packages
    Finished processing dependencies for avocado-framework==50.0

A couple of stevedore requirement mentions were preserved. Explanation:

 * requirements-docs.txt: it's used on readthedocs.org only
 * requirements-travis.txt: it's used Travis-CI only
Signed-off-by: NCleber Rosa <crosa@redhat.com>

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

This change may help to quickly read the conditionals.  It's
already been applied on Fedora packages, so let's sync it here.

Also added here is some small comment section changes (mostly
capitalization), again to sync with downstream.

CC: Lukáš Doktor <ldoktor@redhat.com>
Signed-off-by: NCleber Rosa <crosa@redhat.com>

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

Introducing the --force-execution to create the Job regardless the fact
that some test references might not be resolved to tests.

Reference: https://trello.com/c/bDUJjhq0Signed-off-by: NAmador Pahim <apahim@redhat.com>

This shouldn't change the behavior, but is more compact, and at least
for me, is more readable.

And, it happens to make (pure) pylint happier because there aren't
that many 'return' statements.
Signed-off-by: NCleber Rosa <crosa@redhat.com>

All functions that make use of _get_cpu_info() look for a given
line and return when the first one is found.  Because of that
there's no need to send the more or less identical entries for
other processes available on a system.
Signed-off-by: NCleber Rosa <crosa@redhat.com>

Signed-off-by: NLukáš Doktor <ldoktor@redhat.com>

* https://github.com/avocado-framework/avocado:
  Makefile: also remove 36lts "egg-info" dir

Currently all failures are ignorred, which makes it hard to actually
understand why our beloved test-reference is not a robot test. Let's
report !ROBOT line along with the exceptions details in such case.
Signed-off-by: NLukáš Doktor <ldoktor@redhat.com>

While debugging Avocado I noticed some crashes when attaching debugger.
It was caused by the debugger which force-flushes the STDOUT on some
occasions, which is usually supported, but is not supported by our
Paginator class. This commit adds the flush method which flushes the
output pipe (unless it's already closed)
Signed-off-by: NLukáš Doktor <ldoktor@redhat.com>

Currently we provide a short mapping to notify the user why is this
test_reference not a test along with coloring it by a different color.
Let's also include a shor message to the test name to explain why is it
so to improve the user experience.
Signed-off-by: NLukáš Doktor <ldoktor@redhat.com>

The Loader can report non-tests which are not intended for execution but
only for listing purposes. Let's avoid inheriting them from avocado Test
as it is not really necessary as the tests are basically dummy entities
which does not really do anything.
Signed-off-by: NLukáš Doktor <ldoktor@redhat.com>

When switching between 36lts and master branch, content from 36lts
(the .egg-info dir) may be kept.  The reason for that is that 36lts
used "avocado" as the module name, and thus produces
"avocado.egg-info" dir.
Signed-off-by: NCleber Rosa <crosa@redhat.com>

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

Make clean is missing the egg directories. This patch includes in make
clean a command to remove them.

Reference: https://trello.com/c/VgF0fkEaSigned-off-by: NAmador Pahim <apahim@redhat.com>

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

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

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

If state['params'] is nor present, let's generate the html report
anyway.
Signed-off-by: NAmador Pahim <apahim@redhat.com>

Signed-off-by: NLukáš Doktor <ldoktor@redhat.com>

* https://github.com/avocado-framework/avocado:
  spell: v50 fix

Signed-off-by: NLukáš Doktor <ldoktor@redhat.com>

* https://github.com/avocado-framework/avocado:
  Journal plugin: only attempt to close it if it has been initialized
  Human plugin: only print job results when job is successful

Currently we hardcode the "avocado.app" and "avocado.test" loggers
everywhere in Avocado, let's use the newly defined
`avocado.core.output.LOG_{UI,JOB}` variables everywhere in Avocado
except of `avocado.utils`, which shouldn't have used this interface in
the first place and will be subject of change in the future.
Signed-off-by: NLukáš Doktor <ldoktor@redhat.com>

On many places we are directly getting "avocado.app" and "avocado.test"
loggers and interacting with them. This can become an issue when we try
to replace or modify our logging streams in the future as well as it
requires some hidden knowledge. Let's define LOG_UI and LOG_JOB in
`avocado.core.output` to be available for Avocado core (and silently
also for plugin) developers.
Signed-off-by: NLukáš Doktor <ldoktor@redhat.com>

The `exceptions` module is not imported even though it's suppose to be
used.
Signed-off-by: NLukáš Doktor <ldoktor@redhat.com>

Sometimes we already have the logger so it makes a little sense to pass
logger name only to obtain the logger again in this function. Lets allow
both.
Signed-off-by: NLukáš Doktor <ldoktor@redhat.com>

The python root logger is '""' and not "root".
Signed-off-by: NLukáš Doktor <ldoktor@redhat.com>

Attempting to close a journal that wasn't initialized is certainly a
bug.  This will check for its initialization first.
Signed-off-by: NCleber Rosa <crosa@redhat.com>

The UI was showing information about the job, no matter if the job got
to run tests.

Even though the formal available statuses of a job are not too evolved
and documented, Avocado already sets a job status as "PASS" when it
finishes running the tests.  This involves most of the job phases.

Let's use that to check if the UI should print information about the
test results and job execution time.
Signed-off-by: NCleber Rosa <crosa@redhat.com>

Only one spell fix for this version.
Signed-off-by: NAmador Pahim <apahim@redhat.com>

Signed-off-by: NLukáš Doktor <ldoktor@redhat.com>

* https://github.com/avocado-framework/avocado:
  python-avocado.spec: include package ownership of "base dir" for "data dir"
  selftests: move InterruptTest.tests_well_behaved into level 1
  Travis-CI: run the extended set of tests, AKA "level 1"
  python-avocado.spec: only run "level 0" tests on package builds
  python-avocado.spec: also setup resultsdb plugin on %check
  selftests: introduce multiple levels of testing coverage

We used to use increased verbosity to get name of the test which left
tmp dirs behind, but it is not necessary. This commit decreases the
verbosity and only logs the error as well as test name in case this
failure is detected.
Signed-off-by: NLukáš Doktor <ldoktor@redhat.com>