VPNaaS Service driver should check whether router has gateway

Bug #1255442 reported by Bo Lin
10
This bug affects 2 people
Affects Status Importance Assigned to Milestone
neutron
Invalid
Medium
Bo Lin

Bug Description

When getting the external ip of vpnservice written in https://github.com/openstack/neutron/blob/master/neutron/services/vpn/service_drivers/ipsec.py#L172, it didn't check whether the router has gateway which would lead to the following error:
2013-11-25 19:11:49.176 TRACE neutron.openstack.common.rpc.amqp File "/opt/stack/neutron/neutron/services/vpn/service_drivers/ipsec.py", line 55, in get_vpn_services_on_host
2013-11-25 19:11:49.176 TRACE neutron.openstack.common.rpc.amqp for vpnservice in vpnservices]
2013-11-25 19:11:49.176 TRACE neutron.openstack.common.rpc.amqp
2013-11-25 19:11:49.176 TRACE neutron.openstack.common.rpc.amqp File "/opt/stack/neutron/neutron/services/vpn/service_drivers/ipsec.py", line 173, in _make_vpnservice_dict
2013-11-25 19:11:49.176 TRACE neutron.openstack.common.rpc.amqp 'fixed_ips'][0]['ip_address']
2013-11-25 19:11:49.176 TRACE neutron.openstack.common.rpc.amqp
2013-11-25 19:11:49.176 TRACE neutron.openstack.common.rpc.amqp TypeError: 'NoneType' object has no attribute '__getitem__'
2013-11-25 19:11:49.176 TRACE neutron.openstack.common.rpc.amqp
2013-11-25 19:11:49.176 TRACE neutron.openstack.common.rpc.amqp

Tags: vpnaas
Bo Lin (linb)
Changed in neutron:
assignee: nobody → berlin (linb)
Bo Lin (linb)
Changed in neutron:
assignee: berlin (linb) → nobody
description: updated
Nachi Ueno (nati-ueno)
Changed in neutron:
status: New → Confirmed
importance: Undecided → Medium
Bo Lin (linb)
Changed in neutron:
assignee: nobody → berlin (linb)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to neutron (master)

Fix proposed to branch: master
Review: https://review.openstack.org/60461

Changed in neutron:
status: Confirmed → In Progress
Revision history for this message
Bo Lin (linb) wrote :

It is the same bug with Bug #1258379 which has been resolved.

Changed in neutron:
status: In Progress → Fix Released
status: Fix Released → Invalid
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.