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>
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
Virsh update-device: tiny bugs fix

1). Remove 're' mode as it not used
2). libvirt driver name should be 'qemu' instead of 'file'
3). 'vmxml_devices' is a VMXMLDevices instance, can not use to output device
    xml directly.

Signed-off-by: Yanbing Du <ydu@redhat.com>
libvirt.tests: update_device enhancement

1. Support different disk type: cdrom, floppy
2. Support different disk mode: none, readonly, shareable
3. Cover more sut-tests
4. Tiny bugs fix

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