Merge lp:~james-page/neutron/grizzly-metadata-agent-fixup into lp:~openstack-ubuntu-testing/neutron/grizzly

Proposed by James Page
Status: Merged
Approved by: Yolanda Robla
Approved revision: 141
Merged at revision: 141
Proposed branch: lp:~james-page/neutron/grizzly-metadata-agent-fixup
Merge into: lp:~openstack-ubuntu-testing/neutron/grizzly
Diff against target: 191 lines (+47/-17)
9 files modified
debian/changelog (+3/-0)
debian/control (+27/-3)
debian/patches/fix-ubuntu-tests.patch (+5/-5)
debian/quantum-l3-agent.install (+0/-3)
debian/quantum-lbaas-agent.install (+1/-1)
debian/quantum-metadata-agent.install (+3/-0)
debian/quantum-metadata-agent.logrotate (+7/-0)
debian/quantum-plugin-nicira.install (+1/-0)
debian/rules (+0/-5)
To merge this branch: bzr merge lp:~james-page/neutron/grizzly-metadata-agent-fixup
Reviewer Review Type Date Requested Status
Yolanda Robla (community) Approve
Review via email: mp+152373@code.launchpad.net

Description of the change

1) Refactor metadata agent into separate package to support use by l3 and dhcp agents.

