From Dan Kenigsberg <danken(a)redhat.com>:
Dan Kenigsberg has submitted this change and it was merged.
Change subject: virt net: Create libvirt network only when needed
......................................................................
virt net: Create libvirt network only when needed
The libvirt network is needed only for VM/s with graphics, therefore its
creation and deletion should be dependent directly on it.
On each graphic entity setup, the network will be created if it has not
existed before.
Deletion will occur on teardown if the current instance is the last
one which uses the specific network.
This patch is focused on virt changes, the following patches drops
libvirt creation and deletion from the host networking code.
Change-Id: Ie55b08e9bdffcd8ea663b647c6d6dcf3363b9847
Signed-off-by: Edward Haas <edwardh(a)redhat.com>
---
M lib/vdsm/network/api.py
M lib/vdsm/network/libvirt.py
M lib/vdsm/virt/vmdevices/graphics.py
M tests/network/libvirt_test.py
4 files changed, 127 insertions(+), 1 deletion(-)
Approvals:
Jenkins CI: Passed CI tests
Petr Horáček: Looks good to me, but someone else must approve
Dan Kenigsberg: Looks good to me, approved
Francesco Romani: Looks good to me, but someone else must approve
Edward Haas: Verified
--
To view, visit
https://gerrit.ovirt.org/74390
To unsubscribe, visit
https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Ie55b08e9bdffcd8ea663b647c6d6dcf3363b9847
Gerrit-PatchSet: 8
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Edward Haas <edwardh(a)redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com>
Gerrit-Reviewer: Edward Haas <edwardh(a)redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Martin Polednik <mpolednik(a)redhat.com>
Gerrit-Reviewer: Petr Horáček <phoracek(a)redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>