Merge lp:~zulcss/ceilometer/ceilomter-ftbfs into lp:~openstack-ubuntu-testing/ceilometer/havana

Proposed by Chuck Short
Status: Needs review
Proposed branch: lp:~zulcss/ceilometer/ceilomter-ftbfs
Merge into: lp:~openstack-ubuntu-testing/ceilometer/havana
Diff against target: 106 lines (+38/-27)
6 files modified
debian/changelog (+11/-0)
debian/patches/default-dbconnection-sqlite.patch (+10/-10)
debian/patches/fix-ubuntu-tests.patch (+15/-0)
debian/patches/requests-dep.patch (+0/-15)
debian/patches/series (+1/-1)
debian/rules (+1/-1)
To merge this branch: bzr merge lp:~zulcss/ceilometer/ceilomter-ftbfs
Reviewer Review Type Date Requested Status
Openstack Ubuntu Testers Pending
Review via email: mp+162693@code.launchpad.net

Description of the change

fixes ftbfs and run tests per commit

To post a comment you must log in.

Unmerged revisions

28. By Chuck Short

* New upstream release.
* debian/patches/requests-dep.patch: Dropped no longer needed.
* debian/patches/default-dbconnection-sqlite.patch: Refreshed.
* debian/patches/fix-ubuntu-tests.patch: Skip tests for databases
  that we dont do yet. (hbase)
* debian/rules: FTBS if the package tests fail.

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-04-05 19:34:40 +0000
3+++ debian/changelog 2013-05-07 00:52:58 +0000
4@@ -1,3 +1,14 @@
5+ceilometer (2013.2.a82.g1667ded-0ubuntu1) UNRELEASED; urgency=low
6+
7+ * New upstream release.
8+ * debian/patches/requests-dep.patch: Dropped no longer needed.
9+ * debian/patches/default-dbconnection-sqlite.patch: Refreshed.
10+ * debian/patches/fix-ubuntu-tests.patch: Skip tests for databases
11+ that we dont do yet. (hbase)
12+ * debian/rules: FTBS if the package tests fail.
13+
14+ -- Chuck Short <zulcss@ubuntu.com> Mon, 06 May 2013 18:49:37 -0500
15+
16 ceilometer (2013.1-0ubuntu1) raring; urgency=low
17
18 * New upstream release.
19
20=== modified file 'debian/patches/default-dbconnection-sqlite.patch'
21--- debian/patches/default-dbconnection-sqlite.patch 2013-03-21 16:49:24 +0000
22+++ debian/patches/default-dbconnection-sqlite.patch 2013-05-07 00:52:58 +0000
23@@ -1,11 +1,11 @@
24---- a/etc/ceilometer/ceilometer.conf.sample
25-+++ b/etc/ceilometer/ceilometer.conf.sample
26-@@ -262,7 +262,7 @@
27-
28- ######## defined in ceilometer.storage ########
29-
30--# database_connection=mongodb://localhost:27017/ceilometer
31+diff -Naurp ceilometer-2013.2.a82.g1667ded.orig/etc/ceilometer/ceilometer.conf.sample ceilometer-2013.2.a82.g1667ded/etc/ceilometer/ceilometer.conf.sample
32+--- ceilometer-2013.2.a82.g1667ded.orig/etc/ceilometer/ceilometer.conf.sample 2013-05-06 18:41:53.000000000 -0500
33++++ ceilometer-2013.2.a82.g1667ded/etc/ceilometer/ceilometer.conf.sample 2013-05-06 19:00:42.281951315 -0500
34+@@ -415,6 +415,7 @@
35+
36+ # Database connection string (string value)
37+ #database_connection=mongodb://localhost:27017/ceilometer
38 +database_connection=sqlite:////var/lib/ceilometer/ceilometer.sqlite
39- #### (StrOpt) Database connection string
40-
41-
42+
43+
44+ ######## defined in ceilometer.storage.sqlalchemy.models ########
45
46=== added file 'debian/patches/fix-ubuntu-tests.patch'
47--- debian/patches/fix-ubuntu-tests.patch 1970-01-01 00:00:00 +0000
48+++ debian/patches/fix-ubuntu-tests.patch 2013-05-07 00:52:58 +0000
49@@ -0,0 +1,15 @@
50+diff -Naurp ceilometer-2013.2.a82.g1667ded.orig/ceilometer/tests/db.py ceilometer-2013.2.a82.g1667ded/ceilometer/tests/db.py
51+--- ceilometer-2013.2.a82.g1667ded.orig/ceilometer/tests/db.py 2013-05-06 18:41:53.000000000 -0500
52++++ ceilometer-2013.2.a82.g1667ded/ceilometer/tests/db.py 2013-05-06 19:28:30.865952124 -0500
53+@@ -41,6 +41,10 @@ class TestBase(test_base.TestCase):
54+ def setUp(self):
55+ super(TestBase, self).setUp()
56+ cfg.CONF.database_connection = self.database_connection
57+- self.conn = storage.get_connection(cfg.CONF)
58++ try:
59++ self.conn = storage.get_connection(cfg.CONF)
60++ except RuntimeError:
61++ import nose
62++ raise nose.SkipTest("Unable to connect to database")
63+ self.conn.upgrade()
64+ self.conn.clear()
65
66=== removed file 'debian/patches/requests-dep.patch'
67--- debian/patches/requests-dep.patch 2013-03-21 16:49:24 +0000
68+++ debian/patches/requests-dep.patch 1970-01-01 00:00:00 +0000
69@@ -1,15 +0,0 @@
70-Description: Remove upper bound on requests
71-Author: James Page <james.page@ubuntu.com>
72-Forwarded: no
73-
74---- a/tools/pip-requires
75-+++ b/tools/pip-requires
76-@@ -15,7 +15,7 @@ python-novaclient>=2.6.10
77- python-keystoneclient>=0.2.3
78- python-swiftclient
79- lxml
80--requests<1.0
81-+requests
82- extras
83- wsme>=0.5b1
84- pyyaml
85
86=== modified file 'debian/patches/series'
87--- debian/patches/series 2013-03-12 15:01:59 +0000
88+++ debian/patches/series 2013-05-07 00:52:58 +0000
89@@ -1,3 +1,3 @@
90 remove-hbase-support.patch
91-requests-dep.patch
92 default-dbconnection-sqlite.patch
93+fix-ubuntu-tests.patch
94
95=== modified file 'debian/rules'
96--- debian/rules 2013-03-11 22:42:51 +0000
97+++ debian/rules 2013-05-07 00:52:58 +0000
98@@ -14,7 +14,7 @@
99 ifeq (,$(findstring nocheck, $(DEB_BUILD_OPTIONS)))
100 override_dh_auto_test:
101 mkdir -p ${HOME}
102- PYTHONPATH=$(CURDIR) nosetests || true
103+ PYTHONPATH=$(CURDIR) nosetests
104 endif
105
106 override_dh_install:

Subscribers

People subscribed via source and target branches