Merge lp:~gandelman-a/ubuntu/saucy/nova/depends into lp:~openstack-ubuntu-testing/nova/havana

Proposed by Adam Gandelman
Status: Merged
Approved by: Chuck Short
Approved revision: 581
Merged at revision: 577
Proposed branch: lp:~gandelman-a/ubuntu/saucy/nova/depends
Merge into: lp:~openstack-ubuntu-testing/nova/havana
Diff against target: 96 lines (+32/-4)
4 files modified
debian/changelog (+9/-2)
debian/control (+4/-2)
debian/patches/avoid_requirements_websockify.patch (+18/-0)
debian/patches/series (+1/-0)
To merge this branch: bzr merge lp:~gandelman-a/ubuntu/saucy/nova/depends
Reviewer Review Type Date Requested Status
Openstack Ubuntu Testers Pending
Review via email: mp+165524@code.launchpad.net

Description of the change

nova-manage is now just an entry point into the nova python package and requires that all dependencies listed in the nova's egg-info/requires.txt be satisfied. Since nova-manage comes as part of nova-common package, we need to make sure all of nova's requires.txt dependencies are satisfied or nova-manage will throw Exceptions and postinsts will fail:

* sets various dependency versions
* adds python-babel
* requirements.txt lists websockify, but the only package we ship that needs it is nova-novncproxy. Patch it out so it doesn't require moving to nova-common or python-nova becoming a global dependency pulled in by every package.

To post a comment you must log in.
581. By Adam Gandelman

changelog typo.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'debian/changelog'
--- debian/changelog 2013-05-19 21:54:04 +0000
+++ debian/changelog 2013-05-24 03:03:27 +0000
@@ -1,5 +1,6 @@
1nova (1:2013.2.a732.g7063896-0ubuntu1) UNRELEASED; urgency=low1nova (1:2013.2.a732.g7063896-0ubuntu4) UNRELEASED; urgency=low
22
3 [ Chuck Short ]
3 * New upstream version.4 * New upstream version.
4 * debian/patches/avoid_setuptools_git_dependency.patch: Dropped5 * debian/patches/avoid_setuptools_git_dependency.patch: Dropped
5 no longer needed.6 no longer needed.
@@ -7,7 +8,13 @@
7 * debian/control: Drop pep8 dependency.8 * debian/control: Drop pep8 dependency.
8 * debian/patches/fix-ubuntu-tests.patch: Dropped no longer needed.9 * debian/patches/fix-ubuntu-tests.patch: Dropped no longer needed.
910
10 -- Chuck Short <zulcss@ubuntu.com> Mon, 13 May 2013 11:37:26 -050011 [ Adam Gandelman ]
12 * debian/patches/avoid_requirements_websockify.patch: Remove websockify
13 to avoid making it a dependency of every nova package.
14 * debian/control: Add python-babel.
15 * debian/control: Set min. version 0.7.6 on python-netaddr dependency.
16
17 -- Adam Gandelman <adamg@ubuntu.com> Thu, 23 May 2013 19:25:55 -0700
1118
12nova (1:2013.1-0ubuntu2) raring; urgency=low19nova (1:2013.1-0ubuntu2) raring; urgency=low
1320
1421
=== modified file 'debian/control'
--- debian/control 2013-05-19 21:54:04 +0000
+++ debian/control 2013-05-24 03:03:27 +0000
@@ -10,6 +10,7 @@
10Build-Depends-Indep: openssh-client,10Build-Depends-Indep: openssh-client,
11 openssl,11 openssl,
12 python-amqplib,12 python-amqplib,
13 python-babel,
13 python-boto,14 python-boto,
14 python-carrot,15 python-carrot,
15 python-cheetah,16 python-cheetah,
@@ -27,7 +28,7 @@
27 python-lxml,28 python-lxml,
28 python-migrate,29 python-migrate,
29 python-mox,30 python-mox,
30 python-netaddr,31 python-netaddr (>= 0.7.6),
31 python-novaclient (>= 1:2.9.0),32 python-novaclient (>= 1:2.9.0),
32 python-oslo-config,33 python-oslo-config,
33 python-paramiko,34 python-paramiko,
@@ -59,6 +60,7 @@
59Section: python60Section: python
60Depends: openssh-client,61Depends: openssh-client,
61 openssl,62 openssl,
63 python-babel,
62 python-boto,64 python-boto,
63 python-carrot,65 python-carrot,
64 python-cheetah,66 python-cheetah,
@@ -78,7 +80,7 @@
78 python-lxml,80 python-lxml,
79 python-m2crypto,81 python-m2crypto,
80 python-migrate,82 python-migrate,
81 python-netaddr,83 python-netaddr (>= 0.7.6),
82 python-novaclient (>= 1:2.9.0),84 python-novaclient (>= 1:2.9.0),
83 python-paste,85 python-paste,
84 python-pastedeploy,86 python-pastedeploy,
8587
=== added file 'debian/patches/avoid_requirements_websockify.patch'
--- debian/patches/avoid_requirements_websockify.patch 1970-01-01 00:00:00 +0000
+++ debian/patches/avoid_requirements_websockify.patch 2013-05-24 03:03:27 +0000
@@ -0,0 +1,18 @@
1From: Adam Gandelman <adamg@ubuntu.com>
2Date: Thu May 23 19:45:33 PDT 2013
3Subject: Avoid websockify in requirements.txt
4
5The only component that requires websockify is the nova-novncproxy
6package, which pulls it in as a dependency. Leaving it in requirements.txt
7would turn it into a dependency of python-nova or nova-common.
8
9Index: nova/requirements.txt
10===================================================================
11--- nova.orig/requirements.txt 2013-05-23 19:44:34.183145856 -0700
12+++ nova/requirements.txt 2013-05-23 19:44:43.327145520 -0700
13@@ -29,5 +29,4 @@
14 python-keystoneclient>=0.2.0
15 six
16 stevedore>=0.7
17-websockify<0.4
18 oslo.config>=1.1.0
019
=== modified file 'debian/patches/series'
--- debian/patches/series 2013-05-19 21:54:04 +0000
+++ debian/patches/series 2013-05-24 03:03:27 +0000
@@ -1,3 +1,4 @@
1# Ubuntu specific patches below here. Note these can be dropped eventually.1# Ubuntu specific patches below here. Note these can be dropped eventually.
2avoid_requirements_websockify.patch
2path-to-the-xenhost.conf-fixup.patch3path-to-the-xenhost.conf-fixup.patch
3fix-docs-build-without-network.patch4fix-docs-build-without-network.patch

Subscribers

People subscribed via source and target branches