Change in vdsm[ovirt-3.5]: network: persistence: skip bonds owned and used by vdsm

ybronhei at redhat.com ybronhei at redhat.com
Wed Mar 18 09:17:53 UTC 2015


Yaniv Bronhaim has submitted this change and it was merged.

Change subject: network: persistence: skip bonds owned and used by vdsm
......................................................................


network: persistence: skip bonds owned and used by vdsm

When upgrading bonds to unified persistence, we should skip foreign bond
(a bond that has ifcfg-bond lacking the "generated by vdsm" header).
However, if any of these bonds is used by a vdsm network, it must be
persisted. Otherwise, calling restore_net removes the bond (when it
cleans current running config) but it fails to re-create it from the
persistent config.

Change-Id: Idb643a0c3681631db7bd875a03ebdb91fd893e70
Signed-off-by: Petr Horáček <phoracek at redhat.com>
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1154399
Reviewed-on: https://gerrit.ovirt.org/38587
Reviewed-by: Dan Kenigsberg <danken at redhat.com>
Reviewed-on: https://gerrit.ovirt.org/38597
---
M lib/vdsm/tool/unified_persistence.py
A tests/netUnifiedPersistenceTests.py
2 files changed, 176 insertions(+), 2 deletions(-)

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



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

Gerrit-MessageType: merged
Gerrit-Change-Id: Idb643a0c3681631db7bd875a03ebdb91fd893e70
Gerrit-PatchSet: 4
Gerrit-Project: vdsm
Gerrit-Branch: ovirt-3.5
Gerrit-Owner: Petr Horáček <phoracek at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Ido Barkan <ibarkan at redhat.com>
Gerrit-Reviewer: Ondřej Svoboda <osvoboda 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


More information about the vdsm-patches mailing list