)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"ac953d1cfa809442fc1c0e6bafabd51bba0da213","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Tegile driver doesn\u0027t honor create from snapshot volume size"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Create_volume_from_snapshot can specify a volume size larger than"},{"line_number":10,"context_line":"the orginal snapshot. Tegile driver not appear to handle this size"},{"line_number":11,"context_line":"difference. This patch fixes it."},{"line_number":12,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"3aaa91ec_11c5988d","line":9,"range":{"start_line":9,"start_character":0,"end_line":9,"end_character":1},"updated":"2016-06-22 11:47:00.000000000","message":"Method name is all lower case: create_volume_from_snapshot","commit_id":"dfff3023c7840d48b27e367096009ac93c7eab41"},{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"ac953d1cfa809442fc1c0e6bafabd51bba0da213","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Tegile driver doesn\u0027t honor create from snapshot volume size"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Create_volume_from_snapshot can specify a volume size larger than"},{"line_number":10,"context_line":"the orginal snapshot. Tegile driver not appear to handle this size"},{"line_number":11,"context_line":"difference. This patch fixes it."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Change-Id: Id8cc82ec9d1565f7cbd38684f6d1a712cfba1cd6"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"3aaa91ec_31bc1405","line":10,"range":{"start_line":10,"start_character":36,"end_line":10,"end_character":50},"updated":"2016-06-22 11:47:00.000000000","message":"doesn\u0027t","commit_id":"dfff3023c7840d48b27e367096009ac93c7eab41"},{"author":{"_account_id":10058,"name":"Erlon R. Cruz","email":"erlon.rodrigues.cruz@canonical.com","username":"sombrafam"},"change_message_id":"4ccd8f70051d60d932e78cc2c3e07191c7c0ebd1","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Tegile driver doesn\u0027t honor create from snapshot volume size"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Create_volume_from_snapshot can specify a volume size larger than"},{"line_number":10,"context_line":"the orginal snapshot. Tegile driver not appear to handle this size"},{"line_number":11,"context_line":"difference. This patch fixes it."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Change-Id: Id8cc82ec9d1565f7cbd38684f6d1a712cfba1cd6"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"3aaa91ec_d07811f5","line":10,"range":{"start_line":10,"start_character":4,"end_line":10,"end_character":11},"updated":"2016-06-22 19:05:44.000000000","message":"original","commit_id":"dfff3023c7840d48b27e367096009ac93c7eab41"},{"author":{"_account_id":10058,"name":"Erlon R. Cruz","email":"erlon.rodrigues.cruz@canonical.com","username":"sombrafam"},"change_message_id":"4ccd8f70051d60d932e78cc2c3e07191c7c0ebd1","unresolved":false,"context_lines":[{"line_number":11,"context_line":"difference. This patch fixes it."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Change-Id: Id8cc82ec9d1565f7cbd38684f6d1a712cfba1cd6"},{"line_number":14,"context_line":"Closes-Bug: #1560640"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"3aaa91ec_b073c514","line":14,"updated":"2016-06-22 19:05:44.000000000","message":"Depends-on: I829c0e6f99fff0ec0bd5366c9d2e79b1b073f0a3\n\nTempest currently does not cover the path touched by this fix. So, we should run this extend test as dependency so we can assure they work for all backends after CIs runs.","commit_id":"dfff3023c7840d48b27e367096009ac93c7eab41"}],"cinder/volume/drivers/tegile.py":[{"author":{"_account_id":19153,"name":"jingtao liang","email":"liang.jingtao@zte.com.cn","username":"liangjingtao"},"change_message_id":"6b1ee39b37f304b3d736342390eb748e154388a3","unresolved":false,"context_lines":[{"line_number":291,"context_line":""},{"line_number":292,"context_line":"        self._api_executor.send_api_request(\u0027cloneVolumeSnapshot\u0027, params)"},{"line_number":293,"context_line":"        res \u003d self.get_additional_info(volume, pool, project)"},{"line_number":294,"context_line":"        if volume[\u0027size\u0027] \u003e snapshot[\u0027size\u0027]:"},{"line_number":295,"context_line":"            self.extend_volume(volume, volume[\u0027size\u0027])"},{"line_number":296,"context_line":"        return res"},{"line_number":297,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"5a9d85d2_210ea834","line":294,"range":{"start_line":294,"start_character":28,"end_line":294,"end_character":44},"updated":"2016-06-20 07:31:45.000000000","message":"This is the same error like xio driver, because snapshot object has volume_size instead of size.So it is right to use snapshot[\u0027volume_size\u0027]","commit_id":"dfff3023c7840d48b27e367096009ac93c7eab41"},{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"ac953d1cfa809442fc1c0e6bafabd51bba0da213","unresolved":false,"context_lines":[{"line_number":291,"context_line":""},{"line_number":292,"context_line":"        self._api_executor.send_api_request(\u0027cloneVolumeSnapshot\u0027, params)"},{"line_number":293,"context_line":"        res \u003d self.get_additional_info(volume, pool, project)"},{"line_number":294,"context_line":"        if volume[\u0027size\u0027] \u003e snapshot[\u0027size\u0027]:"},{"line_number":295,"context_line":"            self.extend_volume(volume, volume[\u0027size\u0027])"},{"line_number":296,"context_line":"        return res"},{"line_number":297,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"3aaa91ec_11975891","line":294,"range":{"start_line":294,"start_character":28,"end_line":294,"end_character":44},"in_reply_to":"5a9d85d2_210ea834","updated":"2016-06-22 11:47:00.000000000","message":"I agree, and we should not be using Versioned Objects as dictionaries in new code:\n\n if volume.size \u003e snapshot.volume_size:","commit_id":"dfff3023c7840d48b27e367096009ac93c7eab41"},{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"ac953d1cfa809442fc1c0e6bafabd51bba0da213","unresolved":false,"context_lines":[{"line_number":292,"context_line":"        self._api_executor.send_api_request(\u0027cloneVolumeSnapshot\u0027, params)"},{"line_number":293,"context_line":"        res \u003d self.get_additional_info(volume, pool, project)"},{"line_number":294,"context_line":"        if volume[\u0027size\u0027] \u003e snapshot[\u0027size\u0027]:"},{"line_number":295,"context_line":"            self.extend_volume(volume, volume[\u0027size\u0027])"},{"line_number":296,"context_line":"        return res"},{"line_number":297,"context_line":""},{"line_number":298,"context_line":"    @debugger"}],"source_content_type":"text/x-python","patch_set":1,"id":"3aaa91ec_91e9881d","line":295,"range":{"start_line":295,"start_character":39,"end_line":295,"end_character":53},"updated":"2016-06-22 11:47:00.000000000","message":"volume.size","commit_id":"dfff3023c7840d48b27e367096009ac93c7eab41"}]}