Attachment 851141 Details for Bug 1045034 – Patch for havana branch

[patch] Patch for havana branch

0001-check-whether-db-needs-to-be-upgraded-el6-havana.patch (text/plain), 6.83 KB, created by Jakub Libosvar on 2014-01-16 15:12:51 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Jakub Libosvar

Created: 2014-01-16 15:12:51 UTC

Size: 6.83 KB

patch

obsolete

>From 9a50907e42f1ae804bb7bc2265196516d53d6ad5 Mon Sep 17 00:00:00 2001
>From: Jakub Libosvar <jlibosva@redhat.com>
>Date: Wed, 11 Dec 2013 11:26:30 +0100
>Subject: [PATCH] check whether db needs to be upgraded
>
>If neutron is upgraded from previous release (Grizzly) there
>needs to be run db upgrade before services are started. Starting
>services without upgrading db may lead into inconsistent behavior.
>Neutron service will not start if db is still from previous release
>while packages are from current release.
>
>Resolves: rhbz#1031801
>---
> neutron-db-check            | 41 +++++++++++++++++++++++++++++++++++++++++
> neutron-server.init         |  5 +++++
> neutron-server.upstart      |  6 ++++++
> openstack-neutron.spec      | 25 +++++++++++++++++++++----
> openstack-neutron.sysconfig |  1 +
> 5 files changed, 74 insertions(+), 4 deletions(-)
> create mode 100755 neutron-db-check
> create mode 100644 openstack-neutron.sysconfig
>
>diff --git a/neutron-db-check b/neutron-db-check
>new file mode 100755
>index 0000000..19b8707
>--- /dev/null
>+++ b/neutron-db-check
>@@ -0,0 +1,41 @@
>+#!/bin/sh
>+#
>+# This script checks to make sure the database used by Neutron is at the
>+# correct version.  If the database is not at $NEUTRON_EXPECTED_DB_VERSION, as
>+# set in /etc/sysconfig/openstack-neutron, log an error and exit with an error
>+# code.
>+#
>+# This script is meant to be called from an init script or via ExecPreStart in
>+# a systemd unit.
>+
>+RETCODE=0
>+
>+[ -f /etc/sysconfig/openstack-neutron ] && . /etc/sysconfig/openstack-neutron
>+
>+[ "$NEUTRON_EXPECTED_DB_VERSION" ] || exit 0
>+
>+neutron_db_version=$(
>+        neutron-db-manage \
>+            --config-file /etc/neutron/neutron.conf \
>+            --config-file /etc/neutron/plugin.ini current 2>/dev/null |
>+        awk '{print $NF}')
>+
>+if [ "x$neutron_db_version" = "x" ]; then
>+    logger -s -p daemon.error -t neutron-db-check \
>+        "ERROR: unable to find version information for the neutron database."\
>+        "Check your logs for additional information."
>+    RETCODE=1
>+elif [ "x$neutron_db_version" = "xNone" ]; then
>+    logger -s -p daemon.error -t neutron-db-check \
>+        "ERROR: neutron database does not have version information."\
>+        "You must run the \"neutron-db-manage stamp\" command before"\
>+        "starting neutron services."
>+    RETCODE=1
>+elif [ "x$neutron_db_version" != "x$NEUTRON_EXPECTED_DB_VERSION" ]; then
>+    logger -s -p daemon.error -t neutron-db-check \
>+        "ERROR: you must upgrade the neutron database before starting"\
>+        "neutron services."
>+    RETCODE=1
>+fi
>+
>+exit $RETCODE
>diff --git a/neutron-server.init b/neutron-server.init
>index d5ccc9f..7b0f3e6 100644
>--- a/neutron-server.init
>+++ b/neutron-server.init
>@@ -11,6 +11,7 @@
> 
> prog=neutron
> exec="/usr/bin/$prog-server"
>+dbcheck="/usr/bin/$prog-db-check"
> configs=(
>     "/usr/share/$prog/$prog-dist.conf" \
>     "/etc/$prog/$prog.conf" \
>@@ -25,6 +26,10 @@ lockfile=/var/lock/subsys/$prog-server
> 
> start() {
>     [ -x $exec ] || exit 5
>+
>+    # Make sure database is at the currect version.
>+    [ -x $dbcheck ] && $dbcheck || exit 6
>+
>     for config in ${configs[@]}; do
>         [ -f $config ] || exit 6
>     done
>diff --git a/neutron-server.upstart b/neutron-server.upstart
>index ec14e41..4cb1b79 100644
>--- a/neutron-server.upstart
>+++ b/neutron-server.upstart
>@@ -1,5 +1,11 @@
> description "OpenStack Neutron Server"
> 
>+pre-start
>+    if ! /usr/bin/neutron-db-check; then
>+        stop; exit 0
>+    fi
>+end script
>+
> start on stopped rc RUNLEVEL=[2345]
> stop on runlevel [S016]
> 
>diff --git a/openstack-neutron.spec b/openstack-neutron.spec
>index 8116cf7..1b374d8 100644
>--- a/openstack-neutron.spec
>+++ b/openstack-neutron.spec
>@@ -2,7 +2,7 @@
> 
> Name:		openstack-neutron
> Version:	2013.2.1
>-Release:	3%{?dist}
>+Release:	4%{?dist}
> Provides:	openstack-quantum = %{version}-%{release}
> Obsoletes:	openstack-quantum < 2013.2-0.3.b3
> 
>@@ -47,6 +47,9 @@ Source41:	neutron-vpn-agent.upstart
> Source32:	neutron-metering-agent.init
> Source42:	neutron-metering-agent.upstart
> 
>+Source50:   neutron-db-check
>+Source51:   openstack-neutron.sysconfig
>+
> Source90:	neutron-dist.conf
> #
> # patches_base=2013.2.1+1
>@@ -66,7 +69,7 @@ BuildRequires:	python-sqlalchemy0.7
> BuildRequires:	python-webob1.2
> BuildRequires:	python-paste-deploy1.5
> BuildRequires:	python-routes1.12
>-BuildRequires:  python-jinja2-26
>+BuildRequires:	python-jinja2-26
> BuildRequires:	dos2unix
> BuildRequires:	python-pbr
> BuildRequires:	python-d2to1
>@@ -522,6 +525,12 @@ install -p -m 644 %{SOURCE42} %{buildroot}%{_datadir}/neutron/
> # Install dist conf
> install -p -D -m 640 %{SOURCE90} %{buildroot}%{_datadir}/neutron/neutron-dist.conf
> 
>+# Install neutron-db-check
>+install -m 755 %{SOURCE50} %{buildroot}%{_bindir}/neutron-db-check
>+install -d -m 755 %{buildroot}%{_sysconfdir}/sysconfig
>+install -m 644 %{SOURCE51} %{buildroot}%{_sysconfdir}/sysconfig/openstack-neutron
>+
>+
> # Install version info file
> cat > %{buildroot}%{_sysconfdir}/neutron/release <<EOF
> [Neutron]
>@@ -600,8 +609,10 @@ if [ -e %{_localstatedir}/lib/rpm-state/UPGRADE_FROM_QUANTUM ];then
>         ln -s ${plugin_ini//quantum/neutron} %{_sysconfdir}/neutron/plugin.ini
>     fi
> 
>-    # Stamp the existing db as grizzly to avoid neutron-server breaking db migration
>-    neutron-db-manage --config-file %{_sysconfdir}/neutron/neutron.conf --config-file %{_sysconfdir}/neutron/plugin.ini stamp grizzly || :
>+    # Stamp the existing db as grizzly to avoid neutron-server breaking db
>+    # migration after upgrade
>+    neutron-db-manage --config-file %{_sysconfdir}/neutron/neutron.conf \
>+        --config-file %{_sysconfdir}/neutron/plugin.ini stamp grizzly || :
> 
>     # Restore the enablement of the various neutron services
>     source %{_localstatedir}/lib/rpm-state/UPGRADE_FROM_QUANTUM
>@@ -784,6 +795,8 @@ fi
> %{_bindir}/neutron-server-setup
> %{_bindir}/neutron-usage-audit
> 
>+%{_bindir}/neutron-db-check
>+
> %{_initrddir}/neutron-server
> %{_initrddir}/neutron-dhcp-agent
> %{_initrddir}/neutron-l3-agent
>@@ -808,6 +821,7 @@ fi
> %config(noreplace) %attr(0640, root, neutron) %{_sysconfdir}/neutron/policy.json
> %config(noreplace) %attr(0640, root, neutron) %{_sysconfdir}/neutron/neutron.conf
> %config(noreplace) %{_sysconfdir}/neutron/rootwrap.conf
>+%config %{_sysconfdir}/sysconfig/openstack-neutron
> %dir %{_sysconfdir}/neutron/plugins
> %config(noreplace) %{_sysconfdir}/logrotate.d/*
> %config(noreplace) %{_sysconfdir}/sudoers.d/neutron
>@@ -1005,6 +1019,9 @@ fi
> 
> 
> %changelog
>+* Thu Jan 16 2014 Jakub Libosvar <jlibosva@redhat.com> - 2013.2.1-4
>+- Check whether db needs to be upgraded (rhbz#1031801)
>+
> * Tue Jan 07 2014 Terry Wilson <twilson@redhat.com> - 2013-2.1-3
> - Add python-psutil requirement for openvswitch agent, bz#1049235
> 
>diff --git a/openstack-neutron.sysconfig b/openstack-neutron.sysconfig
>new file mode 100644
>index 0000000..8394337
>--- /dev/null
>+++ b/openstack-neutron.sysconfig
>@@ -0,0 +1 @@
>+NEUTRON_EXPECTED_DB_VERSION=havana
>-- 
>1.8.3.1
>

Actions: View | Diff

Attachments on bug 1045034: 851141 | 851142 | 851143