Merge lp:~ivoks/charms/precise/nova-cloud-controller/fix-neutron-url into lp:~charmers/charms/precise/nova-cloud-controller/trunk

Proposed by Ante Karamatić
Status: Merged
Merged at revision: 60
Proposed branch: lp:~ivoks/charms/precise/nova-cloud-controller/fix-neutron-url
Merge into: lp:~charmers/charms/precise/nova-cloud-controller/trunk
Diff against target: 19 lines (+0/-2)
1 file modified
templates/folsom/nova.conf (+0/-2)
To merge this branch: bzr merge lp:~ivoks/charms/precise/nova-cloud-controller/fix-neutron-url
Reviewer Review Type Date Requested Status
charmers Pending
Review via email: mp+204488@code.launchpad.net

Description of the change

In previous commit, we tried define neutron_url, but used wrong template variables; those for keystone. By removing the neutron_url, we'll use default value (localhost:9696), which is fine since neutron/quantum server is running on the same host.

To post a comment you must log in.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'templates/folsom/nova.conf'
2--- templates/folsom/nova.conf 2014-01-31 16:26:01 +0000
3+++ templates/folsom/nova.conf 2014-02-03 13:45:33 +0000
4@@ -74,7 +74,6 @@
5 {% if network_manager and network_manager == 'quantum' -%}
6 network_api_class = nova.network.quantumv2.api.API
7 {% if auth_host -%}
8-quantum_url = {{ service_protocol }}://{{ service_host }}:{{ service_port }}
9 quantum_auth_strategy = keystone
10 quantum_admin_tenant_name = {{ admin_tenant_name }}
11 quantum_admin_username = {{ admin_user }}
12@@ -84,7 +83,6 @@
13 {% elif network_manager and network_manager == 'neutron' -%}
14 network_api_class = nova.network.neutronv2.api.API
15 {% if auth_host -%}
16-neutron_url = {{ service_protocol }}://{{ service_host }}:{{ service_port }}
17 neutron_auth_strategy = keystone
18 neutron_admin_tenant_name = {{ admin_tenant_name }}
19 neutron_admin_username = {{ admin_user }}

Subscribers

People subscribed via source and target branches