Dan Kenigsberg has submitted this change and it was merged. Change subject: net: always persist ifcfg files. ......................................................................
net: always persist ifcfg files. Currently the persistence of ifcfg files is only done during setSafeNetworkConfig. But, if the user somehow opts out of this (by unticking this option in engine, or if setupNetwork is called out of this flow (e.g. vdsm-restore-net-config) then vdsm unpersists the ifcfg files. If then the host reboots without persisting the unpersisted ifcfg files, those files are gone during boot since they are not part of the original file system. Change-Id: I02ae28c345111309536e4af2bd6a09799f38b82f Signed-off-by: Ido Barkan <[email protected]> Reviewed-on: https://gerrit.ovirt.org/44878 Continuous-Integration: Jenkins CI Reviewed-by: Fabian Deutsch <[email protected]> Reviewed-by: Dan Kenigsberg <[email protected]> --- M vdsm/network/configurators/ifcfg.py 1 file changed, 5 insertions(+), 4 deletions(-) Approvals: Fabian Deutsch: Looks good to me, but someone else must approve Ido Barkan: Verified Jenkins CI: Passed CI tests Dan Kenigsberg: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/44878 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I02ae28c345111309536e4af2bd6a09799f38b82f Gerrit-PatchSet: 3 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Ido Barkan <[email protected]> Gerrit-Reviewer: Dan Kenigsberg <[email protected]> Gerrit-Reviewer: Fabian Deutsch <[email protected]> Gerrit-Reviewer: Ido Barkan <[email protected]> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: [email protected] _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
