Merge lp:~gandelman-a/nova/openstack-ubuntu-packagers into lp:~openstack-ubuntu-packagers/nova/ubuntu

Proposed by Adam Gandelman
Status: Needs review
Proposed branch: lp:~gandelman-a/nova/openstack-ubuntu-packagers
Merge into: lp:~openstack-ubuntu-packagers/nova/ubuntu
Diff against target: 67 lines (+17/-9)
4 files modified
debian/changelog (+6/-0)
debian/nova-common.postinst (+10/-7)
debian/nova-compute.upstart.in (+1/-1)
debian/nova.conf (+0/-1)
To merge this branch: bzr merge lp:~gandelman-a/nova/openstack-ubuntu-packagers
Reviewer Review Type Date Requested Status
OpenStack Ubuntu packagers Pending
Review via email: mp+77202@code.launchpad.net
To post a comment you must log in.
Revision history for this message
Adam Gandelman (gandelman-a) wrote :

first commit addresses Bug #856530

Revision history for this message
Scott Moser (smoser) wrote :

Unmerged revisions

193. By Adam Gandelman

debian/nova.conf, debian/nova-compute.upstart.in: Move reference of
nova-compute.conf from nova.conf to nova-compute's argv. (LP: #839796)

192. By Adam Gandelman

debian/nova-common.postinst: Create 'nova' group, add user to it.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'debian/changelog'
2--- debian/changelog 2011-09-21 21:50:52 +0000
3+++ debian/changelog 2011-09-27 16:59:49 +0000
4@@ -1,5 +1,11 @@
5 nova (2012.1~e1~20110909.1546-0ubuntu0) UNRELEASED; urgency=low
6
7+ [ Adam Gandelman ]
8+ * debian/nova-common.postinst: Create 'nova' group, add user to it
9+ (LP: #856530)
10+ * debian/nova.conf, debian/nova-compute.upstart.in: Move reference of
11+ nova-compute.conf from nova.conf to nova-compute's argv. (LP: #839796)
12+
13 [ Dan Prince ]
14 * Fix dnsmasq line in nova_sudoers.
15
16
17=== modified file 'debian/nova-common.postinst'
18--- debian/nova-common.postinst 2011-08-11 11:27:36 +0000
19+++ debian/nova-common.postinst 2011-09-27 16:59:49 +0000
20@@ -1,16 +1,19 @@
21 #!/bin/sh -e
22
23 if [ "$1" = "configure" ]; then
24+ if ! getent group nova > /dev/null 2>&1; then
25+ addgroup --system nova >/dev/null
26+ fi
27 if ! getent passwd nova > /dev/null 2>&1; then
28- adduser --system --home /var/lib/nova --no-create-home --shell /bin/bash nova
29+ adduser --system --home /var/lib/nova --ingroup nova --no-create-home --shell /bin/bash nova
30 fi
31- chown -R nova:root /var/lib/nova/ /var/log/nova/ /etc/nova/nova.conf
32+ chown -R nova:nova /var/lib/nova/ /var/log/nova/ /etc/nova/nova.conf
33 chmod 600 /etc/nova/nova.conf
34- chmod 0440 /etc/sudoers.d/nova_sudoers
35- if ! grep -q sql_connection /etc/nova/nova.conf
36- then
37- su -c 'nova-manage db sync' nova
38- fi
39+ chmod 0440 /etc/sudoers.d/nova_sudoers
40+ if ! grep -q sql_connection /etc/nova/nova.conf
41+ then
42+ su -c 'nova-manage db sync' nova
43+ fi
44 fi
45
46 #DEBHELPER#
47
48=== modified file 'debian/nova-compute.upstart.in'
49--- debian/nova-compute.upstart.in 2011-08-11 13:29:06 +0000
50+++ debian/nova-compute.upstart.in 2011-09-27 16:59:49 +0000
51@@ -17,4 +17,4 @@
52 modprobe nbd
53 end script
54
55-exec su -c "nova-compute --flagfile=/etc/nova/nova.conf" nova
56+exec su -c "nova-compute --flagfile=/etc/nova/nova.conf --flagfile=/etc/nova/nova-compute.conf" nova
57
58=== modified file 'debian/nova.conf'
59--- debian/nova.conf 2011-09-02 17:21:29 +0000
60+++ debian/nova.conf 2011-09-27 16:59:49 +0000
61@@ -3,6 +3,5 @@
62 --logdir=/var/log/nova
63 --state_path=/var/lib/nova
64 --lock_path=/var/lock/nova
65---flagfile=/etc/nova/nova-compute.conf
66 --use_deprecated_auth
67 --verbose

Subscribers

People subscribed via source and target branches