Dan Kenigsberg has submitted this change and it was merged. Change subject: avoid reading ifcfg files in netinfo ......................................................................
avoid reading ifcfg files in netinfo getting runtime network information is better via quering more standard netinfo api's rather than reading ifcfg configuration files which is more error prone. We need to confine reading ifcfg files only in the context of dealing with the ifcfg configurator. Change-Id: I28e2084a15e1f6cc0cb4f20e83d7751fd8ffe37f Signed-off-by: ibarkan <[email protected]> Reviewed-on: http://gerrit.ovirt.org/35043 Reviewed-by: Ondřej Svoboda <[email protected]> Tested-by: Dan Kenigsberg <[email protected]> Reviewed-by: Dan Kenigsberg <[email protected]> --- M lib/vdsm/netinfo.py M tests/netinfoTests.py M vdsm/network/api.py 3 files changed, 3 insertions(+), 98 deletions(-) Approvals: Ondřej Svoboda: Looks good to me, but someone else must approve Dan Kenigsberg: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/35043 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I28e2084a15e1f6cc0cb4f20e83d7751fd8ffe37f Gerrit-PatchSet: 3 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Ido Barkan <[email protected]> Gerrit-Reviewer: Dan Kenigsberg <[email protected]> Gerrit-Reviewer: Ido Barkan <[email protected]> Gerrit-Reviewer: Ondřej Svoboda <[email protected]> Gerrit-Reviewer: Petr Horáček <[email protected]> Gerrit-Reviewer: [email protected] Gerrit-Reviewer: oVirt Jenkins CI Server _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
