Change in vdsm[master]: network: persistence: skip bonds owned and used by vdsm

danken at redhat.com danken at redhat.com
Fri Mar 13 11:28:43 UTC 2015


Dan Kenigsberg 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>
---
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/38587
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Idb643a0c3681631db7bd875a03ebdb91fd893e70
Gerrit-PatchSet: 8
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Petr Horáček <phoracek at redhat.com>
Gerrit-Reviewer: Antoni Segura Puimedon <celebdor at gmail.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: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server


More information about the vdsm-patches mailing list