)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":10058,"name":"Erlon R. Cruz","email":"erlon.rodrigues.cruz@canonical.com","username":"sombrafam"},"change_message_id":"8f4291683566486a3b9a1dc1ab0e9c8511cd8fd5","unresolved":false,"context_lines":[{"line_number":10,"context_line":"is larger than the image it was cloned from. Extended test case to"},{"line_number":11,"context_line":"cover the change."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Closes-Bug: #1554765"},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Change-Id: I41303451eb167198d7e7533fc66134f7e2fbcd18"},{"line_number":16,"context_line":"Signed-off-by: Danny Al-Gaaf \u003cdanny.al-gaaf@bisect.de\u003e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"3aaa91ec_3f5cc8ec","line":13,"updated":"2016-06-22 17:59:12.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":"c6506bae3ff480c9c63e3191e9a3cfb3dace0246"}],"cinder/tests/unit/test_cloudbyte.py":[{"author":{"_account_id":9171,"name":"Vipin Balachandran","email":"vipin.bl@gmail.com","username":"vbala"},"change_message_id":"038ae1708178abf7cb693dc126a65cdb61852ed9","unresolved":false,"context_lines":[{"line_number":1369,"context_line":"            self.driver.create_volume_from_snapshot(cloned_volume, snapshot))"},{"line_number":1370,"context_line":""},{"line_number":1371,"context_line":"        # assert n api calls were invoked"},{"line_number":1372,"context_line":"        self.assertEqual(5, mock_api_req.call_count)"},{"line_number":1373,"context_line":""},{"line_number":1374,"context_line":"    @mock.patch.object(cloudbyte.CloudByteISCSIDriver,"},{"line_number":1375,"context_line":"                       \u0027_api_request_for_cloudbyte\u0027)"}],"source_content_type":"text/x-python","patch_set":2,"id":"1af94dfe_ce9ebcb7","line":1372,"updated":"2016-03-16 17:01:37.000000000","message":"I think you also need to assert extend_volume was called with the correct arguments.","commit_id":"42c4c21eab4c7bf42d4b59dcf099265822609b04"}],"cinder/volume/drivers/cloudbyte/cloudbyte.py":[{"author":{"_account_id":18026,"name":"huanan","email":"huanan@huawei.com","username":"huanan"},"change_message_id":"8a5ccb65d11ceadcc5223df095e77b6954267f84","unresolved":false,"context_lines":[{"line_number":1068,"context_line":"             \u0027stack_vol\u0027: parent_volume_id})"},{"line_number":1069,"context_line":""},{"line_number":1070,"context_line":"        if ((\u0027size\u0027 in cloned_volume) and (cloned_volume[\u0027size\u0027] \u003e"},{"line_number":1071,"context_line":"                                           snapshot[\u0027size\u0027])):"},{"line_number":1072,"context_line":"            self.extend_volume(cloned_volume, cloned_volume[\u0027size\u0027])"},{"line_number":1073,"context_line":""},{"line_number":1074,"context_line":"        chap_info \u003d {}"}],"source_content_type":"text/x-python","patch_set":3,"id":"1af94dfe_12c0a77a","line":1071,"range":{"start_line":1071,"start_character":57,"end_line":1071,"end_character":62},"updated":"2016-03-17 08:28:49.000000000","message":"I think the following is better.\n\nif ((\u0027size\u0027 in cloned_volume)\n    and (cloned_volume[\u0027size\u0027] \u003e snapshot[\u0027size\u0027])):","commit_id":"0fe9913065a018680c3a9f6a6657045b57f96360"},{"author":{"_account_id":9064,"name":"Danny Al-Gaaf","email":"danny.al-gaaf@bisect.de","username":"dalgaaf"},"change_message_id":"328c2f0067fab0f2abca761261cf0347e20eebcd","unresolved":false,"context_lines":[{"line_number":1068,"context_line":"             \u0027stack_vol\u0027: parent_volume_id})"},{"line_number":1069,"context_line":""},{"line_number":1070,"context_line":"        if ((\u0027size\u0027 in cloned_volume) and (cloned_volume[\u0027size\u0027] \u003e"},{"line_number":1071,"context_line":"                                           snapshot[\u0027size\u0027])):"},{"line_number":1072,"context_line":"            self.extend_volume(cloned_volume, cloned_volume[\u0027size\u0027])"},{"line_number":1073,"context_line":""},{"line_number":1074,"context_line":"        chap_info \u003d {}"}],"source_content_type":"text/x-python","patch_set":3,"id":"1af94dfe_669ded58","line":1071,"range":{"start_line":1071,"start_character":57,"end_line":1071,"end_character":62},"in_reply_to":"1af94dfe_0892389e","updated":"2016-03-17 10:15:06.000000000","message":"And the proposal make the pep8 checks fail.","commit_id":"0fe9913065a018680c3a9f6a6657045b57f96360"},{"author":{"_account_id":9171,"name":"Vipin Balachandran","email":"vipin.bl@gmail.com","username":"vbala"},"change_message_id":"6523e9296a1f56120a15ed06c816d23c3b24cc8e","unresolved":false,"context_lines":[{"line_number":1068,"context_line":"             \u0027stack_vol\u0027: parent_volume_id})"},{"line_number":1069,"context_line":""},{"line_number":1070,"context_line":"        if ((\u0027size\u0027 in cloned_volume) and (cloned_volume[\u0027size\u0027] \u003e"},{"line_number":1071,"context_line":"                                           snapshot[\u0027size\u0027])):"},{"line_number":1072,"context_line":"            self.extend_volume(cloned_volume, cloned_volume[\u0027size\u0027])"},{"line_number":1073,"context_line":""},{"line_number":1074,"context_line":"        chap_info \u003d {}"}],"source_content_type":"text/x-python","patch_set":3,"id":"1af94dfe_0892389e","line":1071,"range":{"start_line":1071,"start_character":57,"end_line":1071,"end_character":62},"in_reply_to":"1af94dfe_12c0a77a","updated":"2016-03-17 09:45:15.000000000","message":"What is your suggestion here? I can only see some formatting change. Giving -1 for formatting change which looks just fine is too harsh.","commit_id":"0fe9913065a018680c3a9f6a6657045b57f96360"},{"author":{"_account_id":9064,"name":"Danny Al-Gaaf","email":"danny.al-gaaf@bisect.de","username":"dalgaaf"},"change_message_id":"273fbee468503a50fa1c1b735952ece38ab9ad3e","unresolved":false,"context_lines":[{"line_number":1068,"context_line":"             \u0027stack_vol\u0027: parent_volume_id})"},{"line_number":1069,"context_line":""},{"line_number":1070,"context_line":"        if ((\u0027size\u0027 in cloned_volume) and (cloned_volume[\u0027size\u0027] \u003e"},{"line_number":1071,"context_line":"                                           snapshot[\u0027size\u0027])):"},{"line_number":1072,"context_line":"            self.extend_volume(cloned_volume, cloned_volume[\u0027size\u0027])"},{"line_number":1073,"context_line":""},{"line_number":1074,"context_line":"        chap_info \u003d {}"}],"source_content_type":"text/x-python","patch_set":3,"id":"1af94dfe_8bb76285","line":1071,"range":{"start_line":1071,"start_character":57,"end_line":1071,"end_character":62},"in_reply_to":"1af94dfe_12c0a77a","updated":"2016-03-17 10:12:45.000000000","message":"Your change would raise a pep8 error: \nE129 visually indented line with same indent as next logical line\n\nTherefore I can\u0027t change it.","commit_id":"0fe9913065a018680c3a9f6a6657045b57f96360"}]}