Merge lp:~james-page/charms/trusty/neutron-api-odl/kilo-support into lp:~openstack-charmers/charms/trusty/neutron-api-odl/vpp
- Trusty Tahr (14.04)
- kilo-support
- Merge into vpp
Proposed by
James Page
Status: | Merged |
---|---|
Merged at revision: | 4 |
Proposed branch: | lp:~james-page/charms/trusty/neutron-api-odl/kilo-support |
Merge into: | lp:~openstack-charmers/charms/trusty/neutron-api-odl/vpp |
Diff against target: |
342 lines (+20/-219) 9 files modified
config.yaml (+4/-14) hooks/odl_data.py (+1/-18) hooks/odl_utils.py (+4/-5) templates/icehouse/neutron.conf (+0/-40) templates/kilo/ml2_conf.ini (+0/-41) templates/kilo/neutron.conf (+0/-42) templates/ml2_conf.ini (+11/-9) templates/odl_registration (+0/-29) templates/parts/rabbitmq (+0/-21) |
To merge this branch: | bzr merge lp:~james-page/charms/trusty/neutron-api-odl/kilo-support |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Liam Young (community) | Approve | ||
Review via email: mp+264850@code.launchpad.net |
Commit message
Description of the change
To post a comment you must log in.
- 10. By James Page
-
Drop backports PPA, networking-odl now in cloud-archive
- 11. By James Page
-
Add extra type drivers for kilo
- 12. By James Page
-
Tidy templates
Revision history for this message
Liam Young (gnuoy) wrote : | # |
FYI Tested with cisco-vpp deploy
Preview Diff
[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1 | === modified file 'config.yaml' |
2 | --- config.yaml 2015-06-24 12:44:33 +0000 |
3 | +++ config.yaml 2015-07-20 10:51:26 +0000 |
4 | @@ -1,18 +1,4 @@ |
5 | options: |
6 | - use-syslog: |
7 | - type: boolean |
8 | - default: False |
9 | - description: | |
10 | - By default, all services will log into their corresponding log files. |
11 | - Setting this to True will force all services to log to the syslog. |
12 | - debug: |
13 | - default: False |
14 | - type: boolean |
15 | - description: Enable debug logging |
16 | - verbose: |
17 | - default: False |
18 | - type: boolean |
19 | - description: Enable verbose logging |
20 | vlan-ranges: |
21 | type: string |
22 | default: "physnet1:1000:2000" |
23 | @@ -29,3 +15,7 @@ |
24 | vxlan |
25 | . |
26 | Multiple types can be provided - field is space delimited. |
27 | + security-groups: |
28 | + default: false |
29 | + type: boolean |
30 | + description: Enable OpenDayLight managed security groups. |
31 | |
32 | === modified file 'hooks/odl_data.py' |
33 | --- hooks/odl_data.py 2015-06-30 15:30:27 +0000 |
34 | +++ hooks/odl_data.py 2015-07-20 10:51:26 +0000 |
35 | @@ -70,26 +70,9 @@ |
36 | # first nodes private-address |
37 | self['odl_ip'] = first_contoller.get('private-address') |
38 | self['odl_port'] = first_contoller.get('port') |
39 | - # ODL passing wrong port atm |
40 | - self['odl_port'] = '8181' |
41 | self['odl_username'] = first_contoller.get('username') |
42 | self['odl_password'] = first_contoller.get('password') |
43 | |
44 | - def provide_data(self): |
45 | - odl_cmds = { |
46 | - 'feature:install': [ |
47 | - 'cosc-cvpn-ovs-rest', |
48 | - 'odl-netconf-connector-all' |
49 | - ], |
50 | - 'log:set': { |
51 | - 'TRACE': ['cosc-cvpn-ovs-rest', 'odl-netconf-connector-all'], |
52 | - } |
53 | - } |
54 | - relation_info = { |
55 | - 'odl-cmds': json.dump(odl_cmds) |
56 | - } |
57 | - return relation_info |
58 | - |
59 | def is_ready(self): |
60 | if 'password' in self.get_first_data(): |
61 | return True |
62 | @@ -99,9 +82,9 @@ |
63 | |
64 | class ConfigTranslation(dict): |
65 | def __init__(self): |
66 | - self['use_syslog'] = config('use-syslog') |
67 | self['vlan_ranges'] = config('vlan-ranges') |
68 | self['overlay_network_type'] = self.get_overlay_network_type() |
69 | + self['security_groups'] = config('security-groups') |
70 | |
71 | def get_overlay_network_type(self): |
72 | overlay_networks = config('overlay-network-type').split() |
73 | |
74 | === modified file 'hooks/odl_utils.py' |
75 | --- hooks/odl_utils.py 2015-06-24 12:44:33 +0000 |
76 | +++ hooks/odl_utils.py 2015-07-20 10:51:26 +0000 |
77 | @@ -1,13 +1,10 @@ |
78 | -from charmhelpers.core.hookenv import( |
79 | - INFO, |
80 | - config, |
81 | - log, |
82 | -) |
83 | from charmhelpers.fetch import ( |
84 | apt_install, |
85 | filter_installed_packages, |
86 | ) |
87 | |
88 | +from charmhelpers.contrib.openstack.utils import os_release |
89 | + |
90 | NEUTRON_CONF_DIR = "/etc/neutron" |
91 | NEUTRON_CONF = '%s/neutron.conf' % NEUTRON_CONF_DIR |
92 | ML2_CONF = '%s/plugins/ml2/ml2_conf.ini' % NEUTRON_CONF_DIR |
93 | @@ -23,6 +20,8 @@ |
94 | |
95 | def install_packages(servicename): |
96 | pkgs = filter_installed_packages(determine_packages()) |
97 | + if os_release('neutron-common') >= 'kilo': |
98 | + pkgs.extend(['python-networking-odl']) |
99 | apt_install(pkgs) |
100 | |
101 | |
102 | |
103 | === removed directory 'templates/icehouse' |
104 | === removed file 'templates/icehouse/neutron.conf' |
105 | --- templates/icehouse/neutron.conf 2015-06-24 12:22:08 +0000 |
106 | +++ templates/icehouse/neutron.conf 1970-01-01 00:00:00 +0000 |
107 | @@ -1,40 +0,0 @@ |
108 | -# icehouse |
109 | -############################################################################### |
110 | -# [ WARNING ] |
111 | -# Configuration file maintained by Juju. Local changes may be overwritten. |
112 | -# Config managed by neutron-openvswitch charm |
113 | -############################################################################### |
114 | -[DEFAULT] |
115 | -verbose = {{ verbose }} |
116 | -debug = {{ debug }} |
117 | -use_syslog = {{ use_syslog }} |
118 | -state_path = /var/lib/neutron |
119 | -lock_path = $state_path/lock |
120 | -bind_host = 0.0.0.0 |
121 | -bind_port = 9696 |
122 | -{% if network_device_mtu -%} |
123 | -network_device_mtu = {{ network_device_mtu }} |
124 | -{% endif -%} |
125 | -{% if core_plugin -%} |
126 | -core_plugin = {{ core_plugin }} |
127 | -{% endif -%} |
128 | - |
129 | -api_paste_config = /etc/neutron/api-paste.ini |
130 | -auth_strategy = keystone |
131 | -{% if notifications == 'True' -%} |
132 | -notification_driver = neutron.openstack.common.notifier.rpc_notifier |
133 | -{% endif -%} |
134 | -default_notification_level = INFO |
135 | -notification_topics = notifications |
136 | - |
137 | -{% include "parts/rabbitmq" %} |
138 | - |
139 | -[QUOTAS] |
140 | - |
141 | -[DEFAULT_SERVICETYPE] |
142 | - |
143 | -[AGENT] |
144 | -root_helper = sudo neutron-rootwrap /etc/neutron/rootwrap.conf |
145 | - |
146 | -[keystone_authtoken] |
147 | -signing_dir = /var/lib/neutron/keystone-signing |
148 | |
149 | === removed directory 'templates/kilo' |
150 | === removed file 'templates/kilo/ml2_conf.ini' |
151 | --- templates/kilo/ml2_conf.ini 2015-06-24 12:22:08 +0000 |
152 | +++ templates/kilo/ml2_conf.ini 1970-01-01 00:00:00 +0000 |
153 | @@ -1,41 +0,0 @@ |
154 | -# kilo |
155 | -############################################################################### |
156 | -# [ WARNING ] |
157 | -# Configuration file maintained by Juju. Local changes may be overwritten. |
158 | -############################################################################### |
159 | -[ml2] |
160 | -type_drivers = {{ overlay_network_type }} |
161 | -tenant_network_types = gre,vxlan,vlan,flat |
162 | -mechanism_drivers = opendaylight |
163 | - |
164 | -[ml2_type_gre] |
165 | -tunnel_id_ranges = 1:1000 |
166 | - |
167 | -[ml2_type_vxlan] |
168 | -vni_ranges = 1001:2000 |
169 | - |
170 | -[ml2_odl] |
171 | -username = {{ odl_username }} |
172 | -password = {{ odl_password }} |
173 | -url = http://{{ odl_ip }}:{{ odl_port }}/controller/nb/v2/neutron |
174 | - |
175 | -[odl] |
176 | -nodes = |
177 | -network_vlan_ranges = {{ vlan_ranges }} |
178 | -tunnel_id_ranges = 1:1000 |
179 | -tun_peer_patch_port = patch-int |
180 | -int_peer_patch_port = patch-tun |
181 | -tenant_network_type = vlan |
182 | -tunnel_bridge = br-tun |
183 | -integration_bridge = br-int |
184 | -controllers = {{ odl_ip }}:{{ odl_port }}:admin:admin |
185 | - |
186 | -[agent] |
187 | -tunnel_types = {{ overlay_network_type }} |
188 | -{% if veth_mtu -%} |
189 | -veth_mtu = {{ veth_mtu }} |
190 | -{% endif %} |
191 | - |
192 | -[securitygroup] |
193 | -firewall_driver = neutron.agent.linux.firewall.NoopFirewallDriver |
194 | -enable_security_group = False |
195 | |
196 | === removed file 'templates/kilo/neutron.conf' |
197 | --- templates/kilo/neutron.conf 2015-06-24 12:22:08 +0000 |
198 | +++ templates/kilo/neutron.conf 1970-01-01 00:00:00 +0000 |
199 | @@ -1,42 +0,0 @@ |
200 | -# icehouse |
201 | -############################################################################### |
202 | -# [ WARNING ] |
203 | -# Configuration file maintained by Juju. Local changes may be overwritten. |
204 | -# Config managed by neutron-openvswitch charm |
205 | -############################################################################### |
206 | -[DEFAULT] |
207 | -verbose = {{ verbose }} |
208 | -debug = {{ debug }} |
209 | -use_syslog = {{ use_syslog }} |
210 | -state_path = /var/lib/neutron |
211 | -bind_host = 0.0.0.0 |
212 | -bind_port = 9696 |
213 | -{% if network_device_mtu -%} |
214 | -network_device_mtu = {{ network_device_mtu }} |
215 | -{% endif -%} |
216 | -{% if core_plugin -%} |
217 | -core_plugin = {{ core_plugin }} |
218 | -{% endif -%} |
219 | - |
220 | -api_paste_config = /etc/neutron/api-paste.ini |
221 | -auth_strategy = keystone |
222 | -notification_driver = neutron.openstack.common.notifier.rpc_notifier |
223 | -default_notification_level = INFO |
224 | -notification_topics = notifications |
225 | - |
226 | -{% include "section-zeromq" %} |
227 | - |
228 | -{% include "section-rabbitmq-oslo" %} |
229 | - |
230 | -[QUOTAS] |
231 | - |
232 | -[DEFAULT_SERVICETYPE] |
233 | - |
234 | -[AGENT] |
235 | -root_helper = sudo neutron-rootwrap /etc/neutron/rootwrap.conf |
236 | - |
237 | -[keystone_authtoken] |
238 | -signing_dir = /var/lib/neutron/keystone-signing |
239 | - |
240 | -[oslo_concurrency] |
241 | -lock_path = $state_path/lock |
242 | |
243 | === renamed file 'templates/icehouse/ml2_conf.ini' => 'templates/ml2_conf.ini' |
244 | --- templates/icehouse/ml2_conf.ini 2015-06-30 15:30:27 +0000 |
245 | +++ templates/ml2_conf.ini 2015-07-20 10:51:26 +0000 |
246 | @@ -1,12 +1,11 @@ |
247 | -# kilo |
248 | ############################################################################### |
249 | # [ WARNING ] |
250 | -# Configuration file maintained by Juju. Local changes may be overwritten. |
251 | +# Configuration file maintained by Juju. Local changes may be overwritten. |
252 | ############################################################################### |
253 | [ml2] |
254 | -type_drivers = {{ overlay_network_type }},local,flat,vlan,gre |
255 | +type_drivers = {{ overlay_network_type }},local,flat,vlan |
256 | tenant_network_types = {{ overlay_network_type }} |
257 | -mechanism_drivers = opendaylight |
258 | +mechanism_drivers = opendaylight |
259 | |
260 | [ml2_type_flat] |
261 | # Provider nets only |
262 | @@ -20,11 +19,14 @@ |
263 | [ml2_type_vxlan] |
264 | vni_ranges = 1001:2000 |
265 | |
266 | -[securitygroup] |
267 | -firewall_driver = neutron.agent.linux.firewall.NoopFirewallDriver |
268 | -enable_security_group = False |
269 | - |
270 | [ml2_odl] |
271 | -url = http://{{ odl_ip }}:{{ odl_port }}/controller/nb/v2/neutron |
272 | username = {{ odl_username }} |
273 | password = {{ odl_password }} |
274 | +url = http://{{ odl_ip }}:{{ odl_port }}/controller/nb/v2/neutron |
275 | + |
276 | +[securitygroup] |
277 | +{% if security_groups -%} |
278 | +enable_security_group = True |
279 | +{% else -%} |
280 | +enable_security_group = False |
281 | +{% endif -%} |
282 | |
283 | === removed file 'templates/odl_registration' |
284 | --- templates/odl_registration 2015-06-24 12:22:08 +0000 |
285 | +++ templates/odl_registration 1970-01-01 00:00:00 +0000 |
286 | @@ -1,29 +0,0 @@ |
287 | -<module xmlns="urn:opendaylight:params:xml:ns:yang:controller:config"> |
288 | - <type xmlns:prefix="urn:opendaylight:params:xml:ns:yang:controller:md:sal:connector:netconf">prefix:sal-netconf-connector</type> |
289 | - <name>{{ vpp_host }}</name> |
290 | - <address xmlns="urn:opendaylight:params:xml:ns:yang:controller:md:sal:connector:netconf">{{ vpp_ip }}</address> |
291 | - <port xmlns="urn:opendaylight:params:xml:ns:yang:controller:md:sal:connector:netconf">2022</port> |
292 | - <username xmlns="urn:opendaylight:params:xml:ns:yang:controller:md:sal:connector:netconf">{{ vpp_username }}</username> |
293 | - <password xmlns="urn:opendaylight:params:xml:ns:yang:controller:md:sal:connector:netconf">{{ vpp_password }}</password> |
294 | - <tcp-only xmlns="urn:opendaylight:params:xml:ns:yang:controller:md:sal:connector:netconf">false</tcp-only> |
295 | - <event-executor xmlns="urn:opendaylight:params:xml:ns:yang:controller:md:sal:connector:netconf"> |
296 | - <type xmlns:prefix="urn:opendaylight:params:xml:ns:yang:controller:netty">prefix:netty-event-executor</type> |
297 | - <name>global-event-executor</name> |
298 | - </event-executor> |
299 | - <binding-registry xmlns="urn:opendaylight:params:xml:ns:yang:controller:md:sal:connector:netconf"> |
300 | - <type xmlns:prefix="urn:opendaylight:params:xml:ns:yang:controller:md:sal:binding">prefix:binding-broker-osgi-registry</type> |
301 | - <name>binding-osgi-broker</name> |
302 | - </binding-registry> |
303 | - <dom-registry xmlns="urn:opendaylight:params:xml:ns:yang:controller:md:sal:connector:netconf"> |
304 | - <type xmlns:prefix="urn:opendaylight:params:xml:ns:yang:controller:md:sal:dom">prefix:dom-broker-osgi-registry</type> |
305 | - <name>dom-broker</name> |
306 | - </dom-registry> |
307 | - <client-dispatcher xmlns="urn:opendaylight:params:xml:ns:yang:controller:md:sal:connector:netconf"> |
308 | - <type xmlns:prefix="urn:opendaylight:params:xml:ns:yang:controller:config:netconf">prefix:netconf-client-dispatcher</type> |
309 | - <name>global-netconf-dispatcher</name> |
310 | - </client-dispatcher> |
311 | - <processing-executor xmlns="urn:opendaylight:params:xml:ns:yang:controller:md:sal:connector:netconf"> |
312 | - <type xmlns:prefix="urn:opendaylight:params:xml:ns:yang:controller:threadpool">prefix:threadpool</type> |
313 | - <name>global-netconf-processing-executor</name> |
314 | - </processing-executor> |
315 | -</module> |
316 | |
317 | === removed directory 'templates/parts' |
318 | === removed file 'templates/parts/rabbitmq' |
319 | --- templates/parts/rabbitmq 2015-06-24 12:22:08 +0000 |
320 | +++ templates/parts/rabbitmq 1970-01-01 00:00:00 +0000 |
321 | @@ -1,21 +0,0 @@ |
322 | -{% if rabbitmq_host or rabbitmq_hosts -%} |
323 | -rabbit_userid = {{ rabbitmq_user }} |
324 | -rabbit_virtual_host = {{ rabbitmq_virtual_host }} |
325 | -rabbit_password = {{ rabbitmq_password }} |
326 | -{% if rabbitmq_hosts -%} |
327 | -rabbit_hosts = {{ rabbitmq_hosts }} |
328 | -{% if rabbitmq_ha_queues -%} |
329 | -rabbit_ha_queues = True |
330 | -rabbit_durable_queues = False |
331 | -{% endif -%} |
332 | -{% else -%} |
333 | -rabbit_host = {{ rabbitmq_host }} |
334 | -{% endif -%} |
335 | -{% if rabbit_ssl_port -%} |
336 | -rabbit_use_ssl = True |
337 | -rabbit_port = {{ rabbit_ssl_port }} |
338 | -{% if rabbit_ssl_ca -%} |
339 | -kombu_ssl_ca_certs = {{ rabbit_ssl_ca }} |
340 | -{% endif -%} |
341 | -{% endif -%} |
342 | -{% endif -%} |
Approve