)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":17151,"name":"chenying","email":"chenyingko@gmail.com","username":"chenying"},"change_message_id":"196d2ee6c93eb035dce65e59cfb10ee300f6c986","unresolved":false,"context_lines":[{"line_number":5,"context_line":"CommitDate: 2015-09-24 11:13:47 +0300"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"XtremIO fix formatting bug"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Change-Id: I05b6468261fe352c80f6f8e115b8e69eea551089"},{"line_number":10,"context_line":"Closes-bug: #1499207"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"ba15a1d1_907becf4","line":8,"updated":"2015-09-24 09:30:24.000000000","message":"I think you should provide a detailed description of the change.\nhttps://wiki.openstack.org/wiki/GitCommitMessages#Summary_of_Git_commit_message_structure","commit_id":"a0fb6b10df350fa929596df00a3e715113a29ec8"},{"author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"change_message_id":"c88a8b026282ebd40b6a9189e5a6ecc4b1f5e05b","unresolved":false,"context_lines":[{"line_number":5,"context_line":"CommitDate: 2015-09-24 17:29:05 +0300"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"XtremIO fix remapping bug"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Change-Id: I05b6468261fe352c80f6f8e115b8e69eea551089"},{"line_number":10,"context_line":"Closes-bug: #1499207"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"9a1a9d01_80ffdc33","line":8,"updated":"2015-09-24 14:41:31.000000000","message":"Please add a short description of the change/fix.","commit_id":"62794a256b9052729e4edf975283f6f38f0eceff"},{"author":{"_account_id":11418,"name":"Shay Halsband","email":"shay.halsband@emc.com","username":"shyh"},"change_message_id":"a1f160a367f1b142cd6631453721e7699ce60302","unresolved":false,"context_lines":[{"line_number":5,"context_line":"CommitDate: 2015-09-24 17:29:05 +0300"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"XtremIO fix remapping bug"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Change-Id: I05b6468261fe352c80f6f8e115b8e69eea551089"},{"line_number":10,"context_line":"Closes-bug: #1499207"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"9a1a9d01_34e84410","line":8,"in_reply_to":"9a1a9d01_80ffdc33","updated":"2015-09-27 06:45:57.000000000","message":"Done","commit_id":"62794a256b9052729e4edf975283f6f38f0eceff"}],"cinder/volume/drivers/emc/xtremio.py":[{"author":{"_account_id":17151,"name":"chenying","email":"chenyingko@gmail.com","username":"chenying"},"change_message_id":"196d2ee6c93eb035dce65e59cfb10ee300f6c986","unresolved":false,"context_lines":[{"line_number":530,"context_line":"            lunmap \u003d self._obj_from_result(res)"},{"line_number":531,"context_line":"            LOG.info(_LI(\u0027Created lun-map:\\n%s\u0027), lunmap)"},{"line_number":532,"context_line":"        except exception.XtremIOAlreadyMappedError:"},{"line_number":533,"context_line":"            LOG.info(_LI(\u0027Volume already mapped, retrieving %(ig)s, %(vol)s\u0027),"},{"line_number":534,"context_line":"                     {\u0027ig\u0027: ig, \u0027vol\u0027: volume[\u0027id\u0027]})"},{"line_number":535,"context_line":"            lunmap \u003d self.client.find_lunmap(ig, volume[\u0027id\u0027])"},{"line_number":536,"context_line":"        return lunmap"}],"source_content_type":"text/x-python","patch_set":1,"id":"ba15a1d1_30f6a08f","line":533,"updated":"2015-09-24 09:30:24.000000000","message":"Add a \u0027.\u0027 to the EOL.","commit_id":"a0fb6b10df350fa929596df00a3e715113a29ec8"},{"author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"change_message_id":"c88a8b026282ebd40b6a9189e5a6ecc4b1f5e05b","unresolved":false,"context_lines":[{"line_number":748,"context_line":"            self.client.req(\u0027initiators\u0027, \u0027PUT\u0027, data, idx\u003dinitiator[\u0027index\u0027])"},{"line_number":749,"context_line":""},{"line_number":750,"context_line":"        # lun mappping"},{"line_number":751,"context_line":"        lunmap \u003d self.create_lun_map(volume, ig[\u0027ig-id\u0027][1])"},{"line_number":752,"context_line":""},{"line_number":753,"context_line":"        properties \u003d self._get_iscsi_properties(lunmap)"},{"line_number":754,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"9a1a9d01_e0ed788f","line":751,"updated":"2015-09-24 14:41:31.000000000","message":"The fact that this didn\u0027t require a test change indicates that the tests aren\u0027t complete enough -- please add something that covers this path.   (Hence the bug.)","commit_id":"62794a256b9052729e4edf975283f6f38f0eceff"},{"author":{"_account_id":11418,"name":"Shay Halsband","email":"shay.halsband@emc.com","username":"shyh"},"change_message_id":"a1f160a367f1b142cd6631453721e7699ce60302","unresolved":false,"context_lines":[{"line_number":748,"context_line":"            self.client.req(\u0027initiators\u0027, \u0027PUT\u0027, data, idx\u003dinitiator[\u0027index\u0027])"},{"line_number":749,"context_line":""},{"line_number":750,"context_line":"        # lun mappping"},{"line_number":751,"context_line":"        lunmap \u003d self.create_lun_map(volume, ig[\u0027ig-id\u0027][1])"},{"line_number":752,"context_line":""},{"line_number":753,"context_line":"        properties \u003d self._get_iscsi_properties(lunmap)"},{"line_number":754,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"9a1a9d01_d1b9da7a","line":751,"in_reply_to":"9a1a9d01_e0ed788f","updated":"2015-09-27 06:45:57.000000000","message":"coverage report shows this line as covered.\nmost functionality work for both index and name, this is why a synthetic test didn\u0027t help here.\nI\u0027ll try to see if something can be improved for next version, not for a last minute bug fix.","commit_id":"62794a256b9052729e4edf975283f6f38f0eceff"}]}