Edward Haas has posted comments on this change.

Change subject: net: get network switch type from libvirt
......................................................................


Patch Set 1: Code-Review-1

I am not clear why libvirt DB is considered 'actual' or real time info.
To me, it is not better then our simple running config.

In general, we would like to be independent of libvirt network DB, so in this 
spirit we better not add to it.

If we would like to be more accurate (we are :)), we should check against 
ovs.netinfo, if the network appears there.
ovs.netinfo should use running config and validate against ovsdb for the 
existence of the network.

-- 
To view, visit https://gerrit.ovirt.org/56838
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Ie733d39efa0fd40a81d3111d55fa19dde9a3c1b6
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Petr Horáček <[email protected]>
Gerrit-Reviewer: Dan Kenigsberg <[email protected]>
Gerrit-Reviewer: Edward Haas <[email protected]>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: gerrit-hooks <[email protected]>
Gerrit-HasComments: No
_______________________________________________
vdsm-patches mailing list
[email protected]
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Reply via email to