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
=== modified file 'debian/changelog'
--- debian/changelog 2013-04-05 19:34:40 +0000
+++ debian/changelog 2013-05-07 00:52:58 +0000
@@ -1,3 +1,14 @@
1ceilometer (2013.2.a82.g1667ded-0ubuntu1) UNRELEASED; urgency=low
2
3 * New upstream release.
4 * debian/patches/requests-dep.patch: Dropped no longer needed.
5 * debian/patches/default-dbconnection-sqlite.patch: Refreshed.
6 * debian/patches/fix-ubuntu-tests.patch: Skip tests for databases
7 that we dont do yet. (hbase)
8 * debian/rules: FTBS if the package tests fail.
9
10 -- Chuck Short <zulcss@ubuntu.com> Mon, 06 May 2013 18:49:37 -0500
11
1ceilometer (2013.1-0ubuntu1) raring; urgency=low12ceilometer (2013.1-0ubuntu1) raring; urgency=low
213
3 * New upstream release. 14 * New upstream release.
415
=== modified file 'debian/patches/default-dbconnection-sqlite.patch'
--- debian/patches/default-dbconnection-sqlite.patch 2013-03-21 16:49:24 +0000
+++ debian/patches/default-dbconnection-sqlite.patch 2013-05-07 00:52:58 +0000
@@ -1,11 +1,11 @@
1--- a/etc/ceilometer/ceilometer.conf.sample1diff -Naurp ceilometer-2013.2.a82.g1667ded.orig/etc/ceilometer/ceilometer.conf.sample ceilometer-2013.2.a82.g1667ded/etc/ceilometer/ceilometer.conf.sample
2+++ b/etc/ceilometer/ceilometer.conf.sample2--- ceilometer-2013.2.a82.g1667ded.orig/etc/ceilometer/ceilometer.conf.sample 2013-05-06 18:41:53.000000000 -0500
3@@ -262,7 +262,7 @@3+++ ceilometer-2013.2.a82.g1667ded/etc/ceilometer/ceilometer.conf.sample 2013-05-06 19:00:42.281951315 -0500
4 4@@ -415,6 +415,7 @@
5 ######## defined in ceilometer.storage ########5
6 6 # Database connection string (string value)
7-# database_connection=mongodb://localhost:27017/ceilometer7 #database_connection=mongodb://localhost:27017/ceilometer
8+database_connection=sqlite:////var/lib/ceilometer/ceilometer.sqlite8+database_connection=sqlite:////var/lib/ceilometer/ceilometer.sqlite
9 #### (StrOpt) Database connection string9
10 10
11 11 ######## defined in ceilometer.storage.sqlalchemy.models ########
1212
=== added file 'debian/patches/fix-ubuntu-tests.patch'
--- debian/patches/fix-ubuntu-tests.patch 1970-01-01 00:00:00 +0000
+++ debian/patches/fix-ubuntu-tests.patch 2013-05-07 00:52:58 +0000
@@ -0,0 +1,15 @@
1diff -Naurp ceilometer-2013.2.a82.g1667ded.orig/ceilometer/tests/db.py ceilometer-2013.2.a82.g1667ded/ceilometer/tests/db.py
2--- ceilometer-2013.2.a82.g1667ded.orig/ceilometer/tests/db.py 2013-05-06 18:41:53.000000000 -0500
3+++ ceilometer-2013.2.a82.g1667ded/ceilometer/tests/db.py 2013-05-06 19:28:30.865952124 -0500
4@@ -41,6 +41,10 @@ class TestBase(test_base.TestCase):
5 def setUp(self):
6 super(TestBase, self).setUp()
7 cfg.CONF.database_connection = self.database_connection
8- self.conn = storage.get_connection(cfg.CONF)
9+ try:
10+ self.conn = storage.get_connection(cfg.CONF)
11+ except RuntimeError:
12+ import nose
13+ raise nose.SkipTest("Unable to connect to database")
14 self.conn.upgrade()
15 self.conn.clear()
016
=== removed file 'debian/patches/requests-dep.patch'
--- debian/patches/requests-dep.patch 2013-03-21 16:49:24 +0000
+++ debian/patches/requests-dep.patch 1970-01-01 00:00:00 +0000
@@ -1,15 +0,0 @@
1Description: Remove upper bound on requests
2Author: James Page <james.page@ubuntu.com>
3Forwarded: no
4
5--- a/tools/pip-requires
6+++ b/tools/pip-requires
7@@ -15,7 +15,7 @@ python-novaclient>=2.6.10
8 python-keystoneclient>=0.2.3
9 python-swiftclient
10 lxml
11-requests<1.0
12+requests
13 extras
14 wsme>=0.5b1
15 pyyaml
160
=== modified file 'debian/patches/series'
--- debian/patches/series 2013-03-12 15:01:59 +0000
+++ debian/patches/series 2013-05-07 00:52:58 +0000
@@ -1,3 +1,3 @@
1remove-hbase-support.patch1remove-hbase-support.patch
2requests-dep.patch
3default-dbconnection-sqlite.patch2default-dbconnection-sqlite.patch
3fix-ubuntu-tests.patch
44
=== modified file 'debian/rules'
--- debian/rules 2013-03-11 22:42:51 +0000
+++ debian/rules 2013-05-07 00:52:58 +0000
@@ -14,7 +14,7 @@
14ifeq (,$(findstring nocheck, $(DEB_BUILD_OPTIONS)))14ifeq (,$(findstring nocheck, $(DEB_BUILD_OPTIONS)))
15override_dh_auto_test:15override_dh_auto_test:
16 mkdir -p ${HOME}16 mkdir -p ${HOME}
17 PYTHONPATH=$(CURDIR) nosetests || true17 PYTHONPATH=$(CURDIR) nosetests
18endif18endif
1919
20override_dh_install:20override_dh_install:

Subscribers

People subscribed via source and target branches