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