This builds a new sub package for this new optional plugin.
Signed-off-by: NCleber Rosa <crosa@redhat.com>

It would be nice to have convinient way to upload results to centralized server.
This plugin superb resultsdb which support external log URL.
Proposed implementation is very simple, but it does it's job.
Signed-off-by: NDmitry Monakhov <dmonakhov@openvz.org>
Signed-off-by: NCleber Rosa <crosa@redhat.com>

Signed-off-by: NAmador Pahim <apahim@redhat.com>

This simple test, after the s390x version, checks the proper parsing
of and x86_64 /proc/cpuinfo.
Signed-off-by: NCleber Rosa <crosa@redhat.com>

The output in /proc/cpuinfo is different between s390x and other archs.
This is to support to parse the output on s390x.
Signed-off-by: NDan Zheng <dzheng@redhat.com>
Signed-off-by: NCleber Rosa <crosa@redhat.com>

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

Commit 51e46987 changed it from
`mux_debug` to `debug`, but using `debug` is actually dangerous if some
higher level plugin creates the same option, since it can make the
varianter plugin to use TreeNodeDebug instead of TreeNode.

Let's use a more exclusive naming, while still in line with the actual
implementation.
Signed-off-by: NAmador Pahim <apahim@redhat.com>

Signed-off-by: NAmador Pahim <apahim@redhat.com>

Signed-off-by: NAmador Pahim <apahim@redhat.com>

Fix the broken algorithm check in the hash_file function, it
should return `None` if the given hash algorithm is unknown.
Signed-off-by: NXu Han <xuhan@redhat.com>

Fix the dead loop caused by a wrong indentation in the
hash_file function.
Signed-off-by: NXu Han <xuhan@redhat.com>

The AvocadoParams, as a database of test parameters, has no need
to know information about the test.  In fact, the test id parameter
has never had any real use so far.

Let's remove it.
Signed-off-by: NCleber Rosa <crosa@redhat.com>

A lot of the core Avocado code still refers to "mux" based terminology.
For instance, what was previously "mux_path" is now "parameter paths",
and what was previously "mux" is now really variants.
Signed-off-by: NCleber Rosa <crosa@redhat.com>

While Avocado is running a test, it constantly requests the test to
send its state, making this operation a good candidate for some
optmization.

This change reduces the amount of attributes that are accessed and
serialized and returned as a test state to the barely minimum
observed, and at the same time, it documents it so that it will
hopefully start to serve as a documentation for the (private) protocol
that runners and tests use.

While at it, as a another smaller optmization, this list is defined at
the module level as a tuple (order doesn't matter here) and prevents
the list from being created on every single state construction.
Signed-off-by: NCleber Rosa <crosa@redhat.com>

The forward reference from the parameter module towards the job log
is not ideal.  The parameter module should be used by test (and
related code) and should not know about the test and job internals.

But, there's the clear requirement and convenience of logging the
access to the parameters.  This makes the user of the parameter
instance define which logger should be used, and not the other
way around.

Logger names are used here, because, the communication between
runner and test (get_state()) currently includes test parameters,
and serializing a logger instance is not possible.
Signed-off-by: NCleber Rosa <crosa@redhat.com>

Commit be8a5129 changed the style of open() and then read()/write()
calls.  But, we mistakenly dropped the write method on this one.
Signed-off-by: NCleber Rosa <crosa@redhat.com>

Commit ed4dcd1d missed this one attribute that reference the old
ResultProxy attribute.  Let's remove it.
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:
  Added required methods for ready-to-build source directory

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

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

Signed-off-by: NAmador Pahim <apahim@redhat.com>

We should only discover tests when we're able to run them. Otherwise we
should let the next available loader to attempt to discover them.
Signed-off-by: NAmador Pahim <apahim@redhat.com>

Let's make the _no_tests() message a parameter so we can re-use the
method with a different message.
Signed-off-by: NAmador Pahim <apahim@redhat.com>

the variater_pict plugin has no verbosity level other than the default,
which is not enough to expose the parameters content.

This patch enables higher verbosity so users can see the parameters
content when using `-c` or `--variants 2`(or above) in the `avocado
variants` command.
Signed-off-by: NAmador Pahim <apahim@redhat.com>

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

IMO, only calling `_prepend_sudo` if sudo support was requested makes
the code clearer.

Additionally, it adds the expected parameter type of sudo.
Signed-off-by: NCleber Rosa <crosa@redhat.com>

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

Commit 2806e740 missed these other two places that attempt to use
this backport.
Signed-off-by: NCleber Rosa <crosa@redhat.com>

The sysinfo code replicates some of the process handling code
already in avocado.utils.process.  Let's use our library instead
and remove code duplication.

This also marks the sysinfo interruption tests as "time and resource
sensitive", and will not run on environments such as Travis-CI, in
a similar way that other interruption based tests are currently set.
Signed-off-by: NCleber Rosa <crosa@redhat.com>

The to_str() method of the varianter_pict plugin is returning the header
string even when no variants are present.

This patch makes the to_str() method to return empty string on no
variants.
Signed-off-by: NAmador Pahim <apahim@redhat.com>

We are including double line break to separate the plugins variants even
if the plugin to_str() method retuns empty string.

This patch makes the varianter to include the double line break only if
the to_str() method returns non-empty string.
Signed-off-by: NAmador Pahim <apahim@redhat.com>

Signed-off-by: NAmador Pahim <apahim@redhat.com>

Signed-off-by: NAmador Pahim <apahim@redhat.com>

Patch adds required methods in YumBackend, ZypperBackend
and AptBackend for creating a ready-to-build directory
Signed-off-by: NHarish <harish@linux.vnet.ibm.com>

This is an attempt to catch most (if not all) usages of open
that do not follow the context manager pattern.

Sometimes, for better readability, our own `genio` library is
used.
Signed-off-by: NCleber Rosa <crosa@redhat.com>

Signed-off-by: NAmador Pahim <apahim@redhat.com>

Signed-off-by: NAmador Pahim <apahim@redhat.com>

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

* https://github.com/avocado-framework/avocado:
  Added two more functions to multipath.py library file

Actually the first~1 is necessary to check the first PR commit.
Signed-off-by: NLukáš Doktor <ldoktor@redhat.com>