Since https://review.openstack.org/67026 the aggregate DB APIs no longer require admin context
One implication of that is that methods like get_host_availability_zone() and get_availability_zones() no longer need to require an admin context and their callers, e.g.
def _describe_availability_zones(self, context, **kwargs):
ctxt = context.elevated()
available_zones, not_available_zones = \
availability_zones.get_availability_zones(ctxt)
no longer need to pass an elevated context
Also, in some of our scheduler filters, we do:
def _get_cpu_allocation_ratio(self, host_state, filter_properties):
context = filter_properties['context'].elevated()
metadata = db.aggregate_metadata_get_by_host(
context, host_state.host, key='cpu_allocation_ratio')
and those too should no longer require an elevated context
Fix proposed to branch: master
Review: https:/