Antoni Segura Puimedon has posted comments on this change. Change subject: BZ#831528, BZ#834008, BZ#829037 - Fix validation for inter-networks compatibility ......................................................................
Patch Set 3: (1 inline comment) Question about a test case I wrote that fails under this new implementation by accepting the addition of a bond consisting of interfaces that are already used as ports in a bridge. .................................................... File vdsm/configNetwork.py Line 601: About this removal. Shouldn't it be still checked, when a bond is requested (in the block between lines 610:616) that the interface is not already used in a bridge? -- To view, visit http://gerrit.ovirt.org/6384 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I1de66dea8cf73ecdde8e6651d4a2209be120a707 Gerrit-PatchSet: 3 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Igor Lvovsky <[email protected]> Gerrit-Reviewer: Antoni Segura Puimedon <[email protected]> Gerrit-Reviewer: Dan Kenigsberg <[email protected]> Gerrit-Reviewer: Igor Lvovsky <[email protected]> Gerrit-Reviewer: Livnat Peer <[email protected]> _______________________________________________ vdsm-patches mailing list [email protected] https://fedorahosted.org/mailman/listinfo/vdsm-patches
