Change in vdsm[master]: iproute2, pyroute2: models.Bond checks for .unifiedPersisten...
osvoboda at redhat.com
osvoboda at redhat.com
Sun Aug 23 08:45:58 UTC 2015
Ondřej Svoboda has uploaded a new change for review.
Change subject: iproute2, pyroute2: models.Bond checks for .unifiedPersistence, so add it
......................................................................
iproute2, pyroute2: models.Bond checks for .unifiedPersistence, so add it
Commit 30a3fc25 fixed a condition in Bond.configure for ifcfg configurator
that has a .runningConfig member only when using unified persistence, but
broke the other two configurators that use unified persistence implicitly.
The simplest fix (and even self-documentation) is to add .unifiedPersistence
to them.
Change-Id: I5dccdd8ea585f4fcbf327f233e68e8fcc5f5133b
Signed-off-by: Ondřej Svoboda <osvoboda at redhat.com>
---
M vdsm/network/configurators/iproute2.py
M vdsm/network/configurators/pyroute_two.py
2 files changed, 2 insertions(+), 0 deletions(-)
git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/12/45212/1
diff --git a/vdsm/network/configurators/iproute2.py b/vdsm/network/configurators/iproute2.py
index 4f58fba..d20f77a 100644
--- a/vdsm/network/configurators/iproute2.py
+++ b/vdsm/network/configurators/iproute2.py
@@ -54,6 +54,7 @@
class Iproute2(Configurator):
def __init__(self, inRollback=False):
+ self.unifiedPersistence = True
super(Iproute2, self).__init__(ConfigApplier(), inRollback)
self.runningConfig = RunningConfig()
diff --git a/vdsm/network/configurators/pyroute_two.py b/vdsm/network/configurators/pyroute_two.py
index 1293310..f6937d2 100644
--- a/vdsm/network/configurators/pyroute_two.py
+++ b/vdsm/network/configurators/pyroute_two.py
@@ -43,6 +43,7 @@
class PyrouteTwo(Iproute2):
def __init__(self, inRollback=False):
+ self.unifiedPersistence = True
super(Iproute2, self).__init__(ConfigApplier(), inRollback)
self.runningConfig = RunningConfig()
--
To view, visit https://gerrit.ovirt.org/45212
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I5dccdd8ea585f4fcbf327f233e68e8fcc5f5133b
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Ondřej Svoboda <osvoboda at redhat.com>
More information about the vdsm-patches
mailing list