yahoo-eng-team team mailing list archive

[Matt Riedemann <mriedem@xxxxxxxxxx>]

Public bug reported:

There are two things we can do to make this more efficient:

https://github.com/openstack/nova/blob/master/nova/network/neutronv2/api.py#L1182

1. Move the list_ports call after the unlimited quota check - if the
quota is unlimited, we don't need to list ports.

2. Filter the list_ports response to only return the port id, we don't
need the other port details in the response since we don't use those
fields, we're just getting a count.

** Affects: nova
     Importance: Low
         Status: Triaged


** Tags: low-hanging-fruit network neutron performance

** Changed in: nova
   Importance: Undecided => Low

** Changed in: nova
       Status: New => Triaged

** Tags added: low-hanging-fruit

** Description changed:

- There are two things we can to make this more efficient:
+ There are two things we can do to make this more efficient:
  
  https://github.com/openstack/nova/blob/master/nova/network/neutronv2/api.py#L1182
  
  1. Move the list_ports call after the unlimited quota check - if the
  quota is unlimited, we don't need to list ports.
  
  2. Filter the list_ports response to only return the port id, we don't
  need the other port details in the response since we don't use those
  fields, we're just getting a count.

-- 
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to OpenStack Compute (nova).
https://bugs.launchpad.net/bugs/1521788

Title:
  nova.network.neutronv2.api.validate_networks could be smarter when
  listing ports

Status in OpenStack Compute (nova):
  Triaged

Bug description:
  There are two things we can do to make this more efficient:

  https://github.com/openstack/nova/blob/master/nova/network/neutronv2/api.py#L1182

  1. Move the list_ports call after the unlimited quota check - if the
  quota is unlimited, we don't need to list ports.

  2. Filter the list_ports response to only return the port id, we don't
  need the other port details in the response since we don't use those
  fields, we're just getting a count.

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