]> review.fuel-infra Code Review - openstack-build/neutron-build.git/commit
Router is not unscheduled when the last port is deleted
authorStephen Ma <stephen.ma@hp.com>
Tue, 24 Feb 2015 23:31:33 +0000 (23:31 +0000)
committerstephen-ma <stephen.ma@hp.com>
Mon, 27 Apr 2015 15:36:10 +0000 (15:36 +0000)
commit1813da49aded224e273e0a33a90dca902fa05b75
tree1aec877506c461e8e440384289b98f6f66ad084f
parent9e113360b9ab55793a1b5d8fc0cb3e428e63c044
Router is not unscheduled when the last port is deleted

When checking for ports that are still in use on a DVR router,
the L3 agent scheduler makes the assumption that a port's
network must be owned by the same tenant. This isn't always
true as the admin could have created a shared network that
other tenants may use. The result of this assumption is that
the router associated with the shared network may not be
unscheduled from a VM host when the last VM (created by a
non-admin tenant) using the shared network is deleted from
the compute node.

The owner of a VM may not own all the ports of a shared
network.  Other tenants may have VMs using the same shared
network running on the same compute node. Also the VM owner
may not own the router ports. In order to check whether a
router can be unscheduled from a node has to be run with
admin context so all the ports associated with router are
returned from database queries.

This patch fixes this problem by using the admin context to
make the queries needed for the DVR scheduler to make the
correct unschedule decision.

Change-Id: I45477713d7ce16f2451fa6fbe04c610388b06867
Closes-bug: #1424096
(cherry picked from commit edbade486102a219810137d1c6b916e87475d477)
neutron/db/l3_dvrscheduler_db.py
neutron/tests/unit/scheduler/test_l3_agent_scheduler.py