Alex Lourie has uploaded a new change for review.

Change subject: packaging: Running systemctl reload before starting service
......................................................................

packaging: Running systemctl reload before starting service

This patch runs "systemctl --system daemon-reload" before
starting the ovirt-engine service. Without this call, the
service is not available and the upgrade flow doesn't finish
correctly.

Change-Id: If36d81fe89218fb3078b281263df5c89907d900f
Signed-off-by: Alex Lourie <alou...@redhat.com>
---
M packaging/fedora/setup/engine-upgrade.py
1 file changed, 17 insertions(+), 0 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/15/15215/1

diff --git a/packaging/fedora/setup/engine-upgrade.py 
b/packaging/fedora/setup/engine-upgrade.py
index cc2ad75..adc32c6 100755
--- a/packaging/fedora/setup/engine-upgrade.py
+++ b/packaging/fedora/setup/engine-upgrade.py
@@ -137,6 +137,7 @@
 MSG_INFO_UPDATE_ENGINE_PROFILE="Updating ovirt-engine Profile"
 MSG_INFO_PKI_PREPARE = "Preparing CA"
 MSG_INFO_PKI_ROLLBACK = "Restoring CA"
+MSG_INFO_SYSTEMD_RELOAD = "Reloading services configuration"
 MSG_INFO_UUID_VALIDATION_INFO = (
     "NOTICE:\n"
     "\n"
@@ -1231,6 +1232,19 @@
                 os.chmod(config, 0o640)
 
 
+def reloadSystemd():
+    cmd = [
+        basedefs.EXEC_SYSTEMCTL,
+        "--system",
+        "daemon-reload",
+    ]
+
+    utils.execCmd(
+        cmdList=cmd,
+        failOnError=True,
+    )
+
+
 def main(options):
     # BEGIN: PROCESS-INITIALIZATION
     miniyumsink = utils.MiniYumSink()
@@ -1437,6 +1451,9 @@
                 # post install conf
                 runFunc(postFunc, MSG_INFO_RUN_POST)
 
+                # reload systemd services configuration
+                runFunc(reloadSystemd, MSG_INFO_SYSTEMD_RELOAD)
+
             except:
                 logging.error(traceback.format_exc())
                 logging.error("Rolling back update")


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: If36d81fe89218fb3078b281263df5c89907d900f
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Alex Lourie <alou...@redhat.com>
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to