Some database accesses in NEC plugin do not support subtransaction

Bug #1133253 reported by Akihiro Motoki
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
neutron
Fix Released
Medium
Akihiro Motoki

Bug Description

Some database accesses in NEC plugin did not support subtransaction. There are three type below.

(a) subtransactions=True is not specified.
quantum/plugins/nec/db/nec_plugin_base.py: update_packet_filter()
        with context.session.begin():
            pf_entry = nmodels.PacketFilter(**params)
            context.session.add(pf_entry)

(b) transaction commit is not ensured.
quantum/plugins/nec/nec_plugin.py: update_packet_filter()
        old_pf = super(NECPluginV2, self).get_packet_filter(context, id)
        new_pf = super(NECPluginV2, self).update_packet_filter(context, id,
                                                               packet_filter)

(c) using session.flush(). It works but with session.begin(subtransactions=True) is recommended.
quantum/plugins/nec/db/api.py: add_ofc_item()
        item = model(**params)
        session.add(item)
        session.flush()

Tags: nec
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to quantum (master)

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

Changed in quantum:
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to quantum (master)

Reviewed: https://review.openstack.org/22979
Committed: http://github.com/openstack/quantum/commit/9f6adb4b26e5b79a214cf3a29a4ccab4510df6d9
Submitter: Jenkins
Branch: master

commit 9f6adb4b26e5b79a214cf3a29a4ccab4510df6d9
Author: Akihiro MOTOKI <email address hidden>
Date: Tue Feb 26 16:43:03 2013 +0900

    Make sure all db accesses use subtransaction

    Fixes bug 1133253

    Change-Id: If952eaa919cdc7536023525a397e7a8ce4b425aa

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