]> review.fuel-infra Code Review - openstack-build/neutron-build.git/commit
nec plugin: Avoid long transaction in delete_ports
authorAkihiro Motoki <motoki@da.jp.nec.com>
Fri, 21 Feb 2014 08:42:46 +0000 (17:42 +0900)
committerGerrit Code Review <review@openstack.org>
Sun, 2 Mar 2014 09:04:13 +0000 (09:04 +0000)
commit64acc3bd63846a6e7da8d1136f946372c698cb76
tree76091d276702db2ac7ab0ae68d962845540221db
parent2964b8483a7a8c2708c2bfe6baadac633b91582d
nec plugin: Avoid long transaction in delete_ports

db_plugin.delete_ports() can lead to long transaction
if plugin.deleete_port talks with external system.
This commit removes a transaction in delete_ports and
allows NEC plugin to use more granular db transactions
in delete_port. It greatly helps db race conditions and
timeouts in delete_port operations.

To avoid to impact other plugins/drivers by changing
db_plugin.delete_ports directly and to land this patch soon,
this commit overrides delete_ports() in NEC plugin.
Further disssion on transaction in delete_ports will be
discussed under bug 1282925.

Closes-Bug: #1282922
Related-Bug: #1282925

Change-Id: I2c00694ad34eb2058bf7a0ff1c920ceded327d43
neutron/plugins/nec/nec_plugin.py
neutron/tests/unit/nec/test_nec_plugin.py