)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":6491,"name":"xing-yang","email":"xingyang105@gmail.com","username":"xing-yang"},"change_message_id":"f708b5c7a9c3f8c7c5a873c2d71216fc95ec7c94","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"CoprHD: Handle duplicate display naming error"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This fix allows an openstack user to create more than 1 resource with same"},{"line_number":10,"context_line":"display names. As CoprHD currently has no support for this, we append a"},{"line_number":11,"context_line":"unique ID to the display name before sending the request to CoprHD."},{"line_number":12,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"ba5201f7_27f82949","line":9,"updated":"2017-01-04 21:58:45.000000000","message":"Please wrap at 72 chars.","commit_id":"c1b7a4fcaf87192bb50d3318d9942518b34a7b05"}],"cinder/volume/drivers/coprhd/common.py":[{"author":{"_account_id":23602,"name":"Michael Dovgal","email":"dovgalmichael@gmail.com","username":"mdovgal"},"change_message_id":"0819e03f3fce57a72285795ee425096f2565c33a","unresolved":false,"context_lines":[{"line_number":89,"context_line":"URI_VPOOL_VARRAY_CAPACITY \u003d \u0027/block/vpools/{0}/varrays/{1}/capacity\u0027"},{"line_number":90,"context_line":"URI_BLOCK_EXPORTS_FOR_INITIATORS \u003d \u0027/block/exports?initiators\u003d{0}\u0027"},{"line_number":91,"context_line":"EXPORT_RETRY_COUNT \u003d 5"},{"line_number":92,"context_line":"MAX_NAME_LENGTH \u003d 91"},{"line_number":93,"context_line":""},{"line_number":94,"context_line":""},{"line_number":95,"context_line":"def retry_wrapper(func):"}],"source_content_type":"text/x-python","patch_set":1,"id":"ba5201f7_7a3582f5","line":92,"range":{"start_line":92,"start_character":0,"end_line":92,"end_character":20},"updated":"2017-01-04 10:46:11.000000000","message":"Is this value a maximum for this storage? Or please, left comment that will define this magic number","commit_id":"c1b7a4fcaf87192bb50d3318d9942518b34a7b05"},{"author":{"_account_id":23602,"name":"Michael Dovgal","email":"dovgalmichael@gmail.com","username":"mdovgal"},"change_message_id":"0819e03f3fce57a72285795ee425096f2565c33a","unresolved":false,"context_lines":[{"line_number":1245,"context_line":"        if not name:"},{"line_number":1246,"context_line":"            name \u003d resource[\u0027name\u0027]"},{"line_number":1247,"context_line":""},{"line_number":1248,"context_line":"        if truncate_name and len(name) \u003e 31:"},{"line_number":1249,"context_line":"            name \u003d self._id_to_base64(resource.id)"},{"line_number":1250,"context_line":"            return name"},{"line_number":1251,"context_line":""},{"line_number":1252,"context_line":"        elif truncate_name:"},{"line_number":1253,"context_line":"            return name"},{"line_number":1254,"context_line":"        elif len(name) \u003e MAX_NAME_LENGTH:"},{"line_number":1255,"context_line":"            return name[0:91] + \"-\" + resource[\u0027id\u0027]"},{"line_number":1256,"context_line":"        else:"},{"line_number":1257,"context_line":"            return name + \"-\" + resource[\u0027id\u0027]"},{"line_number":1258,"context_line":""},{"line_number":1259,"context_line":"    def _get_vpool(self, volume):"},{"line_number":1260,"context_line":"        vpool \u003d {}"}],"source_content_type":"text/x-python","patch_set":1,"id":"ba5201f7_fa697226","line":1257,"range":{"start_line":1248,"start_character":0,"end_line":1257,"end_character":46},"updated":"2017-01-04 10:46:11.000000000","message":"I think it\u0027s better to left one return point from this function. especially given the fact, the only the name variable is returned","commit_id":"c1b7a4fcaf87192bb50d3318d9942518b34a7b05"},{"author":{"_account_id":6491,"name":"xing-yang","email":"xingyang105@gmail.com","username":"xing-yang"},"change_message_id":"f708b5c7a9c3f8c7c5a873c2d71216fc95ec7c94","unresolved":false,"context_lines":[{"line_number":1245,"context_line":"        if not name:"},{"line_number":1246,"context_line":"            name \u003d resource[\u0027name\u0027]"},{"line_number":1247,"context_line":""},{"line_number":1248,"context_line":"        if truncate_name and len(name) \u003e 31:"},{"line_number":1249,"context_line":"            name \u003d self._id_to_base64(resource.id)"},{"line_number":1250,"context_line":"            return name"},{"line_number":1251,"context_line":""},{"line_number":1252,"context_line":"        elif truncate_name:"},{"line_number":1253,"context_line":"            return name"},{"line_number":1254,"context_line":"        elif len(name) \u003e MAX_NAME_LENGTH:"},{"line_number":1255,"context_line":"            return name[0:91] + \"-\" + resource[\u0027id\u0027]"},{"line_number":1256,"context_line":"        else:"},{"line_number":1257,"context_line":"            return name + \"-\" + resource[\u0027id\u0027]"},{"line_number":1258,"context_line":""},{"line_number":1259,"context_line":"    def _get_vpool(self, volume):"},{"line_number":1260,"context_line":"        vpool \u003d {}"}],"source_content_type":"text/x-python","patch_set":1,"id":"ba5201f7_a7eb3977","line":1257,"range":{"start_line":1248,"start_character":0,"end_line":1257,"end_character":46},"in_reply_to":"ba5201f7_fa697226","updated":"2017-01-04 21:58:45.000000000","message":"+1","commit_id":"c1b7a4fcaf87192bb50d3318d9942518b34a7b05"}]}