)]}'
{"magnum/conductor/handlers/docker_conductor.py":[{"author":{"_account_id":11536,"name":"hongbin","email":"hongbin034@gmail.com","username":"hongbin"},"change_message_id":"816438b8acdb130e1b7a90ae3e78fa50191c9a48","unresolved":false,"context_lines":[{"line_number":80,"context_line":"                image_repo, image_tag \u003d docker_utils.parse_docker_image(image)"},{"line_number":81,"context_line":"                docker.pull(image_repo, tag\u003dimage_tag)"},{"line_number":82,"context_line":"                docker.inspect_image(self._encode_utf8(container.image))"},{"line_number":83,"context_line":"                kwargs \u003d dict(name\u003dname,"},{"line_number":84,"context_line":"                              hostname\u003dcontainer_uuid,"},{"line_number":85,"context_line":"                              command\u003dcontainer.command,"},{"line_number":86,"context_line":"                              environment\u003dcontainer.environment)"}],"source_content_type":"text/x-python","patch_set":3,"id":"1af94dfe_40af0b81","line":83,"range":{"start_line":83,"start_character":25,"end_line":83,"end_character":29},"updated":"2016-03-16 22:05:10.000000000","message":"Should use {...} instead of dict constructor. For details, see M336 [1] and this blog post [2]\n\n[1] https://github.com/openstack/magnum/blob/master/HACKING.rst\n[2] https://doughellmann.com/blog/2012/11/12/the-performance-impact-of-using-dict-instead-of-in-cpython-2-7-2/","commit_id":"0d3147297e8ea5d193870070065b220d8ea03936"},{"author":{"_account_id":12175,"name":"Eli Qiao","email":"qiaoliyong@gmail.com","username":"Eli"},"change_message_id":"007393a1a202de1a82d724b143cb860b9c435161","unresolved":false,"context_lines":[{"line_number":80,"context_line":"                image_repo, image_tag \u003d docker_utils.parse_docker_image(image)"},{"line_number":81,"context_line":"                docker.pull(image_repo, tag\u003dimage_tag)"},{"line_number":82,"context_line":"                docker.inspect_image(self._encode_utf8(container.image))"},{"line_number":83,"context_line":"                kwargs \u003d dict(name\u003dname,"},{"line_number":84,"context_line":"                              hostname\u003dcontainer_uuid,"},{"line_number":85,"context_line":"                              command\u003dcontainer.command,"},{"line_number":86,"context_line":"                              environment\u003dcontainer.environment)"}],"source_content_type":"text/x-python","patch_set":3,"id":"1af94dfe_d413d964","line":83,"range":{"start_line":83,"start_character":25,"end_line":83,"end_character":29},"in_reply_to":"1af94dfe_40af0b81","updated":"2016-03-17 01:01:53.000000000","message":"Gotcha, but not sure why M336 not work","commit_id":"0d3147297e8ea5d193870070065b220d8ea03936"},{"author":{"_account_id":11536,"name":"hongbin","email":"hongbin034@gmail.com","username":"hongbin"},"change_message_id":"7fb552d79a8682159c1f66c4f0ccb7189838f427","unresolved":false,"context_lines":[{"line_number":80,"context_line":"                image_repo, image_tag \u003d docker_utils.parse_docker_image(image)"},{"line_number":81,"context_line":"                docker.pull(image_repo, tag\u003dimage_tag)"},{"line_number":82,"context_line":"                docker.inspect_image(self._encode_utf8(container.image))"},{"line_number":83,"context_line":"                kwargs \u003d dict(name\u003dname,"},{"line_number":84,"context_line":"                              hostname\u003dcontainer_uuid,"},{"line_number":85,"context_line":"                              command\u003dcontainer.command,"},{"line_number":86,"context_line":"                              environment\u003dcontainer.environment)"}],"source_content_type":"text/x-python","patch_set":3,"id":"1af94dfe_0f27ecca","line":83,"range":{"start_line":83,"start_character":25,"end_line":83,"end_character":29},"in_reply_to":"1af94dfe_d413d964","updated":"2016-03-17 01:43:42.000000000","message":"Create a ticket for that: https://bugs.launchpad.net/magnum/+bug/1558335","commit_id":"0d3147297e8ea5d193870070065b220d8ea03936"}]}