On 11/04/2015 01:30 AM, Kevin Zhao wrote:
> ---
> virtinst/capabilities.py | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/virtinst/capabilities.py b/virtinst/capabilities.py
> index 4fa1724..10e7ea0 100644
> --- a/virtinst/capabilities.py
> +++ b/virtinst/capabilities.py
> @@ -364,7 +364,7 @@ class _CapsInfo(object):
> return "vexpress-a15"
>
> if self.arch in ["s390x"]:
> - if "s390-ccw-virtio" in self.machines:
> + if any(machine.startswith("s390-ccw") for machine in self.machines):
> return "s390-ccw-virtio"
>
> return None
>
Is this still needed? The capabilities XML you added to the test suite has the
proper s390-ccw-virtio mapping value... why do we need to check for the
s390-ccw prefix, and not the actual value we are going to specify in the XML?
- Cole
_______________________________________________
virt-tools-list mailing list
virt-tools-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/virt-tools-list
