From Dan Kenigsberg danken@redhat.com:
Dan Kenigsberg has submitted this change and it was merged.
Change subject: vmdevices: stop using self.conf ......................................................................
vmdevices: stop using self.conf
We feed every Device object the vm.conf dict, but this is actually used in very few cases.
We fix those cases injecting the needed extra attributes in the device configuration, so the conf attribute is now useless and will be removed in a future patch.
Change-Id: Ib8918e8e073fc55f2bae3a74d6279bda062d9930 Signed-off-by: Francesco Romani fromani@redhat.com --- M lib/vdsm/virt/vmdevices/core.py M lib/vdsm/virt/vmdevices/graphics.py M lib/vdsm/virt/vmdevices/network.py M lib/vdsm/virt/vmdevices/storage.py M tests/device_test.py M tests/vm_test.py M tests/vmstorage_test.py M vdsm/virt/vm.py 8 files changed, 98 insertions(+), 36 deletions(-)
Approvals: Jenkins CI: Passed CI tests Dan Kenigsberg: Looks good to me, approved M.Z.: Looks good to me, but someone else must approve Francesco Romani: Verified Milan Zamazal: Looks good to me, but someone else must approve
vdsm-patches@lists.fedorahosted.org