Attachment 886976 Details for Bug 1036515 – corresponding packaging patch for rhos4 openstack-glance

[patch] corresponding packaging patch for rhos4 openstack-glance

0001-improve-service-startup-with-sysv-initscript.patch (text/plain), 3.98 KB, created by Alan Pevec on 2014-04-16 20:23:21 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Alan Pevec

Created: 2014-04-16 20:23:21 UTC

Size: 3.98 KB

patch

obsolete

>From dfa7f9f57b77e1b5d1a39626859272f7b79d81dd Mon Sep 17 00:00:00 2001
>From: Alan Pevec <apevec@redhat.com>
>Date: Fri, 10 Jan 2014 03:39:20 +0100
>Subject: [PATCH 1/3] improve service startup with sysv initscript
>
>use systemd style readiness notification to signal when daemon is ready
>and avoid premature return from "daemon" command.
>
>Do not log to stderr by default.
>
>We are setting logfile explicitly so parallel output on stderr is not expected.
>This was creating duplicate log output to startup.log in daemon_notify.sh
>it was masked in previous initscripts with redirection to /dev/null.
>
>Fix for rhbz#1036515 in openstack-glance
>---
> openstack-glance-api.init      |  5 ++++-
> openstack-glance-registry.init |  5 ++++-
> openstack-glance.spec          | 11 +++++++++++
> 3 files changed, 19 insertions(+), 2 deletions(-)
>
>diff --git a/openstack-glance-api.init b/openstack-glance-api.init
>index 00c8a51..d36c243 100644
>--- a/openstack-glance-api.init
>+++ b/openstack-glance-api.init
>@@ -21,6 +21,9 @@ prog=openstack-glance-$suffix
> exec="/usr/bin/glance-$suffix"
> config="/etc/glance/glance-$suffix.conf"
> pidfile="/var/run/glance/glance-$suffix.pid"
>+startuplog="/var/log/glance/$prog-startup.log"
>+timeout=60
>+wrapper="/usr/share/glance/daemon_notify.sh"
> 
> [ -e /etc/sysconfig/$prog ] && . /etc/sysconfig/$prog
> 
>@@ -30,7 +33,7 @@ start() {
>     [ -x $exec ] || exit 5
>     [ -f $config ] || exit 6
>     echo -n $"Starting $prog: "
>-    daemon --user glance --pidfile $pidfile "$exec &>/dev/null & echo \$! > $pidfile"
>+    daemon --user glance --pidfile $pidfile "$wrapper $exec $pidfile $startuplog $timeout"
>     retval=$?
>     echo
>     [ $retval -eq 0 ] && touch $lockfile
>diff --git a/openstack-glance-registry.init b/openstack-glance-registry.init
>index 8a8850b..225b7a9 100644
>--- a/openstack-glance-registry.init
>+++ b/openstack-glance-registry.init
>@@ -21,6 +21,9 @@ prog=openstack-glance-$suffix
> exec="/usr/bin/glance-$suffix"
> config="/etc/glance/glance-$suffix.conf"
> pidfile="/var/run/glance/glance-$suffix.pid"
>+startuplog="/var/log/glance/$prog-startup.log"
>+timeout=60
>+wrapper="/usr/share/glance/daemon_notify.sh"
> 
> [ -e /etc/sysconfig/$prog ] && . /etc/sysconfig/$prog
> 
>@@ -30,7 +33,7 @@ start() {
>     [ -x $exec ] || exit 5
>     [ -f $config ] || exit 6
>     echo -n $"Starting $prog: "
>-    daemon --user glance --pidfile $pidfile "$exec &>/dev/null & echo \$! > $pidfile"
>+    daemon --user glance --pidfile $pidfile "$wrapper $exec $pidfile $startuplog $timeout"
>     retval=$?
>     echo
>     [ $retval -eq 0 ] && touch $lockfile
>diff --git a/openstack-glance.spec b/openstack-glance.spec
>index c55c100..71dbd36 100644
>--- a/openstack-glance.spec
>+++ b/openstack-glance.spec
>@@ -21,6 +21,7 @@ Source5:          glance-api-dist.conf
> Source6:          glance-registry-dist.conf
> Source7:          glance-cache-dist.conf
> Source8:          glance-scrubber-dist.conf
>+Source9:          daemon_notify.sh
> 
> #
> # patches_base=2013.2.3+1
>@@ -48,6 +49,11 @@ Requires(post):   chkconfig
> Requires(preun):  initscripts
> Requires(preun):  chkconfig
> Requires(pre):    shadow-utils
>+
>+# for daemon_notify
>+Requires: /usr/bin/uuidgen
>+Requires: /bin/sleep
>+
> Requires:         python-glance = %{version}-%{release}
> Requires:         python-glanceclient >= 1:0
> Requires:         openstack-utils
>@@ -212,6 +218,9 @@ install -p -m 644 %{SOURCE100} %{buildroot}%{_datadir}/glance/
> install -p -m 644 %{SOURCE200} %{buildroot}%{_datadir}/glance/
> install -p -m 644 %{SOURCE300} %{buildroot}%{_datadir}/glance/
> 
>+# Install service readiness wrapper
>+install -p -D -m 755 %{SOURCE9} %{buildroot}%{_datadir}/glance/daemon_notify.sh
>+
> # Logrotate config
> install -p -D -m 644 %{SOURCE4} %{buildroot}%{_sysconfdir}/logrotate.d/openstack-glance
> 
>@@ -313,6 +322,8 @@ fi
> %{_datadir}/glance/openstack-glance-registry.upstart
> %{_datadir}/glance/openstack-glance-scrubber.upstart
> 
>+%{_datadir}/glance/daemon_notify.sh
>+
> %{_datadir}/glance/glance-api-dist.conf
> %{_datadir}/glance/glance-registry-dist.conf
> %{_datadir}/glance/glance-cache-dist.conf
>-- 
>1.9.0
>

Actions: View | Diff

Attachments on bug 1036515: 886114 | 886273 | 886948 | 886960 | 886961 | 886962 | 886976 | 886978 | 887135