Change in vdsm[master]: net: native ovs [2], fake bridgeless

edwardh at redhat.com edwardh at redhat.com
Tue Mar 22 06:09:22 UTC 2016


Edward Haas has posted comments on this change.

Change subject: net: native ovs [2], fake bridgeless
......................................................................


Patch Set 7: Code-Review-1

(1 comment)

https://gerrit.ovirt.org/#/c/54927/7/lib/vdsm/network/ovs_switch/caps.py
File lib/vdsm/network/ovs_switch/caps.py:

This is an 'embedded' lie.
I think we should let ovs report actual and have a normalize layer to adjust to the non perfect reality where we need to lie. Hopefully we will be able to drop the normalization later on.
(maybe we will find out that bridgless is indeed needed in ovs)
Line 1: # Copyright 2015-2016 Red Hat, Inc.
Line 2: #
Line 3: # This program is free software; you can redistribute it and/or modify
Line 4: # it under the terms of the GNU General Public License as published by


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I4bbf23d3864dfdf83d506a3b9885fc94eae01026
Gerrit-PatchSet: 7
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Petr Horáček <phoracek at redhat.com>
Gerrit-Reviewer: Edward Haas <edwardh at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Petr Horáček <phoracek at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>
Gerrit-HasComments: Yes


More information about the vdsm-patches mailing list