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
[email protected]
https://www.redhat.com/mailman/listinfo/virt-tools-list

Reply via email to