[virt-tools-list] [virt-manager PATCH] addhardware: Don't clash with different buses using the same prefix

[Martin Kletzander]

On Tue, Feb 18, 2014 at 02:23:33PM +0100, Giuseppe Scrivano wrote:
> Martin Kletzander <mkletzan at redhat.com> writes:
>
> > Signed-off-by: Martin Kletzander <mkletzan at redhat.com>
> > ---
> >  virtManager/addhardware.py | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/virtManager/addhardware.py b/virtManager/addhardware.py
> > index b1f6251..b446a9d 100644
> > --- a/virtManager/addhardware.py
> > +++ b/virtManager/addhardware.py
> > @@ -1441,7 +1441,7 @@ class vmmAddHardware(vmmGObjectUI):
> >          # Save occupied places per controller
> >          occupied = collections.defaultdict(int)
> >          for d in used_disks:
> > -            if d.bus == disk.bus:
> > +            if d.get_target_prefix() == disk.get_target_prefix():
> >                  num = virtinst.VirtualDisk.target_to_num(d.target)
> >                  occupied[num / 7] += 1
> >          for c in ctrls_scsi:
>
> ACK
>
> Giuseppe

Thanks guys, pushed.

Martin
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://listman.redhat.com/archives/virt-tools-list/attachments/20140218/dfb7bdb8/attachment.sig>