Merge lp:~james-page/ceilometer/grizzly-tidyup into lp:~openstack-ubuntu-testing/ceilometer/grizzly

Proposed by James Page
Status: Merged
Approved by: Yolanda Robla
Approved revision: 21
Merged at revision: 21
Proposed branch: lp:~james-page/ceilometer/grizzly-tidyup
Merge into: lp:~openstack-ubuntu-testing/ceilometer/grizzly
Diff against target: 354 lines (+138/-88)
6 files modified
debian/changelog (+24/-19)
debian/control (+75/-68)
debian/patches/remove-hbase-support.patch (+13/-0)
debian/patches/requests-dep.patch (+15/-0)
debian/patches/series (+2/-0)
debian/rules (+9/-1)
To merge this branch: bzr merge lp:~james-page/ceilometer/grizzly-tidyup
Reviewer Review Type Date Requested Status
Yolanda Robla (community) Approve
Review via email: mp+152793@code.launchpad.net

Description of the change

General tidy of package in preparation for Grizzly rc1

To post a comment you must log in.
Revision history for this message
Yolanda Robla (yolanda.robla) wrote :

Looks good to me

review: Approve

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 2013-03-11 15:40:24 +0000
3+++ debian/changelog 2013-03-11 22:44:20 +0000
4@@ -4,34 +4,39 @@
5 * New upstream release.
6 * debian/watch: point to the right releases.
7 * debian/rules: Enable testsuite.
8- * debian/control: Update dependencies.
9- * debian/patches/remove-unwanted-dependencies.patch: Remove mongodb dependencies.
10 * debian/ceilometer-agent-compute.postinst: Add smarts to
11 configure users after install.
12 * debian/ceilometer-common.postinst: Configure ceilometer db.
13 * debian/ceilometer-common.install: Install configuration files.
14- * debian/*.usptart: Update and use start-stop-daemon.
15- * debian/patches/exclude-tests-when-isntall.patch: Exclude tests so it
16- doesnt conflict with python-boto (LP: #1102110)
17+ * debian/*.upstart: Update and use start-stop-daemon.
18
19 [ James Page ]
20 * Added watch file.
21+ * d/rules: Fixup test suite execution as much as possible:
22+ - Set PYTHONPATH to CURDIR during execution.
23+ - Provide HOME directory for keystone tests.
24+ - Override clean to tidy up after execution.
25+ * d/control: Tidied BD's:
26+ - Drop python-ming (not the right ming).
27+ - python-oslo-config -> python-oslo.config.
28+ - Dropped misc other surplus/duplicate dependencies.
29+ - Wrapped and sorted.
30+ * d/p/request-dep.patch: Remove upper bound on python-requests inline with
31+ the rest of OpenStack.
32+ * d/p/remove-hbase-support.patch: Remove happybase dependency.
33+ * d/control: Bumped Standards-Version, no changes.
34
35 [ Yolanda Robla ]
36- * d/patches: removed unused patches
37- * d/rules: copying ceilometer.conf.sample to ceilometer.conf
38- * d/ceilometer-common.install: fix copying ceilometer.conf, copy ceilometer-dbsync
39- * d/ceilometer-common.postinst: configuring mongo connection by default,
40- * fixing user/group creation
41- * d/control: updated dependencies
42- * d/*.upstart: fixed upstart jobs
43- * d/pydist-overrides: correcting pymongo and wsme package names
44- * d/control: moved mongo recommends to api and collector
45- packages (LP: #1131627)
46- * d/control: refreshed dependencies
47- * d/files: copying pipeline.yaml
48- * d/control: fixed typo (LP: #1140309)
49- * debian/patches/exclude-tests-when-installing.patch: removed, not needed
50+ * d/rules: Copy ceilometer.conf.sample to ceilometer.conf.
51+ * d/ceilometer-common.install: Install ceilometer.conf and ceilometer-dbsync.
52+ * d/ceilometer-common.postinst: Configure mongo connection by default,
53+ fixing user/group creation.
54+ * d/*.upstart: Fix upstart jobs.
55+ * d/pydist-overrides: Correct pymongo and wsme package names.
56+ * d/control: moved mongodb recommends to api and collector
57+ packages (LP: #1131627).
58+ * d/files: copying pipeline.yaml.
59+ * d/control: fixed typo (LP: #1140309).
60
61 -- Yolanda Robla <yolanda.robla@canonical.com> Mon, 11 Mar 2013 14:23:00 +0100
62
63
64=== modified file 'debian/control'
65--- debian/control 2013-03-04 14:20:41 +0000
66+++ debian/control 2013-03-11 22:44:20 +0000
67@@ -2,64 +2,55 @@
68 Section: python
69 Priority: optional
70 Maintainer: Chuck Short <zulcss@ubuntu.com>
71-Build-Depends: debhelper (>= 8.0.0),
72+Build-Depends:
73+ debhelper (>= 8.0.0),
74 python-all (>= 2.6),
75- python-all-dev (>= 2.6.6-3~)
76-Build-Depends-Indep:
77 python-nose,
78- python-unittest2,
79+ python-anyjson (>= 0.2.4),
80+ python-argparse,
81+ python-babel (>= 0.9.6),
82 python-coverage,
83- python-mock,
84- python-mox,
85- python-babel (>= 0.9.6),
86- python-ming,
87- python-nova,
88- python-glance,
89- python-swift,
90- python-setuptools-git (>= 0.4),
91- python-netifaces,
92- python-sphinx,
93 python-docutils,
94- python-oslo-config,
95- python-yaml,
96- python-flask (>= 0.9)
97-Depends:
98- python-webob (>= 1.2),
99+ python-eventlet,
100+ python-extras,
101+ python-flask (>= 0.9),
102+ python-glanceclient,
103+ python-iso8601,
104+ python-keystoneclient (<< 1:0.3),
105 python-kombu,
106- python-iso8601,
107 python-lockfile,
108+ python-lxml,
109+ python-migrate,
110+ python-mock,
111+ python-mox,
112 python-netaddr,
113- python-argparse,
114- python-sqlalchemy (<= 0.7.9),
115- python-migrate,
116+ python-netifaces,
117+ python-novaclient (>= 2.6.10),
118+ python-oslo.config,
119+ python-pecan (>= 0.2.0),
120 python-pymongo (>= 2.2),
121- python-eventlet,
122- python-anyjson (>= 0.2.4),
123- python-flask (>= 0.9),
124- python-pecan (>= 0.2.0),
125+ python-requests,
126+ python-setuptools-git (>= 0.4),
127+ python-sphinx,
128+ python-sqlalchemy,
129 python-stevedore (>= 0.6),
130- python-glanceclient,
131- python-novaclient (>= 2.6.10),
132- python-keystoneclient (<< 0.3),
133+ python-swift,
134 python-swiftclient,
135- python-lxml,
136- python-requests,
137- python-extras,
138+ python-unittest2,
139+ python-webob (>= 1.2),
140 python-wsme (>= 0.5b1),
141- python-pyyaml,
142- python-oslo-config
143-Standards-Version: 3.9.3
144+ python-yaml
145+Standards-Version: 3.9.4
146
147 Package: python-ceilometer
148 Architecture: all
149-Depends: ${python:Depends},
150- ${misc:Depends}
151+Depends: ${misc:Depends}, ${python:Depends}
152 Description: ceilometer python libraries
153- Ceilometer aims to deliver a unique point of contact for billing systems to
154- aquire all counters they need to establish customer billing, accross all
155- current and future OpenStack components. The delivery of counters must
156- be tracable and auditable, the counters must be easily extensible to support
157- new projects, and agents doing data collections should be
158+ Ceilometer aims to deliver a unique point of contact for billing systems to
159+ aquire all counters they need to establish customer billing, accross all
160+ current and future OpenStack components. The delivery of counters must
161+ be tracable and auditable, the counters must be easily extensible to support
162+ new projects, and agents doing data collections should be
163 independent of the overall system.
164 .
165 (A ceilometer is an instrument that measures cloud coverage.)
166@@ -69,14 +60,18 @@
167
168 Package: ceilometer-common
169 Architecture: all
170-Depends: python-ceilometer (= ${binary:Version}), ${python:Depends}, ${misc:Depends}, adduser
171+Depends:
172+ adduser,
173+ python-ceilometer (= ${binary:Version}),
174+ ${misc:Depends},
175+ ${python:Depends}
176 Provides: ${python:Provides}
177 Description: ceilometer common files
178 Ceilometer aims to deliver a unique point of contact for billing systems to
179- aquire all counters they need to establish customer billing, accross all
180- current and future OpenStack components. The delivery of counters must
181- be tracable and auditable, the counters must be easily extensible to support
182- new projects, and agents doing data collections should be
183+ aquire all counters they need to establish customer billing, accross all
184+ current and future OpenStack components. The delivery of counters must
185+ be tracable and auditable, the counters must be easily extensible to support
186+ new projects, and agents doing data collections should be
187 independent of the overall system.
188 .
189 (A ceilometer is an instrument that measures cloud coverage.)
190@@ -85,14 +80,17 @@
191
192 Package: ceilometer-collector
193 Architecture: all
194-Depends: ceilometer-common (= ${binary:Version}), ${python:Depends}, ${misc:Depends}
195+Depends:
196+ ceilometer-common (= ${binary:Version}),
197+ ${misc:Depends},
198+ ${python:Depends}
199 Recommends: mongodb
200 Description: ceilometer collector service
201 Ceilometer aims to deliver a unique point of contact for billing systems to
202- aquire all counters they need to establish customer billing, accross all
203- current and future OpenStack components. The delivery of counters must
204- be tracable and auditable, the counters must be easily extensible to support
205- new projects, and agents doing data collections should be
206+ aquire all counters they need to establish customer billing, accross all
207+ current and future OpenStack components. The delivery of counters must
208+ be tracable and auditable, the counters must be easily extensible to support
209+ new projects, and agents doing data collections should be
210 independent of the overall system.
211 .
212 (A ceilometer is an instrument that measures cloud coverage.)
213@@ -101,27 +99,33 @@
214
215 Package: ceilometer-api
216 Architecture: all
217-Depends: ceilometer-common (= ${binary:Version}), ${python:Depends}, ${misc:Depends}
218+Depends:
219+ ceilometer-common (= ${binary:Version}),
220+ ${misc:Depends},
221+ ${python:Depends}
222 Recommends: mongodb
223 Description: ceilometer api service
224 Ceilometer aims to deliver a unique point of contact for billing systems to
225- aquire all counters they need to establish customer billing, accross all
226- current and future OpenStack components. The delivery of counters must
227- be tracable and auditable, the counters must be easily extensible to support
228- new projects, and agents doing data collections should be
229+ aquire all counters they need to establish customer billing, accross all
230+ current and future OpenStack components. The delivery of counters must
231+ be tracable and auditable, the counters must be easily extensible to support
232+ new projects, and agents doing data collections should be
233 independent of the overall system.
234 .
235 This package contains the api service
236
237 Package: ceilometer-agent-compute
238 Architecture: all
239-Depends: ceilometer-common (= ${binary:Version}), ${python:Depends}, ${misc:Depends}
240+Depends:
241+ ceilometer-common (= ${binary:Version}),
242+ ${misc:Depends},
243+ ${python:Depends}
244 Description: ceilometer compute agent
245 Ceilometer aims to deliver a unique point of contact for billing systems to
246- aquire all counters they need to establish customer billing, accross all
247- current and future OpenStack components. The delivery of counters must
248- be tracable and auditable, the counters must be easily extensible to support
249- new projects, and agents doing data collections should be
250+ aquire all counters they need to establish customer billing, accross all
251+ current and future OpenStack components. The delivery of counters must
252+ be tracable and auditable, the counters must be easily extensible to support
253+ new projects, and agents doing data collections should be
254 independent of the overall system.
255 .
256 (A ceilometer is an instrument that measures cloud coverage.)
257@@ -130,13 +134,16 @@
258
259 Package: ceilometer-agent-central
260 Architecture: all
261-Depends: ceilometer-common (= ${binary:Version}), ${python:Depends}, ${misc:Depends}
262+Depends:
263+ ceilometer-common (= ${binary:Version}),
264+ ${misc:Depends},
265+ ${python:Depends}
266 Description: ceilometer central agent
267 Ceilometer aims to deliver a unique point of contact for billing systems to
268- aquire all counters they need to establish customer billing, accross all
269- current and future OpenStack components. The delivery of counters must
270- be tracable and auditable, the counters must be easily extensible to support
271- new projects, and agents doing data collections should be
272+ aquire all counters they need to establish customer billing, accross all
273+ current and future OpenStack components. The delivery of counters must
274+ be tracable and auditable, the counters must be easily extensible to support
275+ new projects, and agents doing data collections should be
276 independent of the overall system.
277 .
278 (A ceilometer is an instrument that measures cloud coverage.)
279
280=== added directory 'debian/patches'
281=== added file 'debian/patches/remove-hbase-support.patch'
282--- debian/patches/remove-hbase-support.patch 1970-01-01 00:00:00 +0000
283+++ debian/patches/remove-hbase-support.patch 2013-03-11 22:44:20 +0000
284@@ -0,0 +1,13 @@
285+Description: Drop unpackaged happybase from dependencies
286+ This avoids breaking stevedore extensions via endpoints.
287+Author: James Page <james.page@ubuntu.com>
288+Forwarded: not-needed
289+
290+--- a/tools/pip-requires
291++++ b/tools/pip-requires
292+@@ -25,4 +25,4 @@ extras
293+ wsme>=0.5b1
294+ pyyaml
295+ http://tarballs.openstack.org/oslo-config/oslo.config-1.1.0b1.tar.gz#egg=oslo.config
296+-happybase>=0.4
297++#happybase>=0.4
298
299=== added file 'debian/patches/requests-dep.patch'
300--- debian/patches/requests-dep.patch 1970-01-01 00:00:00 +0000
301+++ debian/patches/requests-dep.patch 2013-03-11 22:44:20 +0000
302@@ -0,0 +1,15 @@
303+Description: Remove upper bound on requests
304+Author: James Page <james.page@ubuntu.com>
305+Forwarded: no
306+
307+--- a/tools/pip-requires
308++++ b/tools/pip-requires
309+@@ -20,7 +20,7 @@ python-novaclient>=2.6.10
310+ http://tarballs.openstack.org/python-keystoneclient/python-keystoneclient-master.tar.gz#egg=python-keystoneclient
311+ python-swiftclient
312+ lxml
313+-requests<1.0
314++requests
315+ extras
316+ wsme>=0.5b1
317+ pyyaml
318
319=== added file 'debian/patches/series'
320--- debian/patches/series 1970-01-01 00:00:00 +0000
321+++ debian/patches/series 2013-03-11 22:44:20 +0000
322@@ -0,0 +1,2 @@
323+remove-hbase-support.patch
324+requests-dep.patch
325
326=== modified file 'debian/rules'
327--- debian/rules 2013-01-17 17:45:41 +0000
328+++ debian/rules 2013-03-11 22:44:20 +0000
329@@ -3,6 +3,8 @@
330 # Uncomment this to turn on verbose mode.
331 #export DH_VERBOSE=1
332
333+export HOME="$(CURDIR)/debian/test"
334+
335 %:
336 dh $@ --with python2
337
338@@ -11,9 +13,15 @@
339
340 ifeq (,$(findstring nocheck, $(DEB_BUILD_OPTIONS)))
341 override_dh_auto_test:
342- nosetests || true
343+ mkdir -p ${HOME}
344+ PYTHONPATH=$(CURDIR) nosetests || true
345 endif
346
347 override_dh_install:
348 cp etc/ceilometer/ceilometer.conf.sample etc/ceilometer/ceilometer.conf
349 dh_install
350+
351+override_dh_auto_clean:
352+ rm -f etc/ceilometer/ceilometer.conf
353+ rm -f ${HOME}
354+ dh_auto_clean

Subscribers

People subscribed via source and target branches