Simple syntatic fix removing one of the (duplicate) conditional
`__name__ == '__main__'` section.

To preserve the functionality of the exception hook
(`handle_exception`) the import of Avocado libraries are moved closed
to their usage (inside the condition `__main__` block).
Signed-off-by: NCleber Rosa <crosa@redhat.com>

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

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

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

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

New functions added for memory offline and online, check if memory is
hotpluggable, check if kernel supports memory hotplug.

These functions will be used more often by memory testcases in avocado-misc-tests

Implemented review comments from #2114, #2116
Signed-off-by: NAbdul Haleem <abdhalee@linux.vnet.ibm.com>

The current TestName class records more than a test name (according
to the Avocado specification).  It records, and by default, presents
the Test ID (__str__() and __repr__()).

The Test Name, again according to the Avocado specifications, is
contained within a Test ID, so it's not necessary to have two
different classes IMO.

Finally, the str_filesystem() method was adapted to be coherent with
the fact that it returns a file system representation of the Test ID,
not the Test Name.  It also became a property, to match the same use
of str_uid and str_variant.
Signed-off-by: NCleber Rosa <crosa@redhat.com>

There's a clear difference about of what a test file, and a test
itself, is in Avocado.  The datadir attribute though, has not been
created with such a difference in mind.

The best solution would be to renamed it to something like
"file_datadir", but since it's test a public (and stable) API, it's
not going to be changed anytime soon.

So, for now, let's clarify the meaning of the datadir property.
Signed-off-by: NCleber Rosa <crosa@redhat.com>

The current assignment of variant and str_variant attributes of a
TestName have two extra conditional blocks that are unnecessary.

Besides that, IMO, the proposed code is easier to follow.
Signed-off-by: NCleber Rosa <crosa@redhat.com>

Python code should not be doing type check/enforcement.

This bit was here temporarily, when the name parameter to
avocado.core.test.Test got changed.  Now it's way past time to remove
it.
Signed-off-by: NCleber Rosa <crosa@redhat.com>

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

The current example, on Test Names, where being given on the Test ID
section.  Let's move them closer to the Test Name section, and while
at it, also give Test ID examples.
Signed-off-by: NCleber Rosa <crosa@redhat.com>

A number of example files are not currently being packaged.  Let's
install and package them.
Signed-off-by: NCleber Rosa <crosa@redhat.com>

The current SPEC file changelog is getting really long.  I think
we can do better with a shorter version.
Signed-off-by: NCleber Rosa <crosa@redhat.com>

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

There's no need for this .gitignore file, the dir has other legit
files.
Signed-off-by: NCleber Rosa <crosa@redhat.com>

While a given resolver could return such a test name, I think this is
a bad example for a Test Name in this section.

The reason is that it's not possible, with the available test
resolvers/loaders to create a test such as that (it used to be).
Signed-off-by: NCleber Rosa <crosa@redhat.com>

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

The execute_cmd() method is not supposed to be public, to be used by
instances of the SimpleTest and ExternalRunnerTest class.

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

Because it's a Python standard module, and while it doesn't violate
any style rules, using a different name avoid any confusion.

While changing the parameter name, do a minor improvement of the
docstrings.
Signed-off-by: NCleber Rosa <crosa@redhat.com>

The vast majority of selftests do not use CamelCase, but some still
remain.  Let's stick to a single naming pattern.
Signed-off-by: NCleber Rosa <crosa@redhat.com>

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

* https://github.com/avocado-framework/avocado:
  Fix Travis check

With the EOL of Ubuntu 12.04, Travis CI is now running our tests using
Ubuntu 14.04. With this change, our check started to fail.

Even though I'm not sure about the reasons, I could make it work by
using a newer version of Pillow and also installing libvirt-python from
pip, since it's not working in the virtualenv if installed from apt.
Signed-off-by: NAmador Pahim <apahim@redhat.com>

utils.wait: docstring typo

Parameter is `step`, docstring says `steps`. Fixing.
Signed-off-by: NAmador Pahim <apahim@redhat.com>

This module has a number of functions aiming to retrieve and/or
convert values from /proc/meminfo. It's time to create a more robust
representation of /proc/meminfo. This commit creates a MemInfo class,
instantiated in a meminfo object.

The MemInfo class carries one attribute for each item in /proc/meminfo.
Each attribute is in fact an instance of the _MemInfoItem class, which
contains attributes to expose the item value. The basic 'value'
attribute will show the value as is. The additional attributes will
show the converted value, according to the selected unit.

