blockcopy: Resolve failing cases

Fail #1: blockcopy.positive_test.raw_format
For the raw format, the code mades a special check/call to QemuImg to
get the information about the blockcopy created.  The setup of the
params[] to the initializer was incorrect using 'image_filename'
instead of 'image_name' and the path to the file without the extension.

Had to adjust how the file was created to ensure it was created with
an extension ".raw"; otherwise, the 'image_format' would seem to default
to qcow2 which would be appended onto the "image_file' name passed.

Because the creation was adjusted to add the extenstion we have to
remove it when doing the check call...

Fail #2: blockcopy.negative_test.timeout_check
It seems libvirt commit id '4c297728' has resolved what used to be an
expected failure. The test passes for me now.  The commit in question
deals with blockcopy failing "unexpectedly" which would happen when
the timeout was fired and "quit" would be set in the code. So modify
the expected failure code to check for the expected failure message of
Copy aborted.  Additionally, add a bandwidth argument to hopefully
avoid the possibility that the copy completes very quickly and the
the message "Now in mirroring phase" is provided.
pool_create: Adjust location of --file pool

As of commit id 'c5f6c685' in virt-install (v1.0.0), virt-install
will attempt to create a storage pool for ease of management of
volumes. Thus if the test was started with --install or by chance
an 'images' pool exists using get_data_dir(), then the attempt
to define/create this pool will fail since two pools cannot use
the same path.  Therefore, rather than use the data dir for our
test, let's just use 'tmp' dir and create an images pool there.
domcontrol: fix process poll check problem

The process poll return None when process exist, else return negative
value if terminated. The check at cleanup got this wrong, so fix it.

Signed-off-by: Wayne Sun <gsun@redhat.com>
Add libvirtd start logging test case.

This case check error messages in libvirtd logging when libvirtd
starting.

Implemetent test cases:
with_iptables:  Simply start libvirtd when using iptables service
                as firewall.
with_firewalld: Simply start libvirtd when using firewalld service
                as firewall.

Signed-off-by: Hao Liu <hliu@redhat.com>
Merge pull request #97 from Hao-Liu/libvirtd

Add libvirtd logging test case.
tp-libvirt: Skip test if tgtadm is not found

When the optional binary `tgtadm` is not available on the host, we don't
expect the test case fail and should skip it instead.

Signed-off-by: Hao Liu <hliu@redhat.com>
Merge pull request #131 from jferlan/blockcopy

blockcopy: Resolve failing cases
Merge pull request #132 from jferlan/pool_create

pool_create: Adjust location of --file pool
Merge pull request #150 from waynesun09/domcontrol_fix

domcontrol: fix process poll check problem
Merge pull request #158 from Hao-Liu/fix_attach_detach_disk

tp-libvirt: Skip test if tgtadm is not found
libvirt.tests: Add a test for disk startupPolicy testing

libvirt domain disk XML may has a startupPolicy element:
    <disk type='file' device='floppy'>
      <source file='xxx' startupPolicy='mandatory'/>
      <target dev='fda' bus='fdc'/>
    </disk>
The value may be mandatory, requisite or optional, which corresponds to
differet behavior when start domain. This test case aim to test the three
different values for CD-ROM and floppy disks.

Signed-off-by: Yanbing Du <ydu@redhat.com>