)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":20190,"name":"zhufl","email":"zhu.fanglei@zte.com.cn","username":"zhufl"},"change_message_id":"851dbcd99616d59887ad63ffc8c254bbd094505a","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Remove unnecessary assertIn"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"addCleanup method uses \u0027id\u0027 key for volume type deletion,"},{"line_number":10,"context_line":"so there isn\u0027t need to validate that the key exist."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: I0741de04dbcc329cf642e58987194c11db9e3a9d"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"7f515b1d_f9bf835e","line":9,"range":{"start_line":9,"start_character":0,"end_line":9,"end_character":56},"updated":"2017-09-27 02:35:30.000000000","message":"sorry but this is not the reason why we needn\u0027t check \u0027id\u0027 in body, because addCleanup is after the check,\nso if \u0027id\u0027 was really not in body, then we would get an unexpected error \"KeyError: \u0027id\u0027\"","commit_id":"bbabe5485b444de9850d647f522eeccfb4944650"},{"author":{"_account_id":19262,"name":"Liron Kuchlani","email":"lkuchlan@redhat.com","username":"lkuchlan"},"change_message_id":"85ba196d4b3b540ab16a58327c2393420f4589b8","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Remove unnecessary assertIn"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"addCleanup method uses \u0027id\u0027 key for volume type deletion,"},{"line_number":10,"context_line":"so there isn\u0027t need to validate that the key exist."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: I0741de04dbcc329cf642e58987194c11db9e3a9d"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"7f515b1d_ea7f9fdd","line":9,"range":{"start_line":9,"start_character":0,"end_line":9,"end_character":56},"in_reply_to":"7f515b1d_f9bf835e","updated":"2017-09-27 05:35:49.000000000","message":"I really don\u0027t understand the benefit of this check, if \u0027id\u0027 key isn\u0027t exist so the test will fail with MismatchError or with KeyError, beyond that other methods don\u0027t use this check.","commit_id":"bbabe5485b444de9850d647f522eeccfb4944650"}]}