yahoo-eng-team team mailing list archive

[Thierry Carrez <thierry.carrez+lp@xxxxxxxxx>]

** Changed in: neutron
       Status: Fix Committed => Fix Released

-- 
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to neutron.
https://bugs.launchpad.net/bugs/1248219

Title:
  ExtraRoute_db_mixin._get_extra_routes_by_router_id always returns all
  routes

Status in OpenStack Neutron (virtual network service):
  Fix Released

Bug description:
  Tonight I created a new router and attached an new subnet to it, then I tried to detach this subnet but failed. I debug the code and found ExtraRoute_db_mixin._get_extra_routes_by_router_id always returns all routes, because of the third line:
  1.    def _get_extra_routes_by_router_id(self, context, id):
  2.        query = context.session.query(RouterRoute)
  3.        query.filter(RouterRoute.router_id == id)
  4.        return self._make_extra_route_list(query)

     It should be:
            query = query.filter(RouterRoute.router_id == id)

To manage notifications about this bug go to:
https://bugs.launchpad.net/neutron/+bug/1248219/+subscriptions