Merge lp:~zulcss/glance/lp1307518 into lp:~ubuntu-server-dev/glance/icehouse

Proposed by Chuck Short
Status: Merged
Merged at revision: 293
Proposed branch: lp:~zulcss/glance/lp1307518
Merge into: lp:~ubuntu-server-dev/glance/icehouse
Diff against target: 43 lines (+9/-1)
3 files modified
debian/changelog (+7/-0)
debian/glance-api.install (+1/-0)
debian/glance-common.postinst (+1/-1)
To merge this branch: bzr merge lp:~zulcss/glance/lp1307518
Reviewer Review Type Date Requested Status
James Page Approve
Review via email: mp+215679@code.launchpad.net
To post a comment you must log in.
lp:~zulcss/glance/lp1307518 updated
288. By Chuck Short

debian/glance-common.postinst: Change permissions on
/var/lib/glance/images and /var/lib/glance/image-cache
(LP: #1302044)

289. By Chuck Short

conditional chown

Revision history for this message
James Page (james-page) wrote :

Generally - use a -d instead of -e.

41 + if [ -e /var/lib/glance/image-cache]; then
42 + /var/lib/glance/image-cache
43 + fi

needs a space after (image-cache) on 41

also needs to chown glance:glance -R /var/lib/glance/image-cache

review: Needs Fixing
lp:~zulcss/glance/lp1307518 updated
290. By Chuck Short

Update postiinst

Revision history for this message
James Page (james-page) wrote :

"-e" -> "-d"

review: Needs Fixing
lp:~zulcss/glance/lp1307518 updated
291. By Chuck Short

Revert commit

292. By Chuck Short

[ Chuck Short
* debian/glance-common.postinst: Set correct owner/group for /var/lib/glance
and subdirs only on local filesystems (LP: #1302044).

293. By Chuck Short

Fix up changelog

Revision history for this message
James Page (james-page) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'debian/changelog'
--- debian/changelog 2014-04-12 12:45:23 +0000
+++ debian/changelog 2014-04-14 14:56:54 +0000
@@ -1,7 +1,14 @@
1glance (1:2014.1~rc2-0ubuntu1) trusty; urgency=medium1glance (1:2014.1~rc2-0ubuntu1) trusty; urgency=medium
22
3 [ Chuck Short ]
3 * New upstream release candidate (LP: #1299055).4 * New upstream release candidate (LP: #1299055).
4 * debian/patches/sql_conn.patch: Refreshed.5 * debian/patches/sql_conn.patch: Refreshed.
6 * debian/glance-api.install: Install missing schema.json file.
7 (LP: #1307518)
8
9 [ Thomas Bechtold ]
10 * debian/glance-common.postinst: Set correct owner/group for /var/lib/glance
11 and subdirs only on local filesystems (LP: #1302044).
512
6 -- Chuck Short <zulcss@ubuntu.com> Sat, 12 Apr 2014 08:45:20 -040013 -- Chuck Short <zulcss@ubuntu.com> Sat, 12 Apr 2014 08:45:20 -0400
714
815
=== modified file 'debian/glance-api.install'
--- debian/glance-api.install 2012-11-26 13:36:21 +0000
+++ debian/glance-api.install 2014-04-14 14:56:54 +0000
@@ -2,6 +2,7 @@
2etc/glance-api.conf etc/glance2etc/glance-api.conf etc/glance
3etc/glance-cache.conf etc/glance3etc/glance-cache.conf etc/glance
4etc/glance-scrubber.conf etc/glance4etc/glance-scrubber.conf etc/glance
5etc/schema-image.json etc/glance
5etc/policy.json etc/glance6etc/policy.json etc/glance
6usr/bin/glance-api7usr/bin/glance-api
7usr/bin/glance-cache-cleaner8usr/bin/glance-cache-cleaner
89
=== modified file 'debian/glance-common.postinst'
--- debian/glance-common.postinst 2014-03-17 13:21:04 +0000
+++ debian/glance-common.postinst 2014-04-14 14:56:54 +0000
@@ -16,7 +16,7 @@
16 chmod 0750 /var/log/glance16 chmod 0750 /var/log/glance
17 chown glance:glance -R /etc/glance17 chown glance:glance -R /etc/glance
18 chmod 0750 /etc/glance18 chmod 0750 /etc/glance
19 chown glance:glance /var/lib/glance19 find /var/lib/glance -xdev -type d -exec chown glance:glance {} \;
20fi20fi
2121
22#DEBHELPER#22#DEBHELPER#

Subscribers

People subscribed via source and target branches