)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":17405,"name":"Jacob Gregor","email":"jgregor@us.ibm.com","username":"jgregor"},"change_message_id":"1171b2c6682ff9257035824f923aa39f249692fd","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     zhangsong \u003czhangsong@cmss.chinamobile.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2015-11-13 17:26:19 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Improve imageutils.QemuImgInfo class to parse image info more exactly"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"The current imageutils.QemuImgInfo class parse all the image info"},{"line_number":10,"context_line":"at first, but if one of the info is unavailable, it would raise a"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"da85f559_b878c690","line":7,"updated":"2015-11-13 20:02:53.000000000","message":"First line should be limited to 50 characters.\n\nhttps://wiki.openstack.org/wiki/GitCommitMessages#Summary_of_Git_commit_message_structure","commit_id":"02e076671002be308ed4da72baa592a78ea54406"},{"author":{"_account_id":17405,"name":"Jacob Gregor","email":"jgregor@us.ibm.com","username":"jgregor"},"change_message_id":"1171b2c6682ff9257035824f923aa39f249692fd","unresolved":false,"context_lines":[{"line_number":14,"context_line":"available, but it may be failed because of the ValueError which is"},{"line_number":15,"context_line":"caused by another unavailable info."},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"This patch resolved the problem by parse each of the info separately,"},{"line_number":18,"context_line":"so that getting the info which is cared would not be influenced by"},{"line_number":19,"context_line":"other unavailable info."},{"line_number":20,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"da85f559_b89f66b3","line":17,"updated":"2015-11-13 20:02:53.000000000","message":"resolved -\u003e resolves\n\nparse -\u003e parsing","commit_id":"02e076671002be308ed4da72baa592a78ea54406"},{"author":{"_account_id":17405,"name":"Jacob Gregor","email":"jgregor@us.ibm.com","username":"jgregor"},"change_message_id":"1171b2c6682ff9257035824f923aa39f249692fd","unresolved":false,"context_lines":[{"line_number":15,"context_line":"caused by another unavailable info."},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"This patch resolved the problem by parse each of the info separately,"},{"line_number":18,"context_line":"so that getting the info which is cared would not be influenced by"},{"line_number":19,"context_line":"other unavailable info."},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"For more information, please visit the bug reporting message."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"da85f559_389376a5","line":18,"updated":"2015-11-13 20:02:53.000000000","message":"Suggestion: Change \u0027getting the info which is cared would not be ...\u0027 -\u003e \u0027getting the desired info will not be ...\u0027","commit_id":"02e076671002be308ed4da72baa592a78ea54406"}],"cinder/openstack/common/imageutils.py":[{"author":{"_account_id":17405,"name":"Jacob Gregor","email":"jgregor@us.ibm.com","username":"jgregor"},"change_message_id":"1171b2c6682ff9257035824f923aa39f249692fd","unresolved":false,"context_lines":[{"line_number":144,"context_line":"                    details \u003d self._extract_details(root, root_details, lines)"},{"line_number":145,"context_line":"                    contents[root] \u003d details"},{"line_number":146,"context_line":"                except ValueError:"},{"line_number":147,"context_line":"                    contents[root] \u003d \u0027unknow\u0027"},{"line_number":148,"context_line":"        return contents"},{"line_number":149,"context_line":""},{"line_number":150,"context_line":"    # The available key should be one of the following:"}],"source_content_type":"text/x-python","patch_set":2,"id":"da85f559_5808ba28","line":147,"updated":"2015-11-13 20:02:53.000000000","message":"unknow -\u003e unknown?","commit_id":"02e076671002be308ed4da72baa592a78ea54406"},{"author":{"_account_id":17405,"name":"Jacob Gregor","email":"jgregor@us.ibm.com","username":"jgregor"},"change_message_id":"1171b2c6682ff9257035824f923aa39f249692fd","unresolved":false,"context_lines":[{"line_number":155,"context_line":"            value \u003d self.details.get(\u0027snapshot_list\u0027, [])"},{"line_number":156,"context_line":"        else:"},{"line_number":157,"context_line":"            value \u003d self.details.get(key)"},{"line_number":158,"context_line":"        if value \u003d\u003d \u0027unknow\u0027:"},{"line_number":159,"context_line":"            msg \u003d _(\"Can\u0027t get %s info of image %s!\" % (key, self.image))"},{"line_number":160,"context_line":"            raise ValueError(msg)"},{"line_number":161,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"da85f559_1813125e","line":158,"updated":"2015-11-13 20:02:53.000000000","message":"unknow -\u003e unknown?","commit_id":"02e076671002be308ed4da72baa592a78ea54406"},{"author":{"_account_id":17405,"name":"Jacob Gregor","email":"jgregor@us.ibm.com","username":"jgregor"},"change_message_id":"1171b2c6682ff9257035824f923aa39f249692fd","unresolved":false,"context_lines":[{"line_number":157,"context_line":"            value \u003d self.details.get(key)"},{"line_number":158,"context_line":"        if value \u003d\u003d \u0027unknow\u0027:"},{"line_number":159,"context_line":"            msg \u003d _(\"Can\u0027t get %s info of image %s!\" % (key, self.image))"},{"line_number":160,"context_line":"            raise ValueError(msg)"},{"line_number":161,"context_line":""},{"line_number":162,"context_line":"        return value"}],"source_content_type":"text/x-python","patch_set":2,"id":"da85f559_d8bd8a03","line":160,"updated":"2015-11-13 20:02:53.000000000","message":"I am wondering if we need to update or add a unit test to make sure we are able to get this error.","commit_id":"02e076671002be308ed4da72baa592a78ea54406"},{"author":{"_account_id":8358,"name":"zhangguoqing","email":"zhang.guoqing@99cloud.net","username":"zhangguoqing"},"change_message_id":"875ee2501ffeccb7a29cc4735cef536253a13b90","unresolved":false,"context_lines":[{"line_number":93,"context_line":"                real_details \u003d backing_match.group(2).strip()"},{"line_number":94,"context_line":"        elif root_cmd in [\u0027virtual_size\u0027, \u0027cluster_size\u0027, \u0027disk_size\u0027]:"},{"line_number":95,"context_line":"            # Replace it with the byte amount (if we can convert it)"},{"line_number":96,"context_line":"            if root_details \u003d\u003d \u0027None\u0027 or root_details \u003d\u003d \"unavailable\":"},{"line_number":97,"context_line":"                real_details \u003d 0"},{"line_number":98,"context_line":"            else:"},{"line_number":99,"context_line":"                real_details \u003d self._extract_bytes(root_details)"}],"source_content_type":"text/x-python","patch_set":5,"id":"ba8a016a_3274e59a","line":96,"updated":"2015-11-23 10:28:31.000000000","message":"Why not root_details \u003d\u003d \u0027unavailable\u0027 ?","commit_id":"c48985ff257982cb0b7cfdf47e12a091ed9e7586"},{"author":{"_account_id":18967,"name":"zhangsong","email":"zhangsong@cmss.chinamobile.com","username":"zhangsong"},"change_message_id":"c8372b94cef9a8067cc6a650cd44bbe1b008e819","unresolved":false,"context_lines":[{"line_number":93,"context_line":"                real_details \u003d backing_match.group(2).strip()"},{"line_number":94,"context_line":"        elif root_cmd in [\u0027virtual_size\u0027, \u0027cluster_size\u0027, \u0027disk_size\u0027]:"},{"line_number":95,"context_line":"            # Replace it with the byte amount (if we can convert it)"},{"line_number":96,"context_line":"            if root_details \u003d\u003d \u0027None\u0027 or root_details \u003d\u003d \"unavailable\":"},{"line_number":97,"context_line":"                real_details \u003d 0"},{"line_number":98,"context_line":"            else:"},{"line_number":99,"context_line":"                real_details \u003d self._extract_bytes(root_details)"}],"source_content_type":"text/x-python","patch_set":5,"id":"ba8a016a_f5edd58e","line":96,"in_reply_to":"ba8a016a_3274e59a","updated":"2015-11-24 02:43:34.000000000","message":"Why the previous code use root_details \u003d\u003d \u0027None\u0027? Maybe some else type of image has \u0027None\u0027 size? I\u0027m not sure.","commit_id":"c48985ff257982cb0b7cfdf47e12a091ed9e7586"},{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"0870400157a64e8e557ceb35d9b6cf81613889d9","unresolved":false,"context_lines":[{"line_number":93,"context_line":"                real_details \u003d backing_match.group(2).strip()"},{"line_number":94,"context_line":"        elif root_cmd in [\u0027virtual_size\u0027, \u0027cluster_size\u0027, \u0027disk_size\u0027]:"},{"line_number":95,"context_line":"            # Replace it with the byte amount (if we can convert it)"},{"line_number":96,"context_line":"            if root_details \u003d\u003d \u0027None\u0027 or root_details \u003d\u003d \"unavailable\":"},{"line_number":97,"context_line":"                real_details \u003d 0"},{"line_number":98,"context_line":"            else:"},{"line_number":99,"context_line":"                real_details \u003d self._extract_bytes(root_details)"}],"source_content_type":"text/x-python","patch_set":5,"id":"9a8ffd7b_bdb9387b","line":96,"in_reply_to":"ba8a016a_f5edd58e","updated":"2015-11-26 16:09:32.000000000","message":"-1: To make it more readable please use in:\n\n    if root_details in (\u0027None\u0027, \u0027unavailable\u0027):","commit_id":"c48985ff257982cb0b7cfdf47e12a091ed9e7586"}]}