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
=== modified file 'debian/changelog'
--- debian/changelog 2011-09-21 21:50:52 +0000
+++ debian/changelog 2011-09-27 16:59:49 +0000
@@ -1,5 +1,11 @@
1nova (2012.1~e1~20110909.1546-0ubuntu0) UNRELEASED; urgency=low1nova (2012.1~e1~20110909.1546-0ubuntu0) UNRELEASED; urgency=low
22
3 [ Adam Gandelman ]
4 * debian/nova-common.postinst: Create 'nova' group, add user to it
5 (LP: #856530)
6 * debian/nova.conf, debian/nova-compute.upstart.in: Move reference of
7 nova-compute.conf from nova.conf to nova-compute's argv. (LP: #839796)
8
3 [ Dan Prince ]9 [ Dan Prince ]
4 * Fix dnsmasq line in nova_sudoers.10 * Fix dnsmasq line in nova_sudoers.
511
612
=== modified file 'debian/nova-common.postinst'
--- debian/nova-common.postinst 2011-08-11 11:27:36 +0000
+++ debian/nova-common.postinst 2011-09-27 16:59:49 +0000
@@ -1,16 +1,19 @@
1#!/bin/sh -e1#!/bin/sh -e
22
3if [ "$1" = "configure" ]; then3if [ "$1" = "configure" ]; then
4 if ! getent group nova > /dev/null 2>&1; then
5 addgroup --system nova >/dev/null
6 fi
4 if ! getent passwd nova > /dev/null 2>&1; then7 if ! getent passwd nova > /dev/null 2>&1; then
5 adduser --system --home /var/lib/nova --no-create-home --shell /bin/bash nova8 adduser --system --home /var/lib/nova --ingroup nova --no-create-home --shell /bin/bash nova
6 fi9 fi
7 chown -R nova:root /var/lib/nova/ /var/log/nova/ /etc/nova/nova.conf10 chown -R nova:nova /var/lib/nova/ /var/log/nova/ /etc/nova/nova.conf
8 chmod 600 /etc/nova/nova.conf11 chmod 600 /etc/nova/nova.conf
9 chmod 0440 /etc/sudoers.d/nova_sudoers12 chmod 0440 /etc/sudoers.d/nova_sudoers
10 if ! grep -q sql_connection /etc/nova/nova.conf13 if ! grep -q sql_connection /etc/nova/nova.conf
11 then14 then
12 su -c 'nova-manage db sync' nova15 su -c 'nova-manage db sync' nova
13 fi16 fi
14fi17fi
1518
16#DEBHELPER#19#DEBHELPER#
1720
=== modified file 'debian/nova-compute.upstart.in'
--- debian/nova-compute.upstart.in 2011-08-11 13:29:06 +0000
+++ debian/nova-compute.upstart.in 2011-09-27 16:59:49 +0000
@@ -17,4 +17,4 @@
17 modprobe nbd17 modprobe nbd
18end script18end script
1919
20exec su -c "nova-compute --flagfile=/etc/nova/nova.conf" nova20exec su -c "nova-compute --flagfile=/etc/nova/nova.conf --flagfile=/etc/nova/nova-compute.conf" nova
2121
=== modified file 'debian/nova.conf'
--- debian/nova.conf 2011-09-02 17:21:29 +0000
+++ debian/nova.conf 2011-09-27 16:59:49 +0000
@@ -3,6 +3,5 @@
3--logdir=/var/log/nova3--logdir=/var/log/nova
4--state_path=/var/lib/nova4--state_path=/var/lib/nova
5--lock_path=/var/lock/nova5--lock_path=/var/lock/nova
6--flagfile=/etc/nova/nova-compute.conf
7--use_deprecated_auth6--use_deprecated_auth
8--verbose7--verbose

Subscribers

People subscribed via source and target branches