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