Giuseppe Vallarelli has posted comments on this change. Change subject: net_functional_tests: bondWithManyVlans. ......................................................................
Patch Set 2: I would prefer that you didn't submit this I appreciate that you're adding new tests and they are well written, but let's try to make them as readable and clean as possible otherwise with the time and new tests they will get uglier and not much of a pleasure to work with them. Thanks. -- To view, visit http://gerrit.ovirt.org/16025 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ie9da7bd407cfbd9acab5e20384bff1254c26ffc5 Gerrit-PatchSet: 2 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Antoni Segura Puimedon <asegu...@redhat.com> Gerrit-Reviewer: Antoni Segura Puimedon <asegu...@redhat.com> Gerrit-Reviewer: Dan Kenigsberg <dan...@redhat.com> Gerrit-Reviewer: Giuseppe Vallarelli <gvall...@redhat.com> Gerrit-Reviewer: Livnat Peer <lp...@redhat.com> Gerrit-Reviewer: Mark Wu <wu...@linux.vnet.ibm.com> Gerrit-Reviewer: oVirt Jenkins CI Server _______________________________________________ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches