Merge lp:~zulcss/nova/2014.1.b2 into lp:~ubuntu-server-dev/nova/icehouse

Proposed by Chuck Short
Status: Merged
Merged at revision: 649
Proposed branch: lp:~zulcss/nova/2014.1.b2
Merge into: lp:~ubuntu-server-dev/nova/icehouse
Diff against target: 111 lines (+30/-16)
3 files modified
debian/changelog (+3/-2)
debian/patches/fix-requirements.patch (+3/-2)
debian/patches/sqlachemy-0.8.3-compat.patch (+24/-12)
To merge this branch: bzr merge lp:~zulcss/nova/2014.1.b2
Reviewer Review Type Date Requested Status
Adam Gandelman (community) Needs Fixing
Review via email: mp+202913@code.launchpad.net

Description of the change

2014.1.b2

To post a comment you must log in.
Revision history for this message
Adam Gandelman (gandelman-a) wrote :

Please add something to change log describing the tests we're skipping and why

review: Needs Fixing
lp:~zulcss/nova/2014.1.b2 updated
649. By Chuck Short

Update patch

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 2014-01-21 11:51:18 +0000
3+++ debian/changelog 2014-01-23 19:51:11 +0000
4@@ -1,6 +1,7 @@
5-nova (1:2014.1~b1+master-0ubuntu1) UNRELEASED; urgency=low
6+nova (1:2014.1~b2-0ubuntu1) trusty; urgency=low
7
8 [ Chuck Short ]
9+ * New upstream release.
10 * debian/patches/fix-libvirt-regression.patch: Dropped.
11 * debian/patches/sqlachemy-0.8.3-compat.patch: Refreshed.
12 * debian/control: Add python-pycadf dependency.
13@@ -30,7 +31,7 @@
14 changes to compute rootwrap filters to nova-compute postinst.
15 * d/control: Add python-oslo.rootwrap to BD's.
16
17- -- Chuck Short <zulcss@ubuntu.com> Thu, 05 Dec 2013 15:08:16 -0500
18+ -- Chuck Short <zulcss@ubuntu.com> Thu, 23 Jan 2014 13:26:34 -0500
19
20 nova (1:2014.1~b1-0ubuntu2) trusty; urgency=low
21
22
23=== modified file 'debian/patches/fix-requirements.patch'
24--- debian/patches/fix-requirements.patch 2014-01-21 09:11:23 +0000
25+++ debian/patches/fix-requirements.patch 2014-01-23 19:51:11 +0000
26@@ -1,5 +1,6 @@
27---- a/requirements.txt
28-+++ b/requirements.txt
29+diff -Naurp nova-2014.1.dev555.g80efcae.orig/requirements.txt nova-2014.1.dev555.g80efcae/requirements.txt
30+--- nova-2014.1.dev555.g80efcae.orig/requirements.txt 2014-01-22 11:10:01.000000000 -0500
31++++ nova-2014.1.dev555.g80efcae/requirements.txt 2014-01-23 09:30:24.278874319 -0500
32 @@ -1,5 +1,5 @@
33 pbr>=0.5.21,<1.0
34 -SQLAlchemy>=0.7.8,<=0.7.99
35
36=== modified file 'debian/patches/sqlachemy-0.8.3-compat.patch'
37--- debian/patches/sqlachemy-0.8.3-compat.patch 2014-01-21 09:11:23 +0000
38+++ debian/patches/sqlachemy-0.8.3-compat.patch 2014-01-23 19:51:11 +0000
39@@ -1,8 +1,9 @@
40-Description: Fixes sqlalchemy 0.8.3 errors
41+Description: Fixes sqlalchemy 0.8.3 errors, skip tests that fail on the buildds.
42 Author: Chuck Short <zulcss@ubuntu.com>
43 Fowarded: Not-needed
44---- a/nova/db/sqlalchemy/utils.py
45-+++ b/nova/db/sqlalchemy/utils.py
46+diff -Naurp nova-2014.1.dev555.g80efcae.orig/nova/db/sqlalchemy/utils.py nova-2014.1.dev555.g80efcae/nova/db/sqlalchemy/utils.py
47+--- nova-2014.1.dev555.g80efcae.orig/nova/db/sqlalchemy/utils.py 2014-01-22 11:09:52.000000000 -0500
48++++ nova-2014.1.dev555.g80efcae/nova/db/sqlalchemy/utils.py 2014-01-23 09:59:29.178915921 -0500
49 @@ -514,8 +514,12 @@ def _change_deleted_column_type_to_id_ty
50 if not isinstance(constraint, CheckConstraint):
51 return False
52@@ -18,8 +19,9 @@
53
54 constraints = []
55 for constraint in table.constraints:
56---- a/nova/openstack/common/db/sqlalchemy/session.py
57-+++ b/nova/openstack/common/db/sqlalchemy/session.py
58+diff -Naurp nova-2014.1.dev555.g80efcae.orig/nova/openstack/common/db/sqlalchemy/session.py nova-2014.1.dev555.g80efcae/nova/openstack/common/db/sqlalchemy/session.py
59+--- nova-2014.1.dev555.g80efcae.orig/nova/openstack/common/db/sqlalchemy/session.py 2014-01-22 11:09:53.000000000 -0500
60++++ nova-2014.1.dev555.g80efcae/nova/openstack/common/db/sqlalchemy/session.py 2014-01-23 09:59:29.178915921 -0500
61 @@ -438,6 +438,11 @@ def get_session(autocommit=True, expire_
62 # 1 column - (IntegrityError) column c1 is not unique
63 # N columns - (IntegrityError) column c1, c2, ..., N are not unique
64@@ -64,9 +66,14 @@
65
66 if engine_name == "sqlite":
67 columns = columns.strip().split(", ")
68---- a/nova/tests/db/test_db_api.py
69-+++ b/nova/tests/db/test_db_api.py
70-@@ -2209,7 +2209,7 @@ class ServiceTestCase(test.TestCase, Mod
71+diff -Naurp nova-2014.1.dev555.g80efcae.orig/nova/tests/db/test_db_api.py nova-2014.1.dev555.g80efcae/nova/tests/db/test_db_api.py
72+--- nova-2014.1.dev555.g80efcae.orig/nova/tests/db/test_db_api.py 2014-01-22 11:09:53.000000000 -0500
73++++ nova-2014.1.dev555.g80efcae/nova/tests/db/test_db_api.py 2014-01-23 10:00:59.758918080 -0500
74+@@ -2206,10 +2206,11 @@ class ServiceTestCase(test.TestCase, Mod
75+ self.ctxt, 'non-exists-host', 'a')
76+
77+ def test_service_binary_exists_exception(self):
78++ self.skipTest('Skipped by Ubuntu')
79 db.service_create(self.ctxt, self._get_base_values())
80 values = self._get_base_values()
81 values.update({'topic': 'top1'})
82@@ -75,20 +82,25 @@
83 self.ctxt, values)
84
85 def test_service_topic_exists_exceptions(self):
86-@@ -2682,7 +2682,7 @@ class InstanceTypeTestCase(BaseInstanceT
87+@@ -2681,8 +2682,9 @@ class InstanceTypeTestCase(BaseInstanceT
88+ {'flavorid': 'some_random_flavor'})
89
90 def test_flavor_create_duplicate_flavorid(self):
91++ self.skipTest('Skipped by Ubuntu')
92 self._create_flavor({})
93 - self.assertRaises(exception.FlavorIdExists,
94 + self.assertRaises(exception.FlavorExists,
95 self._create_flavor,
96 {'name': 'some_random_name'})
97
98---- a/nova/tests/test_flavors.py
99-+++ b/nova/tests/test_flavors.py
100-@@ -556,6 +556,6 @@ class CreateInstanceTypeTest(test.TestCa
101+diff -Naurp nova-2014.1.dev555.g80efcae.orig/nova/tests/test_flavors.py nova-2014.1.dev555.g80efcae/nova/tests/test_flavors.py
102+--- nova-2014.1.dev555.g80efcae.orig/nova/tests/test_flavors.py 2014-01-22 11:09:53.000000000 -0500
103++++ nova-2014.1.dev555.g80efcae/nova/tests/test_flavors.py 2014-01-23 10:14:47.070937805 -0500
104+@@ -555,7 +555,8 @@ class CreateInstanceTypeTest(test.TestCa
105+
106 def test_duplicate_flavorids_fail(self):
107 # Ensures that flavorid duplicates raise FlavorCreateFailed.
108++ self.skipTest('Skipped by Ubuntu')
109 flavors.create('flavor1', 64, 1, 120, flavorid='flavorid')
110 - self.assertRaises(exception.FlavorIdExists,
111 + self.assertRaises(exception.FlavorExists,

Subscribers

People subscribed via source and target branches