Logging audit for trove/configuration module

Adjust log messages to conform to logging standards.

Change-Id: I16e48928d7a710e2ba31524be02d358b4e1dd713
Partial-Bug: #1324206
Handle exception from pexpect child.close

When pexpect child.close(force=True) is called to close a sudo process, an
exception may be raised on certain operating systems. Terminating a sudo
process as a non-root user succeeds on Ubuntu 12.04 but fails on 14.04.

Handle this exception and try to kill the child process using 'sudo kill'.

See related change https://review.openstack.org/#/c/100353 that addresses
another instance of the same issue.

Change-Id: I4bd9e4b98479e33604545713337440ce81ca64ee
Partial-Bug: 1343344
guestagent/volume: Remove not necessary sudo call

The sudo call here is not required because `tempfstab.name` is created
by trove user.

Closes-bug: 1347466
Change-Id: Id0d24ea495a6a7bfd4e1a4355a08b58b96ae8a25
Handle error from execute() when deleting non-existant file

Code inspection around an exception found a piece of code handling an
error from execute_with_timeout() in a problematic way. A call was
made to delete a file and if the file did not exist,
execute_with_timeout() would throw an exception. However, 'rm' is not
very friendly, and only returns 0, -1, +1 and not errno
(ENOENT). Therefore the code was (literally) looking in stdout for the
string 'No such file or directory'.

The recommended way of handling missing files with rm is to use the
command line '-f' option which will suppress errors on nonexistent
files. The mechanism of searching for a literal string is not good for
a couple of reasons but the most significant two are that it requires
a thrown exception (which would get logged as an exception), and that
in languages other than English, the error string would be something
else (not 'No such file ...').

A test to ensure that wipe_ib_logfiles() properly suppressed the
exception is now redundant and has been deleted.

Change-Id: Ie99a0843d1e388d508a904fca0a759ca8bcd9f26
Partial-Bug: #1346879
Make configuration tests configurable by datastore

Adds flexibility to configuration tests to make used mysql configs
configurable.

Change-Id: Id7ef621c185af6514b87ec4f79001461bc3851c3
Closes-bug: 1334888
fix pexpect.spawn.match AttributeError

Jenkins randomly throws AttributeErrors on pexpect.spawn.match, mock the
return value to fix this.

Co-Authored-By: Denis M. <dmakogon@mirantis.com>
Change-Id: I8b0ed7f8c474bedbbd60944bdd5a4119b0a6faea
Closes-Bug: #1348601
Merge "guestagent/volume: Remove not necessary sudo call"
Merge "Make configuration tests configurable by datastore"
Merge "Handle exception from pexpect child.close"
Merge "Logging audit for trove/configuration module"
Merge "fix pexpect.spawn.match AttributeError"
Merge "Handle error from execute() when deleting non-existant file"