rpc should return empty list when no l3_plugin is present

Bug #1514144 reported by Hong Hui Xiao
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
neutron
Fix Released
Undecided
Hong Hui Xiao

Bug Description

It is a trivial bug, maybe it is left from a history code change.

In [1], it should be a [] here, instead of {} according to the context.

[1] https://github.com/openstack/neutron/blob/master/neutron/api/rpc/handlers/l3_rpc.py#L77

Hong Hui Xiao (xiaohhui)
Changed in neutron:
assignee: nobody → Hong Hui Xiao (xiaohhui)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to neutron (master)

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

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

Reviewed: https://review.openstack.org/242797
Committed: https://git.openstack.org/cgit/openstack/neutron/commit/?id=4676c55f9d29e5d976951b857ca21e0b994e213c
Submitter: Jenkins
Branch: master

commit 4676c55f9d29e5d976951b857ca21e0b994e213c
Author: Hong Hui Xiao <email address hidden>
Date: Sun Nov 15 08:49:13 2015 -0500

    Remove the useless l3plugin check in l3_rpc.py

    Without "L3_ROUTER_NAT" in neutron's service_plugins, l3_rpc will
    fail when getting l3plugin. So, remove the useless "if" block
    here.

    Change-Id: I56f417e5723ed40c70a186394de0bfcff696e469
    Closes-Bug: #1514144

Changed in neutron:
status: In Progress → Fix Committed
Revision history for this message
Thierry Carrez (ttx) wrote : Fix included in openstack/neutron 8.0.0.0b1

This issue was fixed in the openstack/neutron 8.0.0.0b1 development milestone.

Changed in neutron:
status: Fix Committed → Fix Released
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.