Merge lp:~gandelman-a/ubuntu/precise/nova/lp940780 into lp:~ubuntu-server-dev/nova/essex

Proposed by Adam Gandelman
Status: Merged
Merge reported by: Chuck Short
Merged at revision: not available
Proposed branch: lp:~gandelman-a/ubuntu/precise/nova/lp940780
Merge into: lp:~ubuntu-server-dev/nova/essex
Diff against target: 169 lines (+48/-28)
3 files modified
debian/changelog (+7/-0)
debian/nova-volume.upstart.in (+3/-0)
debian/patches/libvirt-use-console-pipe.patch (+38/-28)
To merge this branch: bzr merge lp:~gandelman-a/ubuntu/precise/nova/lp940780
Reviewer Review Type Date Requested Status
Ubuntu Server Developers Pending
Review via email: mp+94644@code.launchpad.net

Description of the change

Fixes the nova-volume upstart job, refreshes the libvirt-console patch so it applies cleanly to recent revisions of the unit tests.

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
1=== modified file 'debian/changelog'
2--- debian/changelog 2012-02-24 15:08:20 +0000
3+++ debian/changelog 2012-02-25 06:26:17 +0000
4@@ -1,3 +1,10 @@
5+nova (2012.1~e4~20120224.12913-0ubuntu2) UNRELEASED; urgency=low
6+
7+ * debain/nova-volume.upstart.in: Ensure lock directory is created
8+ (LP: #940780)
9+
10+ -- Adam Gandelman <adamg@canonical.com> Fri, 24 Feb 2012 22:14:24 -0800
11+
12 nova (2012.1~e4~20120224.12913-0ubuntu1) precise; urgency=low
13
14 [ Monty Taylor ]
15
16=== modified file 'debian/nova-volume.upstart.in'
17--- debian/nova-volume.upstart.in 2012-01-31 18:29:02 +0000
18+++ debian/nova-volume.upstart.in 2012-02-25 06:26:17 +0000
19@@ -10,6 +10,9 @@
20 pre-start script
21 mkdir -p /var/run/nova
22 chown nova:root /var/run/nova/
23+
24+ mkdir -p /var/lock/nova
25+ chown nova:root /var/lock/nova/
26 end script
27
28 exec su -s /bin/sh -c "exec nova-volume --flagfile=/etc/nova/nova.conf" nova
29
30=== modified file 'debian/patches/libvirt-use-console-pipe.patch'
31--- debian/patches/libvirt-use-console-pipe.patch 2012-02-23 01:59:47 +0000
32+++ debian/patches/libvirt-use-console-pipe.patch 2012-02-25 06:26:17 +0000
33@@ -1,17 +1,19 @@
34-diff -Naur nova.orig/Authors nova/Authors
35---- nova.orig/Authors 2012-02-22 20:57:07.968044568 -0500
36-+++ nova/Authors 2012-02-22 20:58:15.360044601 -0500
37-@@ -144,6 +144,7 @@
38+Index: nova/Authors
39+===================================================================
40+--- nova.orig/Authors 2012-02-24 19:29:28.000000000 -0800
41++++ nova/Authors 2012-02-24 19:30:40.815750120 -0800
42+@@ -146,6 +146,7 @@
43 Rick Clark <rick@openstack.org>
44 Rick Harris <rconradharris@gmail.com>
45 Rob Kost <kost@isi.edu>
46 +Robie Basak <robie.basak@canonical.com>
47 Russell Bryant <rbryant@redhat.com>
48+ Russell Sim <russell.sim@gmail.com>
49 Ryan Lane <rlane@wikimedia.org>
50- Ryan Lucio <rlucio@internap.com>
51-diff -Naur nova.orig/nova/tests/test_libvirt.py nova/nova/tests/test_libvirt.py
52---- nova.orig/nova/tests/test_libvirt.py 2012-02-22 20:57:08.056044568 -0500
53-+++ nova/nova/tests/test_libvirt.py 2012-02-22 20:58:15.360044601 -0500
54+Index: nova/nova/tests/test_libvirt.py
55+===================================================================
56+--- nova.orig/nova/tests/test_libvirt.py 2012-02-24 19:29:28.000000000 -0800
57++++ nova/nova/tests/test_libvirt.py 2012-02-24 19:30:40.815750120 -0800
58 @@ -872,7 +872,7 @@
59 (lambda t: _ipv4_like(t.findall(parameter)[1].get('value'),
60 '192.168.*.1'), True),
61@@ -75,17 +77,19 @@
62 + os.unlink(self.ringbuffer_path)
63 + os.unlink(self.fifo_path)
64 + os.rmdir(self.directory_path)
65-diff -Naur nova.orig/nova/tests/test_utils.py nova/nova/tests/test_utils.py
66---- nova.orig/nova/tests/test_utils.py 2012-02-22 20:57:08.060044569 -0500
67-+++ nova/nova/tests/test_utils.py 2012-02-22 20:58:15.364044601 -0500
68-@@ -17,12 +17,14 @@
69+Index: nova/nova/tests/test_utils.py
70+===================================================================
71+--- nova.orig/nova/tests/test_utils.py 2012-02-24 19:29:06.000000000 -0800
72++++ nova/nova/tests/test_utils.py 2012-02-24 19:30:40.815750120 -0800
73+@@ -17,6 +17,7 @@
74 import __builtin__
75 import datetime
76 import hashlib
77 +import itertools
78 import os
79- import StringIO
80- import tempfile
81+ import os.path
82+ import socket
83+@@ -25,6 +26,7 @@
84
85 import iso8601
86 import mox
87@@ -93,7 +97,7 @@
88
89 import nova
90 from nova import exception
91-@@ -811,3 +813,54 @@
92+@@ -813,6 +815,57 @@
93 west = utils.parse_isotime(str)
94 normed = utils.normalize_time(west)
95 self._instaneous(normed, 2012, 2, 13, 23, 53, 07, 0)
96@@ -148,9 +152,13 @@
97 + yield check_buffer, r, expected
98 + r.close()
99 + f.close()
100-diff -Naur nova.orig/nova/utils.py nova/nova/utils.py
101---- nova.orig/nova/utils.py 2012-02-22 20:57:08.060044569 -0500
102-+++ nova/nova/utils.py 2012-02-22 20:58:15.364044601 -0500
103+
104+
105+ class TestLockCleanup(test.TestCase):
106+Index: nova/nova/utils.py
107+===================================================================
108+--- nova.orig/nova/utils.py 2012-02-24 19:29:28.000000000 -0800
109++++ nova/nova/utils.py 2012-02-24 19:30:40.819750120 -0800
110 @@ -32,6 +32,7 @@
111 import re
112 import shlex
113@@ -159,7 +167,7 @@
114 import struct
115 import sys
116 import time
117-@@ -53,6 +54,7 @@
118+@@ -54,6 +55,7 @@
119 from nova.openstack.common import cfg
120
121
122@@ -167,7 +175,7 @@
123 LOG = logging.getLogger(__name__)
124 ISO_TIME_FORMAT = "%Y-%m-%dT%H:%M:%S"
125 PERFECT_TIME_FORMAT = "%Y-%m-%dT%H:%M:%S.%f"
126-@@ -1434,3 +1436,132 @@
127+@@ -1523,3 +1525,132 @@
128 return out
129 except exception.ProcessExecutionError:
130 raise exception.FileNotFound(file_path=file_path)
131@@ -300,9 +308,10 @@
132 +
133 + def close(self):
134 + self.f.close()
135-diff -Naur nova.orig/nova/virt/libvirt/connection.py nova/nova/virt/libvirt/connection.py
136---- nova.orig/nova/virt/libvirt/connection.py 2012-02-22 20:57:08.064044569 -0500
137-+++ nova/nova/virt/libvirt/connection.py 2012-02-22 20:58:15.368044601 -0500
138+Index: nova/nova/virt/libvirt/connection.py
139+===================================================================
140+--- nova.orig/nova/virt/libvirt/connection.py 2012-02-24 19:29:28.000000000 -0800
141++++ nova/nova/virt/libvirt/connection.py 2012-02-24 19:30:40.819750120 -0800
142 @@ -39,15 +39,19 @@
143
144 """
145@@ -468,7 +477,7 @@
146
147 @staticmethod
148 def get_host_ip_addr():
149-@@ -997,8 +1069,25 @@
150+@@ -998,8 +1070,25 @@
151 container_dir = '%s/rootfs' % basepath(suffix='')
152 libvirt_utils.ensure_tree(container_dir)
153
154@@ -496,10 +505,11 @@
155
156 if not disk_images:
157 disk_images = {'image_id': instance['image_ref'],
158-diff -Naur nova.orig/nova/virt/libvirt.xml.template nova/nova/virt/libvirt.xml.template
159---- nova.orig/nova/virt/libvirt.xml.template 2012-02-22 20:57:08.064044569 -0500
160-+++ nova/nova/virt/libvirt.xml.template 2012-02-22 20:58:15.364044601 -0500
161-@@ -152,8 +152,8 @@
162+Index: nova/nova/virt/libvirt.xml.template
163+===================================================================
164+--- nova.orig/nova/virt/libvirt.xml.template 2012-02-24 19:29:28.000000000 -0800
165++++ nova/nova/virt/libvirt.xml.template 2012-02-24 19:30:40.819750120 -0800
166+@@ -160,8 +160,8 @@
167
168 #end for
169 <!-- The order is significant here. File must be defined first -->

Subscribers

People subscribed via source and target branches