]> review.fuel-infra Code Review - openstack-build/neutron-build.git/commit
Simplify ip allocation/recycling to relieve db pressure
authorCarl Baldwin <carl.baldwin@hp.com>
Mon, 18 Nov 2013 23:32:19 +0000 (23:32 +0000)
committerThomas Goirand <thomas@goirand.fr>
Thu, 13 Mar 2014 07:20:12 +0000 (15:20 +0800)
commit53f11c91772c76f9804754e438170429b9ca93fa
tree6ae80847c616cabce15f670a6de794c950c6a0e7
parentd5b7556206da0deb7c62f2334924df8631f9cf97
Simplify ip allocation/recycling to relieve db pressure

I found that multiple calls to delete_port can pile up on the
_recycle_ip operation.  This patch simplifies this operation.  It
reduces the _recycle_ip operation to a single row delete in the ip
allocations table and doesn't touch the availability table.

To acheive the recycling of ips in a pool, this code runs a more
complex operation of rebuilding the availability table when it is
exhausted.  Only one API process will perform this more expensive
operation and others waiting for allocation will immediately benefit.
The amortized cost of this operation is much less than the cumulative
cost of running the more expensive _recycle_ip operation for every
port delete.

IP allocation behaves a bit differently with this patch.  Instead of
giving out the first IP available in a pool, the entire pool will be
allocated before wrapping around and recycling ip addresses that have
been released.  This is a desirable feature as it puts ip addresses in
a sort of quarantine after they are released.  It is easier to
distinguish newly allocated ips from old ones.

Change-Id: Ia55b66128de9986e075b0f87acc401d211cd91d3
Closes-Bug: #1252506
Closes-Bug: #1257815
neutron/db/db_base_plugin_v2.py
neutron/db/models_v2.py
neutron/tests/unit/nicira/test_networkgw.py
neutron/tests/unit/test_db_plugin.py