]> review.fuel-infra Code Review - openstack-build/cinder-build.git/commitdiff
* Now using keystone_authtoken in cinder.conf instead of api-paste.ini:
authorThomas Goirand <thomas@goirand.fr>
Wed, 30 Apr 2014 16:03:15 +0000 (00:03 +0800)
committerThomas Goirand <thomas@goirand.fr>
Wed, 30 Apr 2014 16:03:15 +0000 (00:03 +0800)
    - Fixed config and postinst scripts of cinder-common to use that.
    - Removed patch for api-paste.ini.

debian/changelog
debian/cinder-common.config.in
debian/cinder-common.postinst.in
debian/patches/re-add-filter-authtoken.patch [deleted file]
debian/patches/series

index 62de46ab01e09cf8104b9c69a50f5730a6c16d9b..ec56fba9346c666433934ee7faed6fd3af97a95c 100644 (file)
@@ -1,3 +1,11 @@
+cinder (2014.1-3) unstable; urgency=medium
+
+  * Now using keystone_authtoken in cinder.conf instead of api-paste.ini:
+    - Fixed config and postinst scripts of cinder-common to use that.
+    - Removed patch for api-paste.ini.
+
+ -- Thomas Goirand <zigo@debian.org>  Thu, 01 May 2014 00:01:50 +0800
+
 cinder (2014.1-2) unstable; urgency=medium
 
   * Applied patch from Mike Perez <thingee@gmail.com> to be able to create
index 80a9a3fa6e0275b1af232a0612b3ee2d291db341..ec9fc6bccc9785153b0b490dafb118c796a2a38e 100644 (file)
@@ -4,14 +4,13 @@ set -e
 
 . /usr/share/debconf/confmodule
 CINDER_CONF=/etc/cinder/cinder.conf
-CINDER_API=/etc/cinder/api-paste.ini
 
 #PKGOS-INCLUDE#
 
 pkgos_var_user_group cinder
 pkgos_dbc_read_conf -pkg cinder-common ${CINDER_CONF} DEFAULT sql_connection cinder $@
 pkgos_rabbit_read_conf ${CINDER_CONF} DEFAULT cinder
-pkgos_read_admin_creds ${CINDER_API} filter:authtoken cinder
+pkgos_read_admin_creds ${CINDER_CONF} keystone_authtoken cinder
 
 pkgos_inifile get ${CINDER_CONF} DEFAULT volume_group
 if [ -n "${RET}" ] && [ ! "${RET}" = "NOT_FOUND" ] ; then
index d7e3b90467273c9fc8d651976fdaefa4ef638930..d949ac9d1d9f7eaba7f89193df290f0273f114e0 100644 (file)
@@ -3,7 +3,6 @@
 set -e
 
 CINDER_CONF=/etc/cinder/cinder.conf
-CINDER_API=/etc/cinder/api-paste.ini
 
 #PKGOS-INCLUDE#
 
@@ -24,7 +23,7 @@ if [ "$1" = "configure" ] || [ "$1" = "reconfigure" ] ; then
        pkgos_write_new_conf cinder logging.conf
        pkgos_dbc_postinst ${CINDER_CONF} DEFAULT sql_connection cinder $@
        pkgos_rabbit_write_conf ${CINDER_CONF} DEFAULT cinder
-       pkgos_write_admin_creds ${CINDER_API} filter:authtoken cinder
+       pkgos_write_admin_creds ${CINDER_CONF} keystone_authtoken cinder
        db_get cinder/volume_group
        if [ -n "${RET}" ] ; then
                pkgos_inifile set ${CINDER_CONF} DEFAULT volume_group ${RET}
diff --git a/debian/patches/re-add-filter-authtoken.patch b/debian/patches/re-add-filter-authtoken.patch
deleted file mode 100644 (file)
index 7d7bc92..0000000
+++ /dev/null
@@ -1,39 +0,0 @@
-Description: <short summary of the patch>
- TODO: Put a short summary on the line above and replace this paragraph
- with a longer explanation of this change. Complete the meta-information
- with other relevant fields (see below for details). To make it easier, the
- information below has been extracted from the changelog. Adjust it or drop
- it.
- .
- cinder (2014.1~rc1-1) experimental; urgency=low
- .
-   * New upstream release.
-   * Removed now useless fix-sqlalchemy-version patch.
-   * Fixed new (build-)dependencies for this release.
-Author: Thomas Goirand <zigo@debian.org>
-
----
-The information above should follow the Patch Tagging Guidelines, please
-checkout http://dep.debian.net/deps/dep3/ to learn about the format. Here
-are templates for supplementary fields that you might want to add:
-
-Origin: <vendor|upstream|other>, <url of original patch>
-Bug: <url in upstream bugtracker>
-Bug-Debian: http://bugs.debian.org/<bugnumber>
-Bug-Ubuntu: https://launchpad.net/bugs/<bugnumber>
-Forwarded: <no|not-needed|url proving that it has been forwarded>
-Reviewed-By: <name and email of someone who approved the patch>
-Last-Update: <YYYY-MM-DD>
-
---- cinder-2014.1~rc1.orig/etc/cinder/api-paste.ini
-+++ cinder-2014.1~rc1/etc/cinder/api-paste.ini
-@@ -53,3 +53,9 @@ paste.filter_factory = cinder.api.middle
- [filter:authtoken]
- paste.filter_factory = keystoneclient.middleware.auth_token:filter_factory
-+auth_host = 127.0.0.1
-+auth_port = 35357
-+auth_protocol = http
-+admin_tenant_name = %SERVICE_TENANT_NAME%
-+admin_user = %SERVICE_USER%
-+admin_password = %SERVICE_PASSWORD%
index 69f34a7a46eff9aaa12b331f041ec83d5f5b507a..5d66926da7d9200ce58d42630947f03a05983217 100644 (file)
@@ -1,3 +1,2 @@
 install-missing-files.patch
-re-add-filter-authtoken.patch
 fallback-to-None-on-missing-Glance-image-attrs.patch