openstack/placement
Code Issues Proposed changes

Merge "Reduce the number of db/rpc calls to get instance rules"

Browse Source
This commit is contained in:
Jenkins 2015-12-30 00:07:27 +00:00 committed by Gerrit Code Review
commit af2d79bafa
2 changed files with 14 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
nova/db/api.py
View File

@ -1329,6 +1329,11 @@ def security_group_rule_get_by_security_group(context, security_group_id,
context, security_group_id, columns_to_join=columns_to_join) context, security_group_id, columns_to_join=columns_to_join)
def security_group_rule_get_by_instance(context, instance_uuid):
"""Get all rules for a given instance."""
return IMPL.security_group_rule_get_by_instance(context, instance_uuid)
def security_group_rule_destroy(context, security_group_rule_id): def security_group_rule_destroy(context, security_group_rule_id):
"""Deletes a security group rule.""" """Deletes a security group rule."""
return IMPL.security_group_rule_destroy(context, security_group_rule_id) return IMPL.security_group_rule_destroy(context, security_group_rule_id)

9
nova/db/sqlalchemy/api.py
View File

@ -4356,6 +4356,15 @@ def security_group_rule_get_by_security_group(context, security_group_id,
return query.all() return query.all()
@require_context
def security_group_rule_get_by_instance(context, instance_uuid):
return (_security_group_rule_get_query(context).
join('parent_group', 'instances').
filter_by(uuid=instance_uuid).
options(joinedload('grantee_group')).
all())
@require_context @require_context
def security_group_rule_create(context, values): def security_group_rule_create(context, values):
return _security_group_rule_create(context, values) return _security_group_rule_create(context, values)