Change in vdsm[master]: net: api: use running config only with unified persistence

danken at redhat.com danken at redhat.com
Fri Feb 19 14:41:44 UTC 2016


Dan Kenigsberg has submitted this change and it was merged.

Change subject: net: api: use running config only with unified persistence
......................................................................


net: api: use running config only with unified persistence

configurator.runningConfig should be used only when unified persistence
is used. If we call it with ifcfg persistence, vdsm explodes.

This patch fixes testBrokenNetworkReplacement.

Change-Id: I76b6c13f97be58a7ad41268a44dee27241c8d2ab
Signed-off-by: Petr Horáček <phoracek at redhat.com>
Reviewed-on: https://gerrit.ovirt.org/53744
Continuous-Integration: Jenkins CI
Reviewed-by: Dan Kenigsberg <danken at redhat.com>
---
M lib/vdsm/network/api.py
1 file changed, 2 insertions(+), 1 deletion(-)

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



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I76b6c13f97be58a7ad41268a44dee27241c8d2ab
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Petr Horáček <phoracek at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Edward Haas <edwardh at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Ondřej Svoboda <osvoboda at redhat.com>
Gerrit-Reviewer: Petr Horáček <phoracek at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>


More information about the vdsm-patches mailing list