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

phoracek at redhat.com phoracek at redhat.com
Fri Feb 19 11:41:47 UTC 2016


Petr Horáček has uploaded a new change for review.

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.

Change-Id: I76b6c13f97be58a7ad41268a44dee27241c8d2ab
Signed-off-by: Petr Horáček <phoracek at redhat.com>
---
M lib/vdsm/network/api.py
1 file changed, 2 insertions(+), 1 deletion(-)


  git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/44/53744/1

diff --git a/lib/vdsm/network/api.py b/lib/vdsm/network/api.py
index af85ca0..75e7b61 100644
--- a/lib/vdsm/network/api.py
+++ b/lib/vdsm/network/api.py
@@ -334,7 +334,8 @@
     elif not os.path.exists('/sys/class/net/' + netAttr['iface']):
         # Bridgeless broken network without underlying device
         libvirt.removeNetwork(network)
-        configurator.runningConfig.removeNetwork(network)
+        if config.get('vars', 'net_persistence') == 'unified':
+            configurator.runningConfig.removeNetwork(network)
         return
     _delNetwork(network, configurator=configurator, bypassValidation=True,
                 implicitBonding=False, _netinfo=_netinfo)


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I76b6c13f97be58a7ad41268a44dee27241c8d2ab
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Petr Horáček <phoracek at redhat.com>


More information about the vdsm-patches mailing list