On 11/22/2017 09:58 PM, Lin Ma wrote: > Libvirt only supports built-in IDE controller so far, There is no > any case that needs us manually add an IDE controller, So remove > it from the controller list. > > Signed-off-by: Lin Ma <lma@xxxxxxxx> > --- > virtManager/addhardware.py | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/virtManager/addhardware.py b/virtManager/addhardware.py > index e563fe6..84bebec 100644 > --- a/virtManager/addhardware.py > +++ b/virtManager/addhardware.py > @@ -886,7 +886,8 @@ class vmmAddHardware(vmmGObjectUI): > model.clear() > > for t in VirtualController.TYPES: > - if t == VirtualController.TYPE_PCI: > + if (t == VirtualController.TYPE_PCI or > + t == VirtualController.TYPE_IDE): > continue > model.append([t, VirtualController.pretty_type(t)]) > > ACK and pushed with some formatting changes (using 'in [...]' instead of string comparisons Thanks, Cole _______________________________________________ virt-tools-list mailing list virt-tools-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/virt-tools-list
