)]}'
{"ceilometer/service.py":[{"author":{"_account_id":6537,"name":"gordon chung","email":"gord@live.ca","username":"chungg"},"change_message_id":"ce3df16855957d96552ec24231c7f96e14a97d57","unresolved":false,"context_lines":[{"line_number":147,"context_line":"    if argv is None:"},{"line_number":148,"context_line":"        argv \u003d sys.argv"},{"line_number":149,"context_line":"    cfg.CONF(argv[1:], project\u003d\u0027ceilometer\u0027)"},{"line_number":150,"context_line":"    if cfg.CONF.database_connection:"},{"line_number":151,"context_line":"        cfg.CONF.set_override(\u0027connection\u0027, cfg.CONF.database_connection,"},{"line_number":152,"context_line":"                              group\u003d\u0027database\u0027)"},{"line_number":153,"context_line":"    log.setup(\u0027ceilometer\u0027)"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAXH%2F%2F4Ks%3D","line":150,"updated":"2014-04-16 18:31:15.000000000","message":"it is somewhat odd we\u0027re setting this for every service, even if its unrelated...\n\nalso we\u0027d need to have an import_opt to ensure we pull in option... or move that option from storage.__init__","commit_id":"13629414895a10c28d5a1d2afc75570fa83b2173"},{"author":{"_account_id":6537,"name":"gordon chung","email":"gord@live.ca","username":"chungg"},"change_message_id":"9313379d91406a3d493c0c11fc06a2bb21aaf235","unresolved":false,"context_lines":[{"line_number":148,"context_line":"        argv \u003d sys.argv"},{"line_number":149,"context_line":"    cfg.CONF(argv[1:], project\u003d\u0027ceilometer\u0027)"},{"line_number":150,"context_line":"    if cfg.CONF.database_connection:"},{"line_number":151,"context_line":"        cfg.CONF.import_opt(\u0027connection\u0027,"},{"line_number":152,"context_line":"                            \u0027ceilometer.openstack.common.db.options\u0027,"},{"line_number":153,"context_line":"                            group\u003d\u0027database\u0027)"},{"line_number":154,"context_line":"        cfg.CONF.set_override(\u0027connection\u0027, cfg.CONF.database_connection,"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAXH%2F%2FAHU%3D","line":151,"updated":"2014-04-24 18:38:35.000000000","message":"should be importing database_connection opt not below... not this one.","commit_id":"87bd76c0ce5cca46c0f50d997c51eea66841c715"}],"ceilometer/storage/__init__.py":[{"author":{"_account_id":6537,"name":"gordon chung","email":"gord@live.ca","username":"chungg"},"change_message_id":"9313379d91406a3d493c0c11fc06a2bb21aaf235","unresolved":false,"context_lines":[{"line_number":32,"context_line":"LOG \u003d log.getLogger(__name__)"},{"line_number":33,"context_line":""},{"line_number":34,"context_line":"STORAGE_ENGINE_NAMESPACE \u003d \u0027ceilometer.storage\u0027"},{"line_number":35,"context_line":""},{"line_number":36,"context_line":"OLD_STORAGE_OPTS \u003d ["},{"line_number":37,"context_line":"    cfg.StrOpt(\u0027database_connection\u0027,"},{"line_number":38,"context_line":"               secret\u003dTrue,"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAXX%2F%2F%2F%2B0%3D","line":35,"updated":"2014-04-24 18:38:35.000000000","message":"used only in service.py now","commit_id":"87bd76c0ce5cca46c0f50d997c51eea66841c715"},{"author":{"_account_id":6537,"name":"gordon chung","email":"gord@live.ca","username":"chungg"},"change_message_id":"9313379d91406a3d493c0c11fc06a2bb21aaf235","unresolved":false,"context_lines":[{"line_number":52,"context_line":"]"},{"line_number":53,"context_line":""},{"line_number":54,"context_line":"cfg.CONF.register_opts(STORAGE_OPTS, group\u003d\u0027database\u0027)"},{"line_number":55,"context_line":""},{"line_number":56,"context_line":"cfg.CONF.import_opt(\u0027connection\u0027,"},{"line_number":57,"context_line":"                    \u0027ceilometer.openstack.common.db.options\u0027,"},{"line_number":58,"context_line":"                    group\u003d\u0027database\u0027)"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAXX%2F%2F%2F%2FM%3D","line":55,"updated":"2014-04-24 18:38:35.000000000","message":"not used anymore","commit_id":"87bd76c0ce5cca46c0f50d997c51eea66841c715"},{"author":{"_account_id":6537,"name":"gordon chung","email":"gord@live.ca","username":"chungg"},"change_message_id":"8f70cd9288c839e96779cb175825b52c2f57df86","unresolved":false,"context_lines":[{"line_number":77,"context_line":"def get_connection(url):"},{"line_number":78,"context_line":"    \"\"\"Return an open connection to the database.\"\"\""},{"line_number":79,"context_line":"    engine_name \u003d urlparse.urlparse(url).scheme"},{"line_number":80,"context_line":"    \"\"\"Load the configured engine and return an instance.\"\"\""},{"line_number":81,"context_line":"    LOG.debug(_(\u0027looking for %(name)r driver in %(namespace)r\u0027) % ("},{"line_number":82,"context_line":"              {\u0027name\u0027: engine_name,"},{"line_number":83,"context_line":"               \u0027namespace\u0027: STORAGE_ENGINE_NAMESPACE}))"}],"source_content_type":"text/x-python","patch_set":4,"id":"bad4c104_52e232a0","line":80,"updated":"2014-04-29 14:23:40.000000000","message":"two docstrings look odd","commit_id":"200cf95119cf1655e51ec330b47bcef50124e9ab"}]}