2) Add NVP configuration checker binary to plugin-nicira

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-04 08:48:11 +0000
3+++ debian/changelog 2013-03-08 10:50:26 +0000
4@@ -20,6 +20,9 @@
5 patch, fixed up linuxbridge path.
6 * d/p/fix-ubuntu-tests.patch: Re-added skipTest for linuxbridge tests
7 that attempt to setup udev monitors.
8+ * d/quantum-metadata-agent.*: Refactor metadata-agent into separate
9+ package and add Depends from quantum-l3-agent and quantum-dhcp-agent.
10+ * d/quantum-plugin-nicira.install: Add quantum-check-nvp-config binary.
11
12 -- Chuck Short <zulcss@ubuntu.com> Tue, 26 Feb 2013 08:27:14 -0600
13
14
15=== modified file 'debian/control'
16--- debian/control 2013-03-03 22:59:19 +0000
17+++ debian/control 2013-03-08 10:50:26 +0000
18@@ -270,12 +270,15 @@
19 Architecture: all
20 Depends:
21 quantum-common (= ${source:Version}),
22+ quantum-metadata-agent (= ${source:Version}),
23 ${misc:Depends},
24 ${python:Depends},
25 ${shlibs:Depends}
26-Breaks: quantum-common ( <= 2012.2~rc1~20120907.1154-0ubuntu1 ),
27+Breaks:
28+ quantum-common ( <= 2012.2~rc1~20120907.1154-0ubuntu1 ),
29 quantum-server ( <= 2013.1~g2-0ubuntu1 )
30-Replaces: quantum-common ( <= 2012.2~rc1~20120907.1154-0ubuntu1 ),
31+Replaces:
32+ quantum-common ( <= 2012.2~rc1~20120907.1154-0ubuntu1 ),
33 quantum-server ( <= 2013.1~g2-0ubuntu1 )
34 Description: Quantum is a virtual network service for Openstack - l3 agent
35 Quantum is a virtual network service for Openstack, and a part of
36@@ -295,6 +298,7 @@
37 dnsmasq-base,
38 dnsmasq-utils,
39 quantum-common (= ${source:Version}),
40+ quantum-metadata-agent (= ${source:Version}),
41 ${misc:Depends},
42 ${python:Depends},
43 ${shlibs:Depends}
44@@ -312,6 +316,27 @@
45 .
46 This package provides the DHCP agent.
47
48+Package: quantum-metadata-agent
49+Architecture: all
50+Depends:
51+ quantum-common (= ${source:Version}),
52+ ${misc:Depends},
53+ ${python:Depends},
54+ ${shlibs:Depends}
55+Breaks: quantum-l3-agent (<= 2013.1.g3-0ubuntu1)
56+Replaces: quantum-l3-agent (<= 2013.1.g3-0ubuntu1)
57+Description: Quantum is a virtual network service for Openstack - metadata agent
58+ Quantum is a virtual network service for Openstack, and a part of
59+ Netstack. Just like OpenStack Nova provides an API to dynamically
60+ request and configure virtual servers, Quantum provides an API to
61+ dynamically request and configure virtual networks. These networks
62+ connect "interfaces" from other OpenStack services (e.g., virtual NICs
63+ from Nova VMs). The Quantum API supports extensions to provide
64+ advanced network capabilities (e.g., QoS, ACLs, network monitoring,
65+ etc.)
66+ .
67+ This package provides the metadata proxy agent.
68+
69 Package: quantum-lbaas-agent
70 Architecture: all
71 Depends:
72@@ -524,4 +549,3 @@
73 etc.)
74 .
75 This package provides the Midonet plugin.
76-
77
78=== modified file 'debian/patches/fix-ubuntu-tests.patch'
79--- debian/patches/fix-ubuntu-tests.patch 2013-03-04 08:48:11 +0000
80+++ debian/patches/fix-ubuntu-tests.patch 2013-03-08 10:50:26 +0000
81@@ -10,7 +10,7 @@
82 self).test_create_subnet_with_two_host_routes()
83 --- a/quantum/tests/unit/test_db_plugin.py
84 +++ b/quantum/tests/unit/test_db_plugin.py
85-@@ -1392,6 +1392,7 @@ fixed_ips=ip_address%%3D%s&fixed_ips=ip_
86+@@ -1394,6 +1394,7 @@ fixed_ips=ip_address%%3D%s&fixed_ips=ip_
87 self._delete('ports', port4['port']['id'])
88
89 def test_range_allocation(self):
90@@ -18,7 +18,7 @@
91 with self.subnet(gateway_ip='10.0.0.3',
92 cidr='10.0.0.0/29') as subnet:
93 kwargs = {"fixed_ips":
94-@@ -2291,6 +2292,7 @@ class TestNetworksV2(QuantumDbPluginV2Te
95+@@ -2293,6 +2294,7 @@ class TestNetworksV2(QuantumDbPluginV2Te
96 class TestSubnetsV2(QuantumDbPluginV2TestCase):
97
98 def _test_create_subnet(self, network=None, expected=None, **kwargs):
99@@ -26,7 +26,7 @@
100 keys = kwargs.copy()
101 keys.setdefault('cidr', '10.0.0.0/24')
102 keys.setdefault('ip_version', 4)
103-@@ -3278,6 +3280,7 @@ class TestSubnetsV2(QuantumDbPluginV2Tes
104+@@ -3280,6 +3282,7 @@ class TestSubnetsV2(QuantumDbPluginV2Tes
105 host_routes=host_routes)
106
107 def test_create_subnet_with_two_host_routes(self):
108@@ -34,7 +34,7 @@
109 gateway_ip = '10.0.0.1'
110 cidr = '10.0.0.0/24'
111 allocation_pools = [{'start': '10.0.0.2',
112-@@ -3382,6 +3385,7 @@ class TestSubnetsV2(QuantumDbPluginV2Tes
113+@@ -3384,6 +3387,7 @@ class TestSubnetsV2(QuantumDbPluginV2Tes
114 self.assertEqual(res.status_int, 400)
115
116 def test_delete_subnet_with_dns(self):
117@@ -44,7 +44,7 @@
118 dns_nameservers = ['1.2.3.4']
119 --- a/quantum/tests/unit/linuxbridge/test_lb_quantum_agent.py
120 +++ b/quantum/tests/unit/linuxbridge/test_lb_quantum_agent.py
121-@@ -26,6 +26,7 @@ class TestLinuxBridge(testtools.TestCase
122+@@ -29,6 +29,7 @@ class TestLinuxBridge(base.BaseTestCase)
123
124 def setUp(self):
125 super(TestLinuxBridge, self).setUp()
126
127=== modified file 'debian/quantum-l3-agent.install'
128--- debian/quantum-l3-agent.install 2013-02-16 23:43:06 +0000
129+++ debian/quantum-l3-agent.install 2013-03-08 10:50:26 +0000
130@@ -1,7 +1,4 @@
131 bin/quantum-l3-agent usr/bin
132-bin/quantum-metadata-agent usr/bin
133-bin/quantum-ns-metadata-proxy usr/bin
134 debian/cron.d/quantum-l3-agent-netns-cleanup etc/cron.d
135 etc/l3_agent.ini etc/quantum
136-etc/metadata_agent.ini etc/quantum
137 etc/quantum/rootwrap.d/l3.filters etc/quantum/rootwrap.d
138
139=== renamed file 'debian/quantum-l3-agent.quantum-l3-agent.upstart' => 'debian/quantum-l3-agent.upstart'
140=== modified file 'debian/quantum-lbaas-agent.install'
141--- debian/quantum-lbaas-agent.install 2013-03-03 22:59:19 +0000
142+++ debian/quantum-lbaas-agent.install 2013-03-08 10:50:26 +0000
143@@ -1,4 +1,4 @@
144 bin/quantum-lbaas-agent usr/bin
145 debian/cron.d/quantum-lbaas-agent-netns-cleanup etc/cron.d
146+etc/lbaas_agent.ini etc/quantum
147 etc/quantum/rootwrap.d/lbaas-haproxy.filters etc/quantum/rootwrap.d
148-etc/lbaas_agent.ini etc/quantum
149
150=== added file 'debian/quantum-metadata-agent.install'
151--- debian/quantum-metadata-agent.install 1970-01-01 00:00:00 +0000
152+++ debian/quantum-metadata-agent.install 2013-03-08 10:50:26 +0000
153@@ -0,0 +1,3 @@
154+bin/quantum-metadata-agent usr/bin
155+bin/quantum-ns-metadata-proxy usr/bin
156+etc/metadata_agent.ini etc/quantum
157
158=== added file 'debian/quantum-metadata-agent.logrotate'
159--- debian/quantum-metadata-agent.logrotate 1970-01-01 00:00:00 +0000
160+++ debian/quantum-metadata-agent.logrotate 2013-03-08 10:50:26 +0000
161@@ -0,0 +1,7 @@
162+/var/log/quantum/metadata-agent.log {
163+ daily
164+ missingok
165+ compress
166+ delaycompress
167+ notifempty
168+}
169
170=== renamed file 'debian/quantum-l3-agent.quantum-metadata-agent.upstart' => 'debian/quantum-metadata-agent.upstart'
171=== modified file 'debian/quantum-plugin-nicira.install'
172--- debian/quantum-plugin-nicira.install 2012-06-22 16:51:14 +0000
173+++ debian/quantum-plugin-nicira.install 2013-03-08 10:50:26 +0000
174@@ -1,1 +1,2 @@
175 etc/quantum/plugins/nicira/* etc/quantum/plugins/nicira
176+bin/quantum-check-nvp-config usr/bin
177
178=== modified file 'debian/rules'
179--- debian/rules 2013-03-03 23:03:06 +0000
180+++ debian/rules 2013-03-08 10:50:26 +0000
181@@ -9,11 +9,6 @@
182 override_dh_install:
183 dh_install --fail-missing -X/usr/etc -X/usr/bin
184
185-override_dh_installinit:
186- dh_installinit -pquantum-l3-agent --name=quantum-l3-agent
187- dh_installinit -pquantum-l3-agent --name=quantum-metadata-agent
188- dh_installinit
189-
190 get-orig-source:
191 uscan --verbose --force-download --rename --destdir=../build-area
192

Subscribers

People subscribed via source and target branches