Ido Barkan has posted comments on this change. Change subject: net: tests: support iperf3 for performance tests ......................................................................
Patch Set 5: (2 comments) https://gerrit.ovirt.org/#/c/46448/5/lib/vdsm/constants.py.in File lib/vdsm/constants.py.in: Line 120: EXT_IPERF3 = '@IPERF3_PATH@' > Dan dislikes constants.py and we are phasing them out (but with no replacem I dislike constants.py as well, but what is the new convention? leave configure.ac unused? template nettestlib with ugly @@? https://gerrit.ovirt.org/#/c/46448/5/tests/nettestlib.py File tests/nettestlib.py: Line 39: CommandPath > You might want to list classes earlier than functions in import statements. I prefer fewer lines changed... -- To view, visit https://gerrit.ovirt.org/46448 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I15657f8844d131c5444dd680b8de7aa1c4ec2638 Gerrit-PatchSet: 5 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Ido Barkan <[email protected]> Gerrit-Reviewer: Ido Barkan <[email protected]> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Ondřej Svoboda <[email protected]> Gerrit-Reviewer: Petr Horáček <[email protected]> Gerrit-Reviewer: [email protected] Gerrit-HasComments: Yes _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
