openstack/cinder
Code Issues Proposed changes

Merge "Add admin ctxt to request qos_specs in SolidFire" into stable/pike

Browse Source
This commit is contained in:
Zuul
2018-01-18 23:13:59 +00:00
committed by Gerrit Code Review
parent 91d012118f d65bf82dda
commit b79e7a4a6f
1 changed files with 7 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
cinder/volume/drivers/solidfire.py
View File

@@ -751,7 +751,13 @@ class SolidFireDriver(san.SanISCSIDriver):
# and over-ride any existing
# extra-specs settings if present
if qos_specs_id is not None:
kvs = qos_specs.get_qos_specs(ctxt, qos_specs_id)['specs']
# Policy changes require admin context to get QoS specs
# at the object layer (base:get_by_id), we can either
# explicitly promote here, or pass in a context of None
# and let the qos_specs api get an admin context for us
# personally I prefer explicit, so here ya go.
admin_ctxt = context.get_admin_context()
kvs = qos_specs.get_qos_specs(admin_ctxt, qos_specs_id)['specs']
else:
kvs = specs