)]}'
{"cinder/quota_utils.py":[{"author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"change_message_id":"18a2736cb908f35714b87c62dbe5e28c105d2633","unresolved":false,"context_lines":[{"line_number":28,"context_line":""},{"line_number":29,"context_line":"CONF \u003d cfg.CONF"},{"line_number":30,"context_line":"CONF.import_opt(\u0027auth_uri\u0027, \u0027keystonemiddleware.auth_token.__init__\u0027,"},{"line_number":31,"context_line":"                \u0027keystone_authtoken\u0027)"},{"line_number":32,"context_line":""},{"line_number":33,"context_line":"LOG \u003d logging.getLogger(__name__)"},{"line_number":34,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"7af24918_0ffec036","line":31,"updated":"2016-03-02 19:21:09.000000000","message":"Does this mean that this file also needs to import keystonemiddleware, so we\u0027re sure this module exists when this CONF import is run?\n\nI\u0027m not sure but I think this may currently be relying on the assumption that something before this point happened to import keystonemiddleware.","commit_id":"ac5717bc4fdf1241aa6408d4c0e91e63b6f27b6d"},{"author":{"_account_id":8912,"name":"Ryan McNair","email":"rdmcnair@us.ibm.com","username":"rdmcnair"},"change_message_id":"4faeff38504fe7c28bed89183fcbaae31a865592","unresolved":false,"context_lines":[{"line_number":28,"context_line":""},{"line_number":29,"context_line":"CONF \u003d cfg.CONF"},{"line_number":30,"context_line":"CONF.import_opt(\u0027auth_uri\u0027, \u0027keystonemiddleware.auth_token.__init__\u0027,"},{"line_number":31,"context_line":"                \u0027keystone_authtoken\u0027)"},{"line_number":32,"context_line":""},{"line_number":33,"context_line":"LOG \u003d logging.getLogger(__name__)"},{"line_number":34,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"7af24918_d242bf3d","line":31,"in_reply_to":"7af24918_0ffec036","updated":"2016-03-02 19:25:01.000000000","message":"I don\u0027t have a great answer to the question.  The only thing that we\u0027re using the keystonemiddleware for here is https://github.com/openstack/cinder/blob/master/cinder/quota_utils.py#L236.  Just need to grab that config option.\n\nI\u0027m pretty sure I tested it out before to validate this works for the c-vol case now, and it did, but I will double check that.","commit_id":"ac5717bc4fdf1241aa6408d4c0e91e63b6f27b6d"},{"author":{"_account_id":8912,"name":"Ryan McNair","email":"rdmcnair@us.ibm.com","username":"rdmcnair"},"change_message_id":"b324463432f42fb719eb7eec013bee512b5f3a9a","unresolved":false,"context_lines":[{"line_number":28,"context_line":""},{"line_number":29,"context_line":"CONF \u003d cfg.CONF"},{"line_number":30,"context_line":"CONF.import_opt(\u0027auth_uri\u0027, \u0027keystonemiddleware.auth_token.__init__\u0027,"},{"line_number":31,"context_line":"                \u0027keystone_authtoken\u0027)"},{"line_number":32,"context_line":""},{"line_number":33,"context_line":"LOG \u003d logging.getLogger(__name__)"},{"line_number":34,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"7af24918_1bd6b7f0","line":31,"in_reply_to":"7af24918_d242bf3d","updated":"2016-03-03 19:39:36.000000000","message":"Verified this works. Looks like import_opt will import the module for us - https://github.com/openstack/oslo.config/blob/master/oslo_config/cfg.py#L2353.","commit_id":"ac5717bc4fdf1241aa6408d4c0e91e63b6f27b6d"}]}