The aexpect module is used by the Docker plugin.  Since we do not
package those separately, let's add it as a global Avocado dependency.
Signed-off-by: NCleber Rosa <crosa@redhat.com>

The `expire` argument was added later and there are usecases where it
was not used. Let's use default which keeps the original handling, which
is unlimited expiration.
Signed-off-by: NLukáš Doktor <ldoktor@redhat.com>

The main change is actually incorrect, the kernel.py library uses
`data_dirs` argument to determine the `cache_dirs`, which is by default
`self.work_dir`.

This reverts commit 36ea7be6.

avocado/utils/kernel.py: fix the use of the asset library [v2]

avocado/plugins/journal.py: make the goal of this plugin more clear

We found some users of this plugin, and we're unsure they actually use
this as intended.  Even if they don't use it as intended, it may be a
good thing, *but* we should make it clear what was the original goal
of this feature.
Signed-off-by: NCleber Rosa <crosa@redhat.com>

And the docstring of the asset library itself.
Signed-off-by: NCleber Rosa <crosa@redhat.com>

Use the avocado.utils.filelock to avoid race conditions:
 - Download the files with a temporary unique name.
 - Lock the original file.
 - Move downloaded file to the original name.
 - Compute the hash, creating the hashfile.
 - Verify the file against the provided hash.
 - Unlock the original file.
 (While the lock is acquired, users trying to use the file will wait until
 the lock is released or, on wait timeout, receive a cache miss)

Drop EnviromentError exceptions on cache miss. Instead, we now only
log an error message and return None.

Clean debug messages. Log was being polluted by asset fetcher. Let's
make it quieter.

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

This is a general purpose file locker.

If the lock file already exists and it contains the current process
id in it, we wait until the timeout for the lock to be released,
raising an AlreadyLocked exception when the timeout is reached.

If the lock file already exists and it contains a pid of another
running process, we raise an AlreadyLocked exception.

If the lock file exists and it has no running processes pid in it,
we try to clean the file and acquire the lock, raising a LockFailed
exception when something goes wrong.
Signed-off-by: NAmador Pahim <apahim@redhat.com>

This allows `AVOCADO_SELF_CHECK=1 ./selftests/checkall` to run
inside virtualenv environemnts.
Signed-off-by: NAmador Pahim <apahim@redhat.com>

Fedora 24 using screen has TERM=screen.xterm-256color. This patch adds
that terminal option in the list of valid options to be colorized.
Signed-off-by: NAmador Pahim <apahim@redhat.com>

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

Those test runners are implemented as TestResult classes, so they may
also signal their intention to use stdout.  If they don't, both the
RemoteTestResult/VMTestResult and HumanTestResult will end up using
the stdout.
Signed-off-by: NCleber Rosa <crosa@redhat.com>

The docker cleanup can sometimes fail, which masks the original failure.
This patch catches all exceptions and only log them to UI as warnings.
Signed-off-by: NLukáš Doktor <ldoktor@redhat.com>

The `docker run` command can actually take longer, when pulling the
image from hub. Let's remove the timeout completelly.
Signed-off-by: NLukáš Doktor <ldoktor@redhat.com>

The `docker run` command can actually produce several lines of output
(eg pulling the image) before it writes the container id. Let's use only
the last line when getting the cid.
Signed-off-by: NLukáš Doktor <ldoktor@redhat.com>

The cleanup fails when the remote docker connection is not established.
This skips the cleanup when the resource was not yet created.
Signed-off-by: NLukáš Doktor <ldoktor@redhat.com>

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

* https://github.com/avocado-framework/avocado:
  avocado.core.runner: Remove the unused setup/tear_down methods
  avocado.core.remote: Style fixes in runner
  avocado.core.remote: Improve the `tear_down` execution
  avocado.core.remote: Improve VMTestRunner tear_down method
  avocado.core.remote: Handle urls without assigned files
  avocado.plugins: Add plugin to run job in docker container

The `setup` and `tear_down` methods are defined in `TestRunner`, but
they are not handleded, nor used by it. This is dangerous, because even
from description they give the impression, that you can simply implement
them and they'll magically does the job.

Those methods are used inside `RemoteRunner` and they should only be
defined there.
Signed-off-by: NLukáš Doktor <ldoktor@redhat.com>

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

For some reason the RemoteTestRunner's tear_down is only called on
`setup` errors, or after the testing. This patch removes the special
handling and adds the `tear_down` into `try/finally` block starting with
`setup` and ending when the testing is done. This should make sure that
we always run `tear_down` in RemoteTestRunner.

This means there is no need for special handling in `VMTestRunner`
anymore so it has been removed too.
Signed-off-by: NLukáš Doktor <ldoktor@redhat.com>

The tear_down method can be currently called even before the `self.vm`
is initialized, producing an ugly traceback. This patch only cleanups
the vm when it's initialized and undefines it afterwards, so multiple
runs won't restore the snapshot multiple times.
Signed-off-by: NLukáš Doktor <ldoktor@redhat.com>

The remote runner url=>file is rather simple, it takes the url and if
the file does not exists, it uses test_dir + url. Fabric should copy
that file, but it silently ignores missing files, therefor (unless the
file already exists on the remote machine) it fails later in the code
with "Unable to discover ...".

This patch adds check to make sure the mapped file exists. If not it
raises exception.
Signed-off-by: NLukáš Doktor <ldoktor@redhat.com>

Similarly to --remote or --vm plugins this plugin allows one to run the
job inside a docker container by specifying the docker image. It
executes a new container, then attaches it and uses it similarly as
--remote plugin uses remote machine.

To check it out you can use "ldoktor/fedora-avocado" image which is
available on (the default) hub.docker.com
Signed-off-by: NCleber Rosa <crosa@redhat.com>
Signed-off-by: NLukáš Doktor <ldoktor@redhat.com>

This reverts commit 00e9aa46.

When the tests are not going to be copied, there's no need to try to
discover them locally.

This patch fakes the test_suite when remote-no-copy is used.
Signed-off-by: NAmador Pahim <apahim@redhat.com>

* https://github.com/avocado-framework/avocado:
  docs: remove mentions of tagged_name test property
  avocado/core/test.py: remove support for string test names and tags

* https://github.com/avocado-framework/avocado:
  Result: use FileOrStdoutAction on result plugins and app
  Parser: introduce FileOrStdoutAction

This change replaces the in-job logic to deal with the standard output
with the FileOrStdoutAction that does it at the command line parsing
time.  Then, it deals with a possible command line option conflict (the
first two players that attempt to use stdout) at the application level,
exiting the application in such case.
Signed-off-by: NCleber Rosa <crosa@redhat.com>