Kuryr checks wrong length of the resources

Bug #1503517 reported by Taku Fukushima
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
kuryr
Fix Committed
Undecided
Unassigned

Bug Description

Neutron returns the response for the request of listing resources as the dictionary with the key, pluralised resource name, i.e., {'networks': ...}.

However, current Kuryr checks the length of the response directly and it's 1 always because the reason above.

Changed in kuryr:
assignee: nobody → Manikanta Srinivas (manikanta777)
Revision history for this message
Taku Fukushima (tfukushima) wrote :

Hi Manikanta,

I submitted a patch for this but somehow it was not associated with this issue. I put the issue number in the commit message of the patch though.

Revision history for this message
Taku Fukushima (tfukushima) wrote :

Ooops, sorry I forgot to put the link to the patch: https://review.openstack.org/#/c/231844/

Revision history for this message
Manikanta Srinivas (manikanta777) wrote :

Ok, I am about to send the fix..!!

I will unassign the bug as the patch is already in place.

Changed in kuryr:
assignee: Manikanta Srinivas (manikanta777) → nobody
Revision history for this message
vikas choudhary (choudharyvikas16) wrote :

@Taku,

Please assign this bug to you.

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

Reviewed: https://review.openstack.org/231844
Committed: https://git.openstack.org/cgit/openstack/kuryr/commit/?id=88c10dc8970c3ec8b44452eab5d03f04751f888a
Submitter: Jenkins
Branch: master

commit 88c10dc8970c3ec8b44452eab5d03f04751f888a
Author: Taku Fukushima <email address hidden>
Date: Wed Oct 7 14:41:05 2015 +0900

    Fix Kuryr check the length of networks and subnets correctly

    This patch fixes Kuryr check the length of the resources in the
    response from Neutron appropriately, which is the dictionary with the
    key, the pluralised resource name.

    Change-Id: I59e91887543fba4f2b4d5a38b28af65b3781febb
    Signed-off-by: Taku Fukushima <email address hidden>
    Closes-Bug: #1503517

Changed in kuryr:
status: In Progress → Fix Committed
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.