getting list of service providers broken with pagination

Bug #1630350 reported by Ankur
10
This bug affects 2 people
Affects Status Importance Assigned to Milestone
neutron
Invalid
High
Unassigned

Bug Description

Bug at Neutron Head as of (10/4)

While implementing service-provider-list into OpenStack Client ran into this issue with pagination.
https://review.openstack.org/#/c/381298/4/openstack/network/v2/_proxy.py

With pagination set to True.
The client returns incorrect data and breaks OpenStack Client.

Neutron Server Trace:
2016-10-04 13:35:21.217 ERROR oslo_messaging.rpc.server [req-60a4325d-a161-4a0a-
b3bf-646561a4c0fa None None] Exception during message handling
2016-10-04 13:35:21.217 TRACE oslo_messaging.rpc.server Traceback (most recent c
all last):
2016-10-04 13:35:21.217 TRACE oslo_messaging.rpc.server File "/usr/local/lib/p
ython2.7/dist-packages/oslo_messaging/rpc/server.py", line 133, in _process_inco
ming
2016-10-04 13:35:21.217 TRACE oslo_messaging.rpc.server res = self.dispatche
r.dispatch(message)
2016-10-04 13:35:21.217 TRACE oslo_messaging.rpc.server File "/usr/local/lib/python2.7/dist-packages/oslo_messaging/rpc/dispatcher.py", line 150, in dispatch
2016-10-04 13:35:21.217 TRACE oslo_messaging.rpc.server return self._do_dispatch(endpoint, method, ctxt, args)
2016-10-04 13:35:21.217 TRACE oslo_messaging.rpc.server File "/usr/local/lib/python2.7/dist-packages/oslo_messaging/rpc/dispatcher.py", line 121, in _do_dispatch
2016-10-04 13:35:21.217 TRACE oslo_messaging.rpc.server result = func(ctxt, **new_args)
2016-10-04 13:35:21.217 TRACE oslo_messaging.rpc.server File "/opt/stack/neutron/neutron/api/rpc/handlers/l3_rpc.py", line 206, in get_external_network_id
2016-10-04 13:35:21.217 TRACE oslo_messaging.rpc.server net_id = self.plugin.get_external_network_id(context)
2016-10-04 13:35:21.217 TRACE oslo_messaging.rpc.server File "/opt/stack/neutron/neutron/db/external_net_db.py", line 199, in get_external_network_id
2016-10-04 13:35:21.217 TRACE oslo_messaging.rpc.server raise n_exc.TooManyExternalNetworks()
2016-10-04 13:35:21.217 TRACE oslo_messaging.rpc.server TooManyExternalNetworks: More than one external network exists.
2016-10-04 13:35:21.217 TRACE oslo_messaging.rpc.server

When pulled Neutron back to commit: commit c7610950f75352be6693ce7da1c52e87eeaf8bc0
Client returns correct data when Pagination set to true.

Tags: api
Doug Wiegley (dougwig)
Changed in neutron:
importance: Undecided → Critical
Doug Wiegley (dougwig)
Changed in neutron:
status: New → Confirmed
assignee: nobody → Brandon Logan (brandon-logan)
Revision history for this message
Brandon Logan (brandon-logan) wrote :

I'm confused by this. Your message says with pagination is set to true it breaks, and then at the end you said the client returns correct data with Pagination set to true. Which is it? (I'm testing right now, so I'll probably figure it out)

tags: added: api
Revision history for this message
Brandon Logan (brandon-logan) wrote :

i can't seem to reproduce this. Can you do an openstack --debug and make the same request and paste the output here.

Changed in neutron:
status: Confirmed → Incomplete
Revision history for this message
Brandon Logan (brandon-logan) wrote :

Changed from critical to high because it doesn't appear to be worthy of critical.

Changed in neutron:
importance: Critical → High
Revision history for this message
Bernard Cafarelli (bcafarel) wrote :

Incomplete status for more than 2 years, closing. Feel free to reopen with asked information if still valid

Changed in neutron:
status: Incomplete → Invalid
assignee: Brandon Logan (brandon-logan) → nobody
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.