Attachment 691642 Details for Bug 887811 – Initial version of opensatck-glance packaging changes for review

[patch] Initial version of opensatck-glance packaging changes for review

0001-Move-boilerplate-glance-api-registry-config-to-usr-s.patch (text/plain), 8.93 KB, created by Eoghan Glynn on 2013-02-01 17:40:49 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Eoghan Glynn

Created: 2013-02-01 17:40:49 UTC

Size: 8.93 KB

patch

obsolete

>From df53ea0187c58e1fa188fc37d737058679c2c0e8 Mon Sep 17 00:00:00 2001
>From: Eoghan Glynn <eglynn@redhat.com>
>Date: Fri, 1 Feb 2013 12:12:04 -0500
>Subject: [PATCH] Move boilerplate glance-{api|registry} config to /usr/share.
>
>Instead of placing the boilerplate glance-{api|registry}.conf
>and glance-{api|registry}-paste.ini under /etc/glance, we now
>create "-dist" versions of these files in /usr/share.
>
>Then the old default config location under /etc/glance is reserved
>for (initially empty) user-overridden config. The %config(noreplace)
>macro is set on these files (so that user edits are not lost
>when the package is removed), but not on the distribution config
>(which is not expected to be modified post-install).
>
>This is a work-in-progress patch, intended to elicit feedback on
>the approach taken. A subsequent version will move the remaining
>boilerplate config to /usr/share.
>---
> glance-api.conf.user              |    3 +++
> glance-registry.conf.user         |    3 +++
> openstack-glance-api.init         |    8 +++++---
> openstack-glance-api.upstart      |    2 +-
> openstack-glance-registry.init    |    9 ++++++---
> openstack-glance-registry.upstart |    2 +-
> openstack-glance.spec             |   22 +++++++++++++++-------
> 7 files changed, 34 insertions(+), 15 deletions(-)
> create mode 100644 glance-api.conf.user
> create mode 100644 glance-registry.conf.user
>
>diff --git a/glance-api.conf.user b/glance-api.conf.user
>new file mode 100644
>index 0000000..85d26c4
>--- /dev/null
>+++ b/glance-api.conf.user
>@@ -0,0 +1,3 @@
>+[DEFAULT]
>+
>+# User-provided configuration overrides for glance-api service.
>diff --git a/glance-registry.conf.user b/glance-registry.conf.user
>new file mode 100644
>index 0000000..4687072
>--- /dev/null
>+++ b/glance-registry.conf.user
>@@ -0,0 +1,3 @@
>+[DEFAULT]
>+
>+# User-provided configuration overrides for glance-registry service.
>diff --git a/openstack-glance-api.init b/openstack-glance-api.init
>index 549b63d..d95ab2d 100644
>--- a/openstack-glance-api.init
>+++ b/openstack-glance-api.init
>@@ -19,7 +19,8 @@
> suffix=api
> prog=openstack-glance-$suffix
> exec="/usr/bin/glance-$suffix"
>-config="/etc/glance/glance-$suffix.conf"
>+dist_config="/usr/share/glance/glance-$suffix-dist.conf"
>+user_config="/etc/glance/glance-$suffix.conf"
> pidfile="/var/run/glance/glance-$suffix.pid"
> 
> [ -e /etc/sysconfig/$prog ] && . /etc/sysconfig/$prog
>@@ -28,9 +29,10 @@ lockfile=/var/lock/subsys/$prog
> 
> start() {
>     [ -x $exec ] || exit 5
>-    [ -f $config ] || exit 6
>+    [ -f $dist_config ] || exit 6
>+    [ -f $user_config ] || exit 7
>     echo -n $"Starting $prog: "
>-    daemon --user glance --pidfile $pidfile "$exec --config-file $config --debug --verbose &>/dev/null & echo \$! > $pidfile"
>+    daemon --user glance --pidfile $pidfile "$exec --config-file $dist_config --config-file $user_config --debug --verbose &>/dev/null & echo \$! > $pidfile"
>     retval=$?
>     echo
>     [ $retval -eq 0 ] && touch $lockfile
>diff --git a/openstack-glance-api.upstart b/openstack-glance-api.upstart
>index 1539397..d6fdd4f 100644
>--- a/openstack-glance-api.upstart
>+++ b/openstack-glance-api.upstart
>@@ -5,4 +5,4 @@ stop on runlevel [S016]
> 
> respawn
> 
>-exec su -s /bin/sh -c "exec /usr/bin/glance-api --config-file /etc/glance/glance-api.conf" glance
>+exec su -s /bin/sh -c "exec /usr/bin/glance-api --config-file /usr/share/lib/glance/glance-api-dist.conf --config-file /etc/glance/glance-api.conf" glance
>diff --git a/openstack-glance-registry.init b/openstack-glance-registry.init
>index 48bab47..6f6c200 100644
>--- a/openstack-glance-registry.init
>+++ b/openstack-glance-registry.init
>@@ -19,7 +19,9 @@
> suffix=registry
> prog=openstack-glance-$suffix
> exec="/usr/bin/glance-$suffix"
>-config="/etc/glance/glance-$suffix.conf"
>+dist_config="/usr/share/glance/glance-$suffix-dist.conf"
>+user_config="/etc/glance/glance-$suffix.conf"
>+
> pidfile="/var/run/glance/glance-$suffix.pid"
> 
> [ -e /etc/sysconfig/$prog ] && . /etc/sysconfig/$prog
>@@ -28,9 +30,10 @@ lockfile=/var/lock/subsys/$prog
> 
> start() {
>     [ -x $exec ] || exit 5
>-    [ -f $config ] || exit 6
>+    [ -f $dist_config ] || exit 6
>+    [ -f $user_config ] || exit 7
>     echo -n $"Starting $prog: "
>-    daemon --user glance --pidfile $pidfile "$exec --config-file $config --debug --verbose &>/dev/null & echo \$! > $pidfile"
>+    daemon --user glance --pidfile $pidfile "$exec --config-file $dist_config --config-file $user_config --debug --verbose &>/dev/null & echo \$! > $pidfile"
>     retval=$?
>     echo
>     [ $retval -eq 0 ] && touch $lockfile
>diff --git a/openstack-glance-registry.upstart b/openstack-glance-registry.upstart
>index eccae41..b0bdecb 100644
>--- a/openstack-glance-registry.upstart
>+++ b/openstack-glance-registry.upstart
>@@ -5,4 +5,4 @@ stop on runlevel [S016]
> 
> respawn
> 
>-exec su -s /bin/sh -c "exec /usr/bin/glance-registry --config-file /etc/glance/glance-registry.conf" glance
>+exec su -s /bin/sh -c "exec /usr/bin/glance-registry --config-file /usr/share/glance/glance-registry-dist.conf --config-file /etc/glance/glance-registry.conf" glance
>diff --git a/openstack-glance.spec b/openstack-glance.spec
>index 40ba0f1..a578b60 100644
>--- a/openstack-glance.spec
>+++ b/openstack-glance.spec
>@@ -3,7 +3,7 @@
> #
> Name:             openstack-glance
> Version:          2012.2.1
>-Release:          5%{?dist}
>+Release:          6%{?dist}
> Summary:          OpenStack Image Service
> 
> Group:            Applications/System
>@@ -15,6 +15,8 @@ Source100:        openstack-glance-api.upstart
> Source2:          openstack-glance-registry.init
> Source200:        openstack-glance-registry.upstart
> Source3:          openstack-glance.logrotate
>+Source4:          glance-api.conf.user
>+Source5:          glance-registry.conf.user
> 
> #
> # patches_base=2012.2.1_rhos
>@@ -163,6 +165,8 @@ openstack-config --set etc/glance-registry.conf keystone_authtoken admin_passwor
> openstack-config --set etc/glance-registry.conf keystone_authtoken auth_host 127.0.0.1
> openstack-config --set etc/glance-registry.conf keystone_authtoken auth_port 35357
> openstack-config --set etc/glance-registry.conf keystone_authtoken auth_protocol http
>+openstack-config --set etc/glance-api.conf paste_deploy config_file %{_datadir}/glance/glance-api-dist-paste.ini
>+openstack-config --set etc/glance-registry.conf paste_deploy config_file %{_datadir}/glance/glance-registry-dist-paste.ini
> 
> %{__python} setup.py build
> 
>@@ -199,10 +203,12 @@ install -d -m 755 %{buildroot}%{_datadir}/glance
> install -d -m 755 %{buildroot}%{_sharedstatedir}/glance/images
> 
> # Config file
>-install -p -D -m 640 etc/glance-api.conf %{buildroot}%{_sysconfdir}/glance/glance-api.conf
>-install -p -D -m 640 etc/glance-api-paste.ini %{buildroot}%{_sysconfdir}/glance/glance-api-paste.ini
>-install -p -D -m 640 etc/glance-registry.conf %{buildroot}%{_sysconfdir}/glance/glance-registry.conf
>-install -p -D -m 640 etc/glance-registry-paste.ini %{buildroot}%{_sysconfdir}/glance/glance-registry-paste.ini
>+install -p -D -m 640 etc/glance-api.conf %{buildroot}%{_datadir}/glance/glance-api-dist.conf
>+install -p -D -m 640 etc/glance-api-paste.ini %{buildroot}%{_datadir}/glance/glance-api-dist-paste.ini
>+install -p -D -m 640 %{SOURCE4} %{buildroot}%{_sysconfdir}/glance/glance-api.conf
>+install -p -D -m 640 etc/glance-registry.conf %{buildroot}%{_datadir}/glance/glance-registry-dist.conf
>+install -p -D -m 640 etc/glance-registry-paste.ini %{buildroot}%{_datadir}/glance/glance-registry-dist-paste.ini
>+install -p -D -m 640 %{SOURCE5} %{buildroot}%{_sysconfdir}/glance/glance-registry.conf
> install -p -D -m 640 etc/glance-cache.conf %{buildroot}%{_sysconfdir}/glance/glance-cache.conf
> install -p -D -m 640 etc/glance-scrubber.conf %{buildroot}%{_sysconfdir}/glance/glance-scrubber.conf
> install -p -D -m 640 etc/policy.json %{buildroot}%{_sysconfdir}/glance/policy.json
>@@ -264,14 +270,16 @@ fi
> %{_mandir}/man1/glance*.1.gz
> %dir %{_sysconfdir}/glance
> %config(noreplace) %attr(-, root, glance) %{_sysconfdir}/glance/glance-api.conf
>-%config(noreplace) %attr(-, root, glance) %{_sysconfdir}/glance/glance-api-paste.ini
> %config(noreplace) %attr(-, root, glance) %{_sysconfdir}/glance/glance-registry.conf
>-%config(noreplace) %attr(-, root, glance) %{_sysconfdir}/glance/glance-registry-paste.ini
> %config(noreplace) %attr(-, root, glance) %{_sysconfdir}/glance/glance-cache.conf
> %config(noreplace) %attr(-, root, glance) %{_sysconfdir}/glance/glance-scrubber.conf
> %config(noreplace) %attr(-, root, glance) %{_sysconfdir}/glance/policy.json
> %config(noreplace) %attr(-, root, glance) %{_sysconfdir}/glance/schema-image.json
> %config(noreplace) %attr(-, root, glance) %{_sysconfdir}/logrotate.d/openstack-glance
>+%attr(-, root, glance) %{_datadir}/glance/glance-api-dist.conf
>+%attr(-, root, glance) %{_datadir}/glance/glance-api-dist-paste.ini
>+%attr(-, root, glance) %{_datadir}/glance/glance-registry-dist.conf
>+%attr(-, root, glance) %{_datadir}/glance/glance-registry-dist-paste.ini
> %dir %attr(0755, glance, glance) %{_sharedstatedir}/glance
> %dir %attr(0755, glance, glance) %{_localstatedir}/log/glance
> %dir %attr(0755, glance, glance) %{_localstatedir}/run/glance
>-- 
>1.7.1
>

Flags:

pbrady: review+

Actions: View | Diff

Attachments on bug 887811: 691642 | 692949 | 693472 | 695133 | 695136