Aravinda VK has uploaded a new change for review.

Change subject: tuned: Set tuned profile issue due to GlusterEnv.ENABLE env var 
issue
......................................................................

tuned: Set tuned profile issue due to GlusterEnv.ENABLE env var issue

At the time of STAGE_SETUP of tuned, env variable odeploycons.GlusterEnv.ENABLE
is not yet set, so self.enabled(tuned) is set to True and hence tuned-adm
profile is set to "virtual-host"

Added additional condition in STAGE_MISC of tuned to prevent setting
tuned profile as "virtual-host" if it is a gluster enabled.

Change-Id: I1973b00816be20e8dec4863f18cbad58fe1200e2
Signed-off-by: Aravinda VK <[email protected]>
---
M src/plugins/ovirt-host-deploy/tune/tuned.py
1 file changed, 6 insertions(+), 1 deletion(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-host-deploy 
refs/changes/16/17416/1

diff --git a/src/plugins/ovirt-host-deploy/tune/tuned.py 
b/src/plugins/ovirt-host-deploy/tune/tuned.py
index 4caa267..d493603 100644
--- a/src/plugins/ovirt-host-deploy/tune/tuned.py
+++ b/src/plugins/ovirt-host-deploy/tune/tuned.py
@@ -64,7 +64,12 @@
 
     @plugin.event(
         stage=plugin.Stages.STAGE_MISC,
-        condition=lambda self: self.enabled,
+        condition=lambda self: (
+            self.enabled and
+            not self.environment[
+                odeploycons.GlusterEnv.ENABLE
+                ]
+        ),
     )
     def _misc(self):
         # tuned-adm does not work if daemon is down!


-- 
To view, visit http://gerrit.ovirt.org/17416
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I1973b00816be20e8dec4863f18cbad58fe1200e2
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-host-deploy
Gerrit-Branch: master
Gerrit-Owner: Aravinda VK <[email protected]>
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to