Dan Kenigsberg has posted comments on this change. Change subject: netconf: Add config option for network configurator ......................................................................
Patch Set 11: Yes, it's fine to have only partial coverage by this new configurator, but I find it important to declare that clearly (in the commit message, or even in the config.py description). We would like to make sure that no one starts to blindly use it before it's done. -- To view, visit http://gerrit.ovirt.org/18210 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I426a38f229131ce6d32568387be7a809b09ae0c1 Gerrit-PatchSet: 11 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Mark Wu <[email protected]> Gerrit-Reviewer: Antoni Segura Puimedon <[email protected]> Gerrit-Reviewer: Assaf Muller <[email protected]> Gerrit-Reviewer: Dan Kenigsberg <[email protected]> Gerrit-Reviewer: Giuseppe Vallarelli <[email protected]> Gerrit-Reviewer: Mark Wu <[email protected]> Gerrit-Reviewer: oVirt Jenkins CI Server Gerrit-HasComments: No _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
