)]}'
{"keystoneauth1/_utils.py":[{"author":{"_account_id":2,"name":"Monty Taylor","email":"mordred@inaugust.com","username":"mordred"},"change_message_id":"4e6f8ea0d913b65c980460d0f7857fa11caae2e5","unresolved":false,"context_lines":[{"line_number":20,"context_line":""},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"def getLogger(name):"},{"line_number":23,"context_line":"    name \u003d name.replace(__name__, \u0027keystoneauth\u0027)"},{"line_number":24,"context_line":"    return logging.getLogger(name)"},{"line_number":25,"context_line":""},{"line_number":26,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"1a4dcd0f_4e020cb6","line":23,"updated":"2015-08-16 01:09:26.000000000","message":"Does this work? It seems to me like this is going to say \"please replace the string \u0027keystoneauth1.utils\u0027 with \u0027keystoneauth\u0027 which is not going to work anywhere except for keystoneauth1._utils. I think you want name.replace(__name__.split(\u0027.\u0027)[0], \u0027keystoneauth\u0027)","commit_id":"420a56af658ec9d2582773c7e739287e4035fa50"},{"author":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"change_message_id":"2d3e36d10982e280caeb33d2723ee0125e003f0d","unresolved":false,"context_lines":[{"line_number":19,"context_line":"import six"},{"line_number":20,"context_line":""},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"def getLogger(name):"},{"line_number":23,"context_line":"    name \u003d name.replace(__name__.split(\u0027.\u0027)[0], \u0027keystoneauth\u0027)"},{"line_number":24,"context_line":"    return logging.getLogger(name)"},{"line_number":25,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"fa1b9901_66d0dec9","line":22,"updated":"2015-08-20 16:52:47.000000000","message":"should we change this to follow a more pythonic convention?\n\n    def get_logger(name):","commit_id":"f85246b44d9135b952d4b44db217c0d8c50557c7"},{"author":{"_account_id":8736,"name":"Terry Howe","email":"terrylhowe@gmail.com","username":"terrylhowe"},"change_message_id":"bd06e6a6f67cede97e919aa62a1ea20a0092277e","unresolved":false,"context_lines":[{"line_number":19,"context_line":"import six"},{"line_number":20,"context_line":""},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"def getLogger(name):"},{"line_number":23,"context_line":"    name \u003d name.replace(__name__.split(\u0027.\u0027)[0], \u0027keystoneauth\u0027)"},{"line_number":24,"context_line":"    return logging.getLogger(name)"},{"line_number":25,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"fa1b9901_e6806e8d","line":22,"in_reply_to":"fa1b9901_66d0dec9","updated":"2015-08-20 16:55:22.000000000","message":"I thought about that Lance and I could go either way.  This name is consistent with the Python logger.  I assume the Python logger was written by a Java programmer.","commit_id":"f85246b44d9135b952d4b44db217c0d8c50557c7"},{"author":{"_account_id":17123,"name":"Tom Cocozzello (tjcocozz)","email":"tjcocozz@us.ibm.com","username":"tcocozzello"},"change_message_id":"51f77a4741d057bf6888d92ff7e04d54ed790299","unresolved":false,"context_lines":[{"line_number":19,"context_line":"import six"},{"line_number":20,"context_line":""},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"def getLogger(name):"},{"line_number":23,"context_line":"    name \u003d name.replace(__name__.split(\u0027.\u0027)[0], \u0027keystoneauth\u0027)"},{"line_number":24,"context_line":"    return logging.getLogger(name)"},{"line_number":25,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"fa1b9901_af66c627","line":22,"in_reply_to":"fa1b9901_e6806e8d","updated":"2015-08-20 18:54:56.000000000","message":"I agree with Lance, to keep it consistent.","commit_id":"f85246b44d9135b952d4b44db217c0d8c50557c7"}]}