OneConvergence plugin create_network fails for external networks

Bug #1299946 reported by Hemanth Ravi
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
neutron
Fix Released
Medium
Hemanth Ravi

Bug Description

create_network for networks with --router:external=True call doesn't add the network to the externalnetworks table. This is resulting in failures due to the missing entry in externalnetworks table. The reason is _process_l3_create() (called from create_network) expects to be invoked with a plugin session. Until recently process_l3_create was adding the entry to externalnetworks table even without the plugin session. Plugin third-party tests have been failing the last few days due to this.

Changed in neutron:
assignee: nobody → Hemanth Ravi (hemanth-ravi)
status: New → In Progress
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/84038

Kyle Mestery (mestery)
Changed in neutron:
importance: Undecided → Medium
Changed in neutron:
status: In Progress → Fix Committed
Revision history for this message
Oleg Bondarev (obondarev) wrote :

Hi Hemanth Ravi, please do not set status to Fix Commited until it is merged to master

Changed in neutron:
status: Fix Committed → In Progress
tags: added: icehouse-rd-potential oneconvergence
tags: added: icehouse-rc-potential
removed: icehouse-rd-potential
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to neutron (master)

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

commit d9eea51187ddd9b951e9943030724489f1c21a26
Author: Hemanth Ravi <email address hidden>
Date: Sun Mar 30 17:48:21 2014 -0700

    Invoke _process_l3_create within plugin session

    _process_l3_create should be invoked with a plugin session, else
    externalnetworks table is not populated for networks with
    router:external=True. Added the missing plugin session.

    Change-Id: I09c49dea9474f91a70aef829c1596fbb9089d6cf
    Closes-Bug: #1299946

Changed in neutron:
status: In Progress → Fix Committed
Thierry Carrez (ttx)
Changed in neutron:
milestone: none → icehouse-rc1
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in neutron:
milestone: icehouse-rc1 → 2014.1
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.