openstack
/
neutron
Code Issues Proposed changes

Make default nova_url use a version 

The default nova_url for neutron is missing an API
version number. This can cause requests to fail
because the Nova /versions API cannot respond
to Neutron notification requests.

It seems reasonable for the default value to
at least have a chance at being correct so
this patch upgrades the default Nova API url to
use the Nova 'v2' API.

Related-bug: #1298640
Change-Id: Ib1449de84fbc01fb704ebfe4a016ac8f4932be96
This commit is contained in:
Dan Prince 2014-04-10 12:40:13 -04:00 committed by Henry Gessau
parent d7821281e9
commit c09a14089a
2 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
etc/neutron.conf
View File

@ -303,7 +303,7 @@ notification_driver = neutron.openstack.common.notifier.rpc_notifier
# notify_nova_on_port_data_changes = True # notify_nova_on_port_data_changes = True
# URL for connection to nova (Only supports one nova region currently). # URL for connection to nova (Only supports one nova region currently).
# nova_url = http://127.0.0.1:8774 # nova_url = http://127.0.0.1:8774/v2
# Name of nova region to use. Useful if keystone manages more than one region # Name of nova region to use. Useful if keystone manages more than one region
# nova_region_name = # nova_region_name =

2
neutron/common/config.py
View File

@ -87,7 +87,7 @@ core_opts = [
help=_("Send notification to nova when port data (fixed_ips/" help=_("Send notification to nova when port data (fixed_ips/"
"floatingip) changes so nova can update its cache.")), "floatingip) changes so nova can update its cache.")),
cfg.StrOpt('nova_url', cfg.StrOpt('nova_url',
default='http://127.0.0.1:8774', default='http://127.0.0.1:8774/v2',
help=_('URL for connection to nova')), help=_('URL for connection to nova')),
cfg.StrOpt('nova_admin_username', cfg.StrOpt('nova_admin_username',
help=_('Username for connecting to nova in admin context')), help=_('Username for connecting to nova in admin context')),