VPNaaS: Cisco validator incorrectly checking public IP of router

Bug #1356127 reported by Paul Michali
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
neutron
Fix Released
Medium
Paul Michali

Bug Description

The IPSec site connection validation checks that the router has a GW specified. It incorrectly accesses this information via the VPN service table.

As a result, all validations fail.

Tags: vpnaas cisco
Paul Michali (pcm)
Changed in neutron:
assignee: nobody → Paul Michali (pcm)
status: New → In Progress
description: updated
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/113751

Changed in neutron:
importance: Undecided → Medium
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to neutron (master)

Reviewed: https://review.openstack.org/113751
Committed: https://git.openstack.org/cgit/openstack/neutron/commit/?id=7370b3df04a9f5dddb3bddc9bf6f8b1f3c205d72
Submitter: Jenkins
Branch: master

commit 7370b3df04a9f5dddb3bddc9bf6f8b1f3c205d72
Author: Paul Michali <email address hidden>
Date: Tue Aug 12 21:26:27 2014 -0400

    VPNaaS: Cisco fix validation for GW IP

    The validation to check that the router has a GW IP
    for the Cisco service driver was trying to directly
    get the gw_port information, which is not available, as
    the validator only has a dict of the vpn_service info.

    Modified validator to get the router object (not a
    dict representation), using the vpn_service's router_id
    and then check the attributes for the router directly.

    Change-Id: I48286cda23d9737ddd90251dc0d1db1c310e7784
    Closes-Bug: 1356127

Changed in neutron:
status: In Progress → Fix Committed
Thierry Carrez (ttx)
Changed in neutron:
milestone: none → juno-3
status: Fix Committed → Fix Released
Paul Michali (pcm)
tags: added: vpnaas
removed: vnaas
Thierry Carrez (ttx)
Changed in neutron:
milestone: juno-3 → 2014.2
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.