Usage:

    >>> from avocado.utils import memory

    >>> memory.meminfo
    <avocado.utils.memory.MemInfo object at 0x7fe8e737f250>

    >>> dir(memory.meminfo)
    ['Active', 'Active_anon_', 'Active_file_', 'AnonHugePages',
    'AnonPages', 'Bounce', 'Buffers', 'Cached', 'CmaFree', 'CmaTotal',
    'CommitLimit', 'Committed_AS', 'DirectMap1G', 'DirectMap2M',
    'DirectMap4k', 'Dirty', 'HardwareCorrupted', 'HugePages_Free',
    'HugePages_Rsvd', 'HugePages_Surp', 'HugePages_Total', 'Hugepagesize',
    'Inactive', 'Inactive_anon_', 'Inactive_file_', 'KernelStack',
    'Mapped', 'MemAvailable', 'MemFree', 'MemInfoObject', 'MemTotal',
    'Mlocked', 'NFS_Unstable', 'PageTables', 'SReclaimable', 'SUnreclaim',
    'Shmem', 'ShmemHugePages', 'ShmemPmdMapped', 'Slab', 'SwapCached',
    'SwapFree', 'SwapTotal', 'Unevictable', 'VmallocChunk', 'VmallocTotal',
    'VmallocUsed', 'Writeback', 'WritebackTmp', '_MemInfo__meminfo',
    '__class__', '__delattr__', '__dict__', '__dir__', '__doc__',
    '__format__', '__getattr__', '__getattribute__', '__hash__',
    '__init__', '__iter__', '__module__', '__new__', '__reduce__',
    '__reduce_ex__', '__repr__', '__setattr__', '__sizeof__', '__str__',
    '__subclasshook__', '__weakref__']

    >>> memory.meminfo.MemTotal
    <avocado.utils.memory._MemInfoItem object at 0x7fe8f6ff8750>

    >>> dir(memory.meminfo.MemTotal)
    ['_MemInfoObject__units', '__call__', '__class__', '__delattr__',
    '__dict__', '__dir__', '__doc__', '__format__', '__getattr__',
    '__getattribute__', '__hash__', '__init__', '__module__', '__new__',
    '__reduce__', '__reduce_ex__', '__repr__', '__setattr__', '__sizeof__',
    '__str__', '__subclasshook__', '__weakref__', '_convert', 'name',
    'value', 'b', 'gb', 'kb', 'mb', 'tb']

    >>> memory.meminfo.MemTotal.value
    15853856

    >>> memory.meminfo.MemTotal.mb
    15482

    >>> for item in memory.meminfo:
    ...     print item.name, item.mb
    ...
    WritebackTmp 0
    SwapTotal 7744
    SwapFree 7744
    DirectMap4k 384
    KernelStack 5
    MemFree 10785
    HugePages_Rsvd 0
    Committed_AS 2108
    SUnreclaim 197
    NFS_Unstable 0
    VmallocChunk 0
    CmaFree 0
    Writeback 0
    MemTotal 15482
    VmallocUsed 0
    DirectMap1G 6144
    HugePages_Free 0
    AnonHugePages 0
    Active_file_ 1259
    AnonPages 440
    Inactive_anon_ 67
    Active 1719
    CommitLimit 15486
    Hugepagesize 2
    Active_anon_ 459
    Cached 2969
    SwapCached 0
    VmallocTotal 33554431
    CmaTotal 0
    Shmem 87
    Mapped 242
    ShmemPmdMapped 0
    Unevictable 0
    SReclaimable 536
    MemAvailable 13874
    Mlocked 0
    DirectMap2M 10310
    HugePages_Surp 0
    Bounce 0
    Inactive 1691
    PageTables 19
    HardwareCorrupted 0
    ShmemHugePages 0
    HugePages_Total 0
    Slab 734
    Inactive_file_ 1623
    Buffers 2
    Dirty 0
Signed-off-by: NAmador Pahim <apahim@redhat.com>

selftests: Two small selftest-related tweaks

This test is failing on my "slow" system, let's increase the timeouts,
they are used in `wait_for` so it shouldn't slow down the "quick"
systems.
Signed-off-by: NLukáš Doktor <ldoktor@redhat.com>

Recently we changed the inspekt execution to use the 0.4.5 syntax and
forgot to include this change in `requirements-selftest`. Fixing it now.
Signed-off-by: NLukáš Doktor <ldoktor@redhat.com>

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

* https://github.com/avocado-framework/avocado:
  travis: Update to inspektor 0.4.5
  Linter Fixes

Inspektor 0.4.5 brings a number of updates, including
porting to the cliff command line application framework,
which makes inspekt commands to look more natural in
terms of arguments.  Let's update and reflect that
in the CI scripts.
Signed-off-by: NLucas Meneghel Rodrigues <lookkas@gmail.com>

Found by the newest version of inspektor, 0.4.5.
Signed-off-by: NLucas Meneghel Rodrigues <lookkas@gmail.com>

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

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

Fixing spell error introduced recently.
Signed-off-by: NAmador Pahim <apahim@redhat.com>

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

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

* https://github.com/avocado-framework/avocado:
  YAML Loader: move it into its own plugin and package

The SIGINT/SIGTERM handlers are currently broken. When we execute a
SIMPLE TEST, which ignores SIGINT/SIGTERM, the test process will not
finish on SIGINT/SIGTERM, becoming orphan.

Reproducer::

    $ cat /tmp/test.py
    #!/usr/bin/env python
    import signal
    import time

    if __name__ == "__main__":
        signal.signal(signal.SIGINT, signal.SIG_IGN)
        signal.signal(signal.SIGTERM, signal.SIG_IGN)
        signal.signal(signal.SIGQUIT, signal.SIG_IGN)
        while True:
            time.sleep(0.1)

    $ avocado run /tmp/test.py

Then play with `Ctrl+C` (once and twice), with `kill -2 <pid>` and
`kill <pid>`. Check if Avocado hangs or whether any process is left
behind.

Also, the only test we have for those cases is supposed to catch the
issue with SIGINT (test_interrupt.InterruptTest.test_badly_behaved), but
it's not really working.

This patch fixes the SIGINT/SIGTERM actions, as follows:

- When a SIGINT is sent to the main main process, we send the same SIGINT
  to all the children hope for the best. This reproduces the behaviour
  on Ctrl+C.
- On a second SIGINT (after the 2s ignore period), meaning process is
  still running, we send a SIGKILL to all the children.
- When a SIGTERM is sent to the main process, we now send a SIGKILL to
  all the children (instead of a SIGTERM), so any subprocess ignoring
  SIGTERM will be terminated anyway.

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

The newly introduced YAML Loader plugin is currently hosted on the
same plugin (directory and module) as the varianter one.  While they
share a lot of code, they have different purposes.

Let's move the YAML loader into its own plugin, so that its
installation is also optional and the separation is clearer.
Signed-off-by: NCleber Rosa <crosa@redhat.com>

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