NEC Plugin: All packet filters are deleted in a port deletion

Bug #1212102 reported by Ryota Mibu
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
neutron
Fix Released
Low
Akihiro Motoki

Bug Description

This is a high importance bug in packet filter extension which is in NEC Plugin.

When we deleting a port, the plugin automatically deletes packet filters associated with the port. It is intended behavior. However, the plugin also delete other packet filters. This is due to a mistake of DB filtering. We have to filter entries by "in_port" instead of "port_id".

Tags: nec
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/41842

Changed in neutron:
assignee: nobody → Ryota Mibu (r-mibu)
status: New → In Progress
Changed in neutron:
assignee: Ryota Mibu (r-mibu) → Akihiro Motoki (amotoki)
Akihiro Motoki (amotoki)
Changed in neutron:
milestone: none → havana-rc1
importance: Undecided → Low
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to neutron (master)

Reviewed: https://review.openstack.org/41842
Committed: http://github.com/openstack/neutron/commit/6ef28110c0c2b11ec63266e4d22a1b7c75cf7f5f
Submitter: Jenkins
Branch: master

commit 6ef28110c0c2b11ec63266e4d22a1b7c75cf7f5f
Author: Akihiro MOTOKI <email address hidden>
Date: Mon Sep 23 18:03:39 2013 +0900

    Fix port deletion in NEC Plugin

    Cascade on delete from ports.id to packetfitlers.in_port is added
    to ensure packet filter entries associated with a port.
    Also joined query of packetfilter with port query is added
    to avoid additional packetfilter query by port_id.

    Fixes: bug #1212102

    Change-Id: I8a67649f3361480eda6377b1d8a30bebd18aa714

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