[virt-tools-list] [virt-manager PATCH] controller: don't set default controller index
Pavel Hrdina
phrdina at redhat.com
Tue Dec 3 12:07:17 UTC 2019
Libvirt is able to figure this out and it will make usage of the CLI
options more user-friendly.
For example if users wants to add a new pcie-root-port to existing VM
they have to figure out the latest controller index and call it like
this:
virt-xml \
--add-device \
--controller pci,model=pcie-root-port,index=$nextIndex \
$VM
After this change it will be simply:
virt-xml \
--add-device \
--controller pci,model=pcie-root-port \
$VM
Signed-off-by: Pavel Hrdina <phrdina at redhat.com>
---
virtinst/devices/controller.py | 9 ---------
1 file changed, 9 deletions(-)
diff --git a/virtinst/devices/controller.py b/virtinst/devices/controller.py
index 114d8b39..22e1311f 100644
--- a/virtinst/devices/controller.py
+++ b/virtinst/devices/controller.py
@@ -77,12 +77,3 @@ class DeviceController(Device):
driver_queues = XMLProperty("./driver/@queues", is_int=True)
master_startport = XMLProperty("./master/@startport", is_int=True)
-
-
- ##################
- # Default config #
- ##################
-
- def set_defaults(self, _guest):
- if self.index is None:
- self.index = 0
--
2.23.0
More information about the virt-tools-list
mailing list