Shubhendu Tripathi has uploaded a new change for review.

Change subject: setup: Conditional statrup state for websocket proxy
......................................................................

setup: Conditional statrup state for websocket proxy

Modified to set the startup state during the cleanup
for the service ovirt-websocket-proxy only if it exists.
Unconditional code was causing an issue while cleanup
as the service details were not found and cleanup was
failing with error -

"[ ERROR ] Failed to execute stage 'Misc configuration': Command
'/sbin/chkconfig' failed to execute"

Change-Id: Icbafa3ed20b9ed9ed04a16e99cae7b4ae394db1c
Bug-Url: https://bugzilla.redhat.com/1095583
Signed-off-by: Shubhendu Tripathi <[email protected]>
---
M packaging/setup/plugins/ovirt-engine-remove/websocket_proxy/misc.py
1 file changed, 7 insertions(+), 4 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/90/27490/1

diff --git 
a/packaging/setup/plugins/ovirt-engine-remove/websocket_proxy/misc.py 
b/packaging/setup/plugins/ovirt-engine-remove/websocket_proxy/misc.py
index e5dd390..5ce91ee 100644
--- a/packaging/setup/plugins/ovirt-engine-remove/websocket_proxy/misc.py
+++ b/packaging/setup/plugins/ovirt-engine-remove/websocket_proxy/misc.py
@@ -41,10 +41,13 @@
         ],
     )
     def _misc(self):
-        self.services.startup(
-            name=osetupcons.Const.WEBSOCKET_PROXY_SERVICE_NAME,
-            state=False,
-        )
+        if self.services.exists(
+            name=name=osetupcons.Const.WEBSOCKET_PROXY_SERVICE_NAME
+        ):
+            self.services.startup(
+                name=osetupcons.Const.WEBSOCKET_PROXY_SERVICE_NAME,
+                state=False,
+            )
 
 
 # vim: expandtab tabstop=4 shiftwidth=4


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

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

Reply via email to