Merge lp:~mbp/keystone/930444-db-sync into lp:ubuntu/precise/keystone

Proposed by Martin Pool
Status: Merged
Merge reported by: Stefano Rivera
Merged at revision: not available
Proposed branch: lp:~mbp/keystone/930444-db-sync
Merge into: lp:ubuntu/precise/keystone
Diff against target: 26 lines (+7/-1)
2 files modified
debian/changelog (+6/-0)
debian/keystone.postinst (+1/-1)
To merge this branch: bzr merge lp:~mbp/keystone/930444-db-sync
Reviewer Review Type Date Requested Status
Ubuntu branches Pending
Review via email: mp+92698@code.launchpad.net

Description of the change

Per bug 930444, it looks like the postinst script is using the wrong (old?) syntax to keystone-manage to update the database schema.

To post a comment you must log in.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'debian/changelog'
--- debian/changelog 2012-02-10 10:39:48 +0000
+++ debian/changelog 2012-02-13 01:04:19 +0000
@@ -1,3 +1,9 @@
1keystone (2012.1~e4~20120203.1574-0ubuntu3) precise; urgency=low
2
3 * Update "db sync" to "sync_database" in postinst. (LP: #930444)
4
5 -- Martin Pool <mbp@canonical.com> Mon, 13 Feb 2012 11:51:01 +1100
6
1keystone (2012.1~e4~20120203.1574-0ubuntu2) precise; urgency=low7keystone (2012.1~e4~20120203.1574-0ubuntu2) precise; urgency=low
28
3 [Chuck Short]9 [Chuck Short]
410
=== modified file 'debian/keystone.postinst'
--- debian/keystone.postinst 2012-02-10 10:39:48 +0000
+++ debian/keystone.postinst 2012-02-13 01:04:19 +0000
@@ -56,7 +56,7 @@
5656
57 if [ "$dbc_upgrade" = "true" ]57 if [ "$dbc_upgrade" = "true" ]
58 then58 then
59 su -s /bin/sh -c 'exec keystone-manage db sync' keystone59 su -s /bin/sh -c 'exec keystone-manage sync_database' keystone
60 fi60 fi
61 fi61 fi
62 chown keystone:keystone -R /var/lib/keystone/ /var/log/keystone/ /etc/keystone/62 chown keystone:keystone -R /var/lib/keystone/ /var/log/keystone/ /etc/keystone/

Subscribers

People subscribed via source and target branches