Antoni Segura Puimedon has posted comments on this change.
Change subject: WIP: Add netcf support ......................................................................
Patch Set 1:
I have not watched the code much yet. But I wonder if it would not be a good idea to have the native configurator and the netcf configurator in separate files from configNetwork.py.
What do you, Igor and Mark, think?
-- To view, visit http://gerrit.ovirt.org/8618 To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment Gerrit-Change-Id: I0070ea45de9d75570168410913a0aef62bcd940a Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Mark Wu wudxw@linux.vnet.ibm.com Gerrit-Reviewer: Antoni Segura Puimedon asegurap@redhat.com Gerrit-Reviewer: Igor Lvovsky ilvovsky@redhat.com