From Dan Kenigsberg danken@redhat.com:
Dan Kenigsberg has posted comments on this change.
Change subject: vmdevices: use default if no displayNetwork ......................................................................
Patch Set 2:
(1 comment)
https://gerrit.ovirt.org/#/c/75134/2/lib/vdsm/virt/vmdevices/graphics.py File lib/vdsm/virt/vmdevices/graphics.py:
Line 235: # the device is defaulted to the network name (i.e. northbound port). Line 236: device = (nets[network].get('iface', network) Line 237: if network in nets else network) Line 238: ip, _, _, _ = net_api.ip_addrs_info(device) Line 239: # TODO: is this still an option with on-demand created networks?
I don't understand this TODO.
maybe I do. please rebase on top of https://gerrit.ovirt.org/#/c/75150/ Line 240: except (libvirt.libvirtError, KeyError, IndexError): Line 241: ip = config.get('addresses', 'guests_gateway_ip') Line 242: finally: Line 243: if ip == '':
vdsm-patches@lists.fedorahosted.org