Petr Horáček has posted comments on this change. Change subject: net: native ovs: validate networks and bonds ......................................................................
Patch Set 6: (7 comments) https://gerrit.ovirt.org/#/c/55310/6/tests/network/ovs_test.py File tests/network/ovs_test.py: PS6, Line 31: test_net_only_one_untagged_network_no_initial_net > What about: test_adding_a_new_single_untagged_net Done PS6, Line 39: test_net_only_one_untagged_network_initial_net_to_be_edited > What about: test_edit_single_untagged_net_nic Done PS6, Line 47: test_net_only_one_untagged_network_setup_second_untagged_net > What about: test_adding_a_second_untagged_net Done PS6, Line 55: test_bond_at_least_two_slaves_no_slave > test_bond_with_no_slaves Done PS6, Line 60: test_bond_at_least_two_slaves_one_slave > test_bond_with_one_slave Done PS6, Line 65: test_bond_at_least_two_slaves_two_slaves > test_bond_with_two_slaves Done PS6, Line 70: test_bond_slaves_must_exist > test_bond_with_no_available_slaves not only they are not available (for example attached to another bonding), they are not existing -- To view, visit https://gerrit.ovirt.org/55310 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I5bceae0aada964c47aacfcfdbe93ca19eb8f3d14 Gerrit-PatchSet: 6 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: Petr Horáček <[email protected]> Gerrit-Reviewer: gerrit-hooks <[email protected]> Gerrit-HasComments: Yes _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
