Dan Kenigsberg has submitted this change and it was merged. Change subject: Cleanup the usages of configWriter in configNetwork ......................................................................
Cleanup the usages of configWriter in configNetwork We should keep the interaction with configWriter inside configurator. So this patch changes to construct new configWriter instantance in ifcfg's __init__() and use configurator to implement the transaction management of configuration. Signed-off-by: Mark Wu <[email protected]> Change-Id: I19310221ec2c5d7c621c121b77ed87d060676d42 Reviewed-on: http://gerrit.ovirt.org/16343 Reviewed-by: Dan Kenigsberg <[email protected]> Tested-by: Dan Kenigsberg <[email protected]> --- M vdsm/configNetwork.py M vdsm/netconf/ifcfg.py 2 files changed, 21 insertions(+), 31 deletions(-) Approvals: Dan Kenigsberg: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/16343 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I19310221ec2c5d7c621c121b77ed87d060676d42 Gerrit-PatchSet: 4 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Mark Wu <[email protected]> Gerrit-Reviewer: Antoni Segura Puimedon <[email protected]> Gerrit-Reviewer: Dan Kenigsberg <[email protected]> Gerrit-Reviewer: Mark Wu <[email protected]> Gerrit-Reviewer: oVirt Jenkins CI Server _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
