OpenStack Compute (nova)

Do not check neutron port quota in API layer

Bug #1467728 reported by shihanzhang
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Compute (nova)
Confirmed
Low
Unassigned

Bug Description

Now neutron API does not provide reservation mechanism, so if a tenant has a large number of ports, in function validate_networks,
'list_ports' will be very expensive, and port creation depends in some cases on mac addresses only available on the compute manager.

See original description
shihanzhang (shihanzhang)
Changed in nova:
assignee: nobody → shihanzhang (shihanzhang)
OpenStack Infra (hudson-openstack)
Changed in nova:
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on nova (master)

Change abandoned by Matt Riedemann (<email address hidden>) on branch: master
Review: https://review.openstack.org/175742
Reason: This is in merge conflict and looks abandoned, please restore if you plan to continue working on this.

Davanum Srinivas (DIMS) (dims-v)
Changed in nova:
assignee: shihanzhang (shihanzhang) → nobody
status: In Progress → Confirmed
Revision history for this message
John Garbutt (johngarbutt) wrote :

So agreed with the problem, but the suggested resolution is incorrect.

Now create-port doesn't require data from the hypervisor we can call that inside the API instead of listing the ports.

It's worth nothing the list ports only returns ids now, so it shouldn't be too expensive, assuming neutron is optimising the call correctly, but it's totally still worth fixing this.

This will probably get captured into some neutron refactor work.

tags: added: neutron performance
Changed in nova:
importance: Undecided → Low
Revision history for this message
John Garbutt (johngarbutt) wrote :

Yeah this is much less bad than when this was reported.

Revision history for this message
shihanzhang (shihanzhang) wrote :

John Garbutt, thx for your comments, I agree with you, I will address the bug description!

description: updated
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.