yahoo-eng-team team mailing list archive
-
yahoo-eng-team team
-
Mailing list archive
-
Message #50723
[Bug 1580621] [NEW] Avoid looking up a source security group twice when creating new rules
Public bug reported:
When passing a source group_id for creating a new security group rule,
the code is looking up that group twice, once here:
https://github.com/openstack/nova/blob/f8a01ccdffc13403df77148867ef3821100b5edb/nova/api/openstack/compute/security_groups.py#L246
->
https://github.com/openstack/nova/blob/f8a01ccdffc13403df77148867ef3821100b5edb/nova/api/openstack/compute/security_groups.py#L299
And then once again before creating the rule:
https://github.com/openstack/nova/blob/f8a01ccdffc13403df77148867ef3821100b5edb/nova/api/openstack/compute/security_groups.py#L272
We should just look that up once at the beginning and reuse it so we
don't have to hit the nova db (nova-net) or neutron twice for the same
thing.
This is a low severity performance optimization.
** Affects: nova
Importance: Low
Status: Triaged
** Tags: perf security-groups
--
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/1580621
Title:
Avoid looking up a source security group twice when creating new rules
Status in OpenStack Compute (nova):
Triaged
Bug description:
When passing a source group_id for creating a new security group rule,
the code is looking up that group twice, once here:
https://github.com/openstack/nova/blob/f8a01ccdffc13403df77148867ef3821100b5edb/nova/api/openstack/compute/security_groups.py#L246
->
https://github.com/openstack/nova/blob/f8a01ccdffc13403df77148867ef3821100b5edb/nova/api/openstack/compute/security_groups.py#L299
And then once again before creating the rule:
https://github.com/openstack/nova/blob/f8a01ccdffc13403df77148867ef3821100b5edb/nova/api/openstack/compute/security_groups.py#L272
We should just look that up once at the beginning and reuse it so we
don't have to hit the nova db (nova-net) or neutron twice for the same
thing.
This is a low severity performance optimization.
To manage notifications about this bug go to:
https://bugs.launchpad.net/nova/+bug/1580621/+subscriptions
Follow ups