Fabian Deutsch has uploaded a new change for review.

Change subject: hook: Start vdsmd explicitly again
......................................................................

hook: Start vdsmd explicitly again

It's not really clear, but … If a dependency is brought down during a
service is queued, then the start job is canceled. And even if the
depdendency comes up again, the unit is not enqueued again.

Change-Id: I4796c96a8cd97d19beeafabc6c2a52b3817b9d12
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1156369
Signed-off-by: Fabian Deutsch <[email protected]>
---
M configure.ac
M hooks/Makefile.am
A hooks/on-boot/90-start-vdsm
M ovirt-node-plugin-vdsm.spec.in
4 files changed, 27 insertions(+), 3 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-node-plugin-vdsm 
refs/changes/50/34950/1

diff --git a/configure.ac b/configure.ac
index 3a2cfdd..73b113f 100644
--- a/configure.ac
+++ b/configure.ac
@@ -1,6 +1,6 @@
 define([VERSION_MAJOR], [0])
 define([VERSION_MINOR], [2])
-define([VERSION_FIX], [1])
+define([VERSION_FIX], [2])
 define([VERSION_NUMBER], VERSION_MAJOR[.]VERSION_MINOR[.]VERSION_FIX)
 define([VERSION_SUFFIX], [_ovirt35])
 
diff --git a/hooks/Makefile.am b/hooks/Makefile.am
index 5eb29fb..2d01579 100644
--- a/hooks/Makefile.am
+++ b/hooks/Makefile.am
@@ -30,4 +30,5 @@
 dist_onboot_SCRIPTS = \
        on-boot/01-vdsm-configure \
        on-boot/02-vdsm-sebool-config \
+       on-boot/90-start-vdsm \
        $(NULL)
diff --git a/hooks/on-boot/90-start-vdsm b/hooks/on-boot/90-start-vdsm
new file mode 100644
index 0000000..7f48e35
--- /dev/null
+++ b/hooks/on-boot/90-start-vdsm
@@ -0,0 +1,19 @@
+#!/bin/sh
+#
+# 90-start-vdsm - Copyright (C) 2014 Red Hat, Inc.
+#
+# This program is free software; you can redistribute it and/or modify
+# it under the terms of the GNU General Public License as published by
+# the Free Software Foundation, version 2 of the License.
+#
+# This program is distributed in the hope that it will be useful,
+# but WITHOUT ANY WARRANTY; without even the implied warranty of
+# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+# GNU General Public License for more details.
+
+# Explicitly start vdsmd again
+# https://bugzilla.redhat.com/show_bug.cgi?id=1156369
+
+service vdsmd start &
+
+exit 0
diff --git a/ovirt-node-plugin-vdsm.spec.in b/ovirt-node-plugin-vdsm.spec.in
index b9a4462..f2eb540 100644
--- a/ovirt-node-plugin-vdsm.spec.in
+++ b/ovirt-node-plugin-vdsm.spec.in
@@ -77,12 +77,16 @@
 %{_libexecdir}/ovirt-node/hooks/post-upgrade/01-sanlock-check
 %{_libexecdir}/ovirt-node/hooks/on-boot/01-vdsm-configure
 %{_libexecdir}/ovirt-node/hooks/on-boot/02-vdsm-sebool-config
+%{_libexecdir}/ovirt-node/hooks/on-boot/90-start-vdsm
 %{_sysconfdir}/ovirt-plugins.d
 %{_sysconfdir}/default/version.ovirt-node-plugin-vdsm
 
 %changelog
-* Wed Nov 04 2014 Fabian Deutsch <[email protected]> - 0.4.2
-- Move sanlock chek to post-upgrade hook rhbz#1156369
+* Fri Nov 07 2014 Fabian Deutsch <[email protected]> - 0.4.3
+- Explicitly start vdsmd again rhbz#1156369
+
+* Wed Nov 05 2014 Fabian Deutsch <[email protected]> - 0.4.2
+- Move sanlock check to post-upgrade hook rhbz#1156369
 
 * Thu Oct 23 2014 Fabian Deutsch <[email protected]> - 0.4.1
 - Add sebool handler rhbz#1150243 rhbz#1156038


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I4796c96a8cd97d19beeafabc6c2a52b3817b9d12
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-node-plugin-vdsm
Gerrit-Branch: ovirt-3.5
Gerrit-Owner: Fabian Deutsch <[email protected]>
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to