docker: Pylint and pep8 fixes

no code changes, just style updates.

Signed-off-by: Lukáš Doktor <ldoktor@redhat.com>
dockertest.images & containers: Only check SubBase

Similar to change made for DockerCmd, to allow instances
implementing the common aspects of Subtest and SubSubtest
to be used interchangably.

Signed-off-by: Chris Evich <cevich@redhat.com>
dockertest.images: Switch to TextTable

Switch to using output.TextTable to parse
docker images output.  This is for consistency
with containers module and to help verify
header-column widths always matches content widths.

Also added ``images_args`` attribute in case
instances need to pass more/different arguments
than just ``--no-trunc``.  Removed a no-longer
relevant FIXME.

Signed-off-by: Chris Evich <cevich@redhat.com>
Merge pull request #193 from cevich/TextTable_images

dockertest.images: Switch to TextTable
Merge branch 'img_cnt_subbase' of https://github.com/cevich/autotest-docker

Conflicts:
	dockertest/images.py
docker: Add .travis.yml file fo Travis CI

This file configures the Travis CI to check pylint, pep8 and run all
unittests for each pull request/push

Signed-off-by: Lukáš Doktor <ldoktor@redhat.com>
Merge pull request #207 from ldoktor/travis

docker: Support for Travis CI
config: Remove run_volumes default option

Signed-off-by: Chris Evich <cevich@redhat.com>
subtests...stop/stopped: Fix config. bug.

Stdout for command 'true' was being incorrectly checked.

Signed-off-by: Chris Evich <cevich@redhat.com>
subtests...build_paths: Remove dep. config opt.

Signed-off-by: Chris Evich <cevich@redhat.com>
dockertest.subtests: Fix spaces in subsubtests CSV

Signed-off-by: Chris Evich <cevich@redhat.com>
dockertest.dockercmd: Print full command when not quiet

Instead of `docker run` print `docker run -t -i ...`. The length is
almost the same but the value of actual command is great.

With this patch the output looks like this:
quiet: (nothing)
nonverbose: Executing: docker run -t -i fedora bash
verbose: Execute docker run -t -i fedora bash
         (program is running)
         Command: ...
         Executed: ...
         ...

Signed-off-by: Lukáš Doktor <ldoktor@redhat.com>
subtests.docker_cli.kill: Pylint cleanup

Signed-off-by: Lukáš Doktor <ldoktor@redhat.com>
subtests.docker_cli.kill: Fix incorrect Ouput() usage and dkrcmd verbosity

Previously some dkrcmd.stdouts were replaced by Output(), which by
default resets the output (sets the index at the end of the output).
In logs no lines were printed (as no new lines occured after the call.
This patch adds the possibility to set idx and fixes this issue.

Additionally it replaces all Q*DockerCmds for non-verbose ones, whose
behavior was modified to suit (not only) this purpose.

Signed-off-by: Lukáš Doktor <ldoktor@redhat.com>
subtests.docker_cli.kill: Add go_lang_bad_signals test

This patch adds new test, which focuses on bad signals caused by golang
bug. Also it removes these signals from all other subtests.

This subtest is intended to be made dependent on the golang bug
and when it's resolved, we can start testing these signals.

Signed-off-by: Lukáš Doktor <ldoktor@redhat.com>
subtests.docker_cli.kill: Add run_sigproxy_stress_parallel

This test does the same as parallel_stress only it sends the signals
using sigproxy (signals directly to process)

NOTE: Currently this test generates couple of bugs (usually container
crash, sometimes daemon crash)

Signed-off-by: Lukáš Doktor <ldoktor@redhat.com>
subtests.docker_cli.kill: Add sigstop_sigproxy test

This test is similar to sigstop tests, only it uses sigproxy to send
the signals. The behavior is not always the same thus it's necessarily
to test booth ways.

Signed-off-by: Lukáš Doktor <ldoktor@redhat.com>
Merge pull request #235 from ldoktor/kill

subtests.docker_cli: Update and add couple of new kill tests
subtests...stop:  Update docs. to match config.

Signed-off-by: Chris Evich <cevich@redhat.com>
Merge pull request #211 from cevich/smallfixes

Smallfixes