[Openstack-devel] Bug#706013: nova: init scripts should have dependencies on db backends

[Julien Cristau]

Source: nova
Version: 2012.1.1-18
Severity: important

When nova is configured to use postgresql (or, I guess, anything but
sqlite), its services need to start after the dbms.  See below.

Cheers,
Julien

From: Julien Cristau <julien.cristau at logilab.fr>
Date: Tue, 23 Apr 2013 16:19:57 +0200
Subject: [PATCH 2/2] Add optional postgresql dependency to a number of init script

Should ensure proper startup ordering if nova is configured to use a
local postgresql backend.
---
 debian/changelog                          |    3 +++
 debian/nova-cert.init                     |    2 ++
 debian/nova-console.init                  |    2 ++
 debian/nova-console.nova-consoleauth.init |    2 ++
 debian/nova-scheduler.init                |    2 ++
 debian/nova-volume.init                   |    2 ++
 6 files changed, 13 insertions(+), 0 deletions(-)

diff --git a/debian/changelog b/debian/changelog
index a52eb60..94556e0 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -1,6 +1,9 @@
 nova (2012.1.1-19) UNRELEASED; urgency=low
 
   * Fix logrotate config for nova-xvpvncproxy and nova-consoleauth.
+  * Add optional postgresql dependency to a number of init script to ensure
+    proper startup ordering if nova is configured to use a local postgresql
+    backend.
 
  -- Julien Cristau <jcristau at debian.org>  Tue, 23 Apr 2013 16:08:19 +0200
 
diff --git a/debian/nova-cert.init b/debian/nova-cert.init
index b8d822e..f86003b 100644
--- a/debian/nova-cert.init
+++ b/debian/nova-cert.init
@@ -3,6 +3,8 @@
 # Provides:          nova-cert
 # Required-Start:    $network $local_fs $remote_fs $syslog
 # Required-Stop:     $remote_fs
+# Should-Start:      postgresql
+# Should-Stop:       postgresql
 # Default-Start:     2 3 4 5
 # Default-Stop:      0 1 6
 # Short-Description: Nova Cert server
diff --git a/debian/nova-console.init b/debian/nova-console.init
index 399354e..a389573 100644
--- a/debian/nova-console.init
+++ b/debian/nova-console.init
@@ -3,6 +3,8 @@
 # Provides:          nova-console
 # Required-Start:    $network $local_fs $remote_fs $syslog
 # Required-Stop:     $remote_fs
+# Should-Start:      postgresql
+# Should-Stop:       postgresql
 # Default-Start:     2 3 4 5
 # Default-Stop:      0 1 6
 # Short-Description: Nova Console for XenServer and XVP
diff --git a/debian/nova-console.nova-consoleauth.init b/debian/nova-console.nova-consoleauth.init
index 1a5f2a5..6eea7ea 100644
--- a/debian/nova-console.nova-consoleauth.init
+++ b/debian/nova-console.nova-consoleauth.init
@@ -3,6 +3,8 @@
 # Provides:          nova-consoleauth
 # Required-Start:    $network $local_fs $remote_fs $syslog
 # Required-Stop:     $remote_fs
+# Should-Start:      postgresql
+# Should-Stop:       postgresql
 # Default-Start:     2 3 4 5
 # Default-Stop:      0 1 6
 # Short-Description: Nova Console auth system for XenServer and XVP
diff --git a/debian/nova-scheduler.init b/debian/nova-scheduler.init
index 19337af..0050d5a 100644
--- a/debian/nova-scheduler.init
+++ b/debian/nova-scheduler.init
@@ -3,6 +3,8 @@
 # Provides:          nova-scheduler
 # Required-Start:    $network $local_fs $remote_fs $syslog
 # Required-Stop:     $remote_fs
+# Should-Start:      postgresql
+# Should-Stop        postgresql
 # Default-Start:     2 3 4 5
 # Default-Stop:      0 1 6
 # Short-Description: Nova Scheduler
diff --git a/debian/nova-volume.init b/debian/nova-volume.init
index ab77119..d366675 100644
--- a/debian/nova-volume.init
+++ b/debian/nova-volume.init
@@ -3,6 +3,8 @@
 # Provides:          nova-volume
 # Required-Start:    $network $local_fs $remote_fs $syslog
 # Required-Stop:     $remote_fs
+# Should-Start:      postgresql
+# Should-Stop:       postgresql
 # Default-Start:     2 3 4 5
 # Default-Stop:      0 1 6
 # Short-Description: Nova Volume
-- 
1.7.2.5

-- 
Julien Cristau          <julien.cristau at logilab.fr>
Logilab		        http://www.logilab.fr/
Informatique scientifique & gestion de connaissances