)]}'
{"glance/common/utils.py":[{"author":{"_account_id":12000,"name":"Ian Cordasco","email":"sigmavirus24@gmail.com","username":"sigmavirus24"},"change_message_id":"1cbbb98869f09b871284ff332430ab845e855d18","unresolved":false,"context_lines":[{"line_number":130,"context_line":"    return readfn"},{"line_number":131,"context_line":""},{"line_number":132,"context_line":""},{"line_number":133,"context_line":"MAX_COOP_READER_BUFFER_SIZE \u003d 134217728  # 128M seems like a sane buffer limit"},{"line_number":134,"context_line":""},{"line_number":135,"context_line":""},{"line_number":136,"context_line":"class CooperativeReader(object):"}],"source_content_type":"text/x-python","patch_set":5,"id":"9a80dd14_94a87c8e","line":133,"updated":"2015-03-08 16:47:53.000000000","message":"Can we use the units module from oslo_utils to create this number perhaps? For example:\n\n    128 * units.Mi","commit_id":"305046272d5a7f30a6e53c69461658f8b936a848"}],"glance/service.py":[{"author":{"_account_id":12000,"name":"Ian Cordasco","email":"sigmavirus24@gmail.com","username":"sigmavirus24"},"change_message_id":"1cbbb98869f09b871284ff332430ab845e855d18","unresolved":false,"context_lines":[{"line_number":23,"context_line":"from oslo import i18n"},{"line_number":24,"context_line":"import oslo.messaging"},{"line_number":25,"context_line":""},{"line_number":26,"context_line":"from glance.i18n import _"},{"line_number":27,"context_line":"from glance.openstack.common import log"},{"line_number":28,"context_line":""},{"line_number":29,"context_line":"OPTS \u003d ["}],"source_content_type":"text/x-python","patch_set":5,"id":"9a80dd14_d4ae7497","line":26,"updated":"2015-03-08 16:47:53.000000000","message":"It seems you never use this.","commit_id":"305046272d5a7f30a6e53c69461658f8b936a848"},{"author":{"_account_id":12000,"name":"Ian Cordasco","email":"sigmavirus24@gmail.com","username":"sigmavirus24"},"change_message_id":"1cbbb98869f09b871284ff332430ab845e855d18","unresolved":false,"context_lines":[{"line_number":29,"context_line":"OPTS \u003d ["},{"line_number":30,"context_line":"    cfg.StrOpt(\u0027host\u0027,"},{"line_number":31,"context_line":"               default\u003dsocket.gethostname(),"},{"line_number":32,"context_line":"               help\u003d\u0027Name of this node, which must be valid in an AMQP \u0027"},{"line_number":33,"context_line":"               \u0027key. Can be an opaque identifier. For ZeroMQ only, must \u0027"},{"line_number":34,"context_line":"               \u0027be a valid host name, FQDN, or IP address.\u0027),"},{"line_number":35,"context_line":"    cfg.IntOpt(\u0027listener_workers\u0027,"}],"source_content_type":"text/x-python","patch_set":5,"id":"9a80dd14_349d1067","line":32,"updated":"2015-03-08 16:47:53.000000000","message":"I think you meant to use the imported _ here and in the other config options help.","commit_id":"305046272d5a7f30a6e53c69461658f8b936a848"},{"author":{"_account_id":12000,"name":"Ian Cordasco","email":"sigmavirus24@gmail.com","username":"sigmavirus24"},"change_message_id":"1cbbb98869f09b871284ff332430ab845e855d18","unresolved":false,"context_lines":[{"line_number":47,"context_line":"    cfg.StrOpt(\u0027os-username\u0027,"},{"line_number":48,"context_line":"               deprecated_group\u003d\"DEFAULT\","},{"line_number":49,"context_line":"               default\u003dos.environ.get(\u0027OS_USERNAME\u0027, \u0027glance\u0027),"},{"line_number":50,"context_line":"               help\u003d\u0027User name to use for OpenStack service access.\u0027),"},{"line_number":51,"context_line":"    cfg.StrOpt(\u0027os-password\u0027,"},{"line_number":52,"context_line":"               deprecated_group\u003d\"DEFAULT\","},{"line_number":53,"context_line":"               secret\u003dTrue,"}],"source_content_type":"text/x-python","patch_set":5,"id":"9a80dd14_748a281c","line":50,"updated":"2015-03-08 16:47:53.000000000","message":"Again, help\u003d_(...)","commit_id":"305046272d5a7f30a6e53c69461658f8b936a848"}]}