Change in vdsm[ovirt-3.5]: network: store non-Vdsm ifcfgs with unified persistence

danken at redhat.com danken at redhat.com
Mon Mar 2 11:23:33 UTC 2015


Dan Kenigsberg has submitted this change and it was merged.

Change subject: network: store non-Vdsm ifcfgs with unified persistence
......................................................................


network: store non-Vdsm ifcfgs with unified persistence

Unified restoration first removes everything in the running config and
then tries to regenerate everything that's in the persisted config,
problem is, that original devices are not persisted.

In this patch we (temporarily) use ifcfg persistence for non-Vdsm devices
(even when unified persistence is set), so we are able to restore them.

Change-Id: Iaac0d82863334238e6db71b4a6a4f6b76a260223
Bug-Url: https://bugzilla.redhat.com/1197668
Signed-off-by: Petr Horáček <phoracek at redhat.com>
Reviewed-on: http://gerrit.ovirt.org/37453
Reviewed-by: Ondřej Svoboda <osvoboda at redhat.com>
Reviewed-by: Dan Kenigsberg <danken at redhat.com>
Reviewed-on: https://gerrit.ovirt.org/38240
---
M vdsm/network/configurators/ifcfg.py
M vdsm/vdsm-restore-net-config
M vdsm/vdsm-store-net-config.in
3 files changed, 25 insertions(+), 11 deletions(-)

Approvals:
  Petr Horáček: Verified
  Dan Kenigsberg: Looks good to me, approved



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

Gerrit-MessageType: merged
Gerrit-Change-Id: Iaac0d82863334238e6db71b4a6a4f6b76a260223
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: ovirt-3.5
Gerrit-Owner: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Ido Barkan <ibarkan at redhat.com>
Gerrit-Reviewer: Petr Horáček <phoracek at redhat.com>
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server


More information about the vdsm-patches mailing list