Hi,

This patch tries to solve the following github issue,
to add other image types too and invalid image type will lead to fall back of 'raw' type
https://github.com/autotest/virt-test/issues/169

Please comment.
Signed-off-by: NSatheesh Rajendran <sathnaga@linux.vnet.ibm.com>

* Fixed incorrect order of parameters in __init__
* Removed extranious VMXML(virsh) call
* Utilized staticmethod of VMXML class
Signed-off-by: NChris Evich <cevich@redhat.com>

Let's move high-level functionality into vm class and away from virsh module.
This allows virsh module return value to be more supportive of error testing
and allow test code to examine result more closely.
Signed-off-by: NLi Yang <liyang.fnst@cn.fujitsu,com>

@ client/tests/virt/ $ ./run -c qemu/cfg/tests.cfg
  IOError: [Errno 20] Not a directory: '/home/devel/
  autotest/client/tests/virt/cfg/cfg/virtio-win.cfg'

The processed type name is 'cfg', it should be 'qemu'

I tried to execute run without 'type' parameter,
and got an error of wrong path of cfg file.
This patches fixed this problem. 'options.type' is
also used in other part, so assign the processed
type name to it.

v2: fix a wrong assignment
Signed-off-by: NAmos Kong <akong@redhat.com>

Fedora test release means the next major release of Fedora.
Now the test release is Fedora-18 Beta.
Signed-off-by: NQingtang Zhou <qzhou@redhat.com>

rhel-server-5.9-i386-dvd.iso   e47da2a9b05a97cea5663a30642ecdd4
rhel-server-5.9-x86_64-dvd.iso 83e601da2282073aef3fcbd03b50802f
Signed-off-by: NQingtang Zhou <qzhou@redhat.com>

Signed-off-by: NYu Mingfei <yumingfei@cn.fujitsu.com>

Signed-off-by: NYu Mingfei <yumingfei@cn.fujitsu.com>

Signed-off-by: NLi Yang <liyang.fnst@cn.fujitsu.com>

Signed-off-by: NLi Yang <liyang.fnst@cn.fujitsu.com>

Signed-off-by: NLi Yang <liyang.fnst@cn.fujitsu.com>

Signed-off-by: NLi Yang <liyang.fnst@cn.fujitsu.com>

Signed-off-by: NLucas Meneghel Rodrigues <lmr@redhat.com>

Signed-off-by: NLucas Meneghel Rodrigues <lmr@redhat.com>

Signed-off-by: NLucas Meneghel Rodrigues <lmr@redhat.com>

Signed-off-by: NLucas Meneghel Rodrigues <lmr@redhat.com>

Signed-off-by: NYu Mingfei <yumingfei@cn.fujitsu.com>

Signed-off-by: NYu Mingfei <yumingfei@cn.fujitsu.com>

Signed-off-by: NYu Mingfei <yumingfei@cn.fujitsu.com>

Signed-off-by: NYu Mingfei <yumingfei@cn.fujitsu.com>

Signed-off-by: NYu Mingfei <yumingfei@cn.fujitsu.com>

Since libguestfs has a serial of commands.
Support libguestfs test will be helpful.
Signed-off-by: NYu Mingfei <yumingfei@cn.fujitsu.com>

This patch adds a function in vm.verify_alive method,
to check if the image is bootable. If image can't bootup,
autotest will raise an ImageUnbootableError exception.

There are 2 parameters for this function:
 * image_verify_bootable: Turn on/off this checking function.
 * image_unbootable_pattern: Pattern for bootable checking.
Signed-off-by: NQingtang Zhou <qzhou@redhat.com>

This patch adds common variables into good-names pylint list.
Signed-off-by: NLukáš Doktor <ldoktor@redhat.com>

Qemu 1.3.50 changes cpu command line from -cpu ?cpuid,?dump,?models
to one command -cpu ?. -cpu ? command show joined information
only -cpu ?cpuid and -cpu ?models command. And developers don't
want add this functionality again because nobody use -cpu ?dump any more.
libvirt use file cpu_map.xml where should be all information about
cpu flags. And by this reason cpuflag test have to add this file too.
Moreover, it should help with check of qemu cpu flags stability.
Signed-off-by: NJiří Župka <jzupka@redhat.com>

* Added a verifyable default value for remote_ip/local_ip parameters
* Wraped two long lines
* Added a FIXME to catch-all 'except' statement
* Print out debugging info when there's a problem
* Add check for older libvirt version on command error
Signed-off-by: NChris Evich <cevich@redhat.com>

Signed-off-by: NLi Yang <liyang.fnst@cn.fujitsu.com>

Signed-off-by: NLi Yang <liyang.fnst@cn.fujitsu.com>

Signed-off-by: NLi Yang <liyang.fnst@cn.fujitsu.com>

Signed-off-by: NLucas Meneghel Rodrigues <lmr@redhat.com>

Add the ability to set the disk size in the virt disk
utility.
Signed-off-by: NLucas Meneghel Rodrigues <lmr@redhat.com>

Similarly to our cdrom creation code, we can replace
using loop devices for the mtools, that are pretty much
ubiquitous on linux distros.
Signed-off-by: NLucas Meneghel Rodrigues <lmr@redhat.com>

Add an asset file for the public virtio windows drivers
ISO. This will come handy for people that want to install
and test their windows guests using the drivers.
Signed-off-by: NLucas Meneghel Rodrigues <lmr@redhat.com>

Some assets may not have a url with SHA1sum of its files.
Let's just not check the SHA1sum then.
Signed-off-by: NLucas Meneghel Rodrigues <lmr@redhat.com>

So we only have to evaluate those once, reducing the time
to evaluate values by caching.
Signed-off-by: NLucas Meneghel Rodrigues <lmr@redhat.com>

I couldn't figure out how to automagically strip
test names in all occasions, but now we have a way:
find where is the guest name, and strip all the full
name until the guest name.

This technique is simple, effective and performing
(current guest name dict size is about 96 elements
in size), so let's go with it.
Signed-off-by: NLucas Meneghel Rodrigues <lmr@redhat.com>

So that we don't fall into the traps of 'shortnames',
that is, parts of the names that are not printed. For
example:

Win7.64

is in fact Win7.64.sp0, so if one puts only Win7.64 in
the command line, we'll execute both Win7.64.sp0 and
Win7.64.sp1. Printing a part of the full name avoids
that.
Signed-off-by: NLucas Meneghel Rodrigues <lmr@redhat.com>

Due to the new parsing of all options, we had a problem
on --list-tests that listed a huge amount of tests.

That's because we no longer have all only filters in
place that we used to have in tests.cfg. Therefore,
we must apply all cartesian parsing after the test
list is priinted.
Signed-off-by: NLucas Meneghel Rodrigues <lmr@redhat.com>

We did miss the fact that a bunch of our new command
line flags were breaking -c usage, given that those
new flags parsed "only" filters to the existing cartesian
configurations.

So reorganize the general options parsing, and carefully
review which options can go with -c and which ones will
be ignored, documenting this on --help output.

This also resolves one of the requests from Igor Mammedov,
of not compulsorily setting --qemu-bin when using custom
config files. So if -c is provided and no --qemu-bin was
provided in the command line, no attempt to set it will
be made. Of course, that was also explained in the --help
output.

CC: Igor Mammedov <imammedo@redhat.com>
Signed-off-by: NLucas Meneghel Rodrigues <lmr@redhat.com>