Change in vdsm[ovirt-3.6]: net: always persist ifcfg files.

danken at redhat.com danken at redhat.com
Mon Aug 17 14:21:52 UTC 2015


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 <ibarkan at redhat.com>
Bug-Url: https://bugzilla.redhat.com/1252268
Reviewed-on: https://gerrit.ovirt.org/44929
Reviewed-by: Dan Kenigsberg <danken at redhat.com>
Continuous-Integration: Dan Kenigsberg <danken at redhat.com>
---
M vdsm/network/configurators/ifcfg.py
1 file changed, 5 insertions(+), 4 deletions(-)

Approvals:
  Ido Barkan: Verified
  Dan Kenigsberg: Looks good to me, approved; Passed CI tests



-- 
To view, visit https://gerrit.ovirt.org/44929
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I02ae28c345111309536e4af2bd6a09799f38b82f
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: ovirt-3.6
Gerrit-Owner: Ido Barkan <ibarkan at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Ido Barkan <ibarkan at redhat.com>
Gerrit-Reviewer: automation at ovirt.org


More information about the vdsm-patches mailing list