)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":9003,"name":"Tom Barron","email":"tpb@dyncloud.net","username":"tbarron"},"change_message_id":"995d0f9993d1458b1759a7cabd889fb148c14a66","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Use assert_valid_body in volume update"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"The change is to use assert_valid_body to check body in volume update,"},{"line_number":10,"context_line":"which avoids code duplicatin."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: Icf9d88bbf5171faed6049b909a1a94795fb006b2"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"dae33548_ecadc089","line":10,"updated":"2016-02-15 10:14:33.000000000","message":"s/duplicatin/duplication if you push another patch.","commit_id":"5b001f44eeca0445c91310c58d2258389194362b"}],"cinder/api/v2/volumes.py":[{"author":{"_account_id":2759,"name":"Huang Zhiteng","email":"winston.d@gmail.com","username":"zhiteng-huang"},"change_message_id":"96838774189ec54e1a8506bea081c113b66d071e","unresolved":false,"context_lines":[{"line_number":431,"context_line":"        if not body:"},{"line_number":432,"context_line":"            msg \u003d _(\"Missing request body\")"},{"line_number":433,"context_line":"            raise exc.HTTPBadRequest(explanation\u003dmsg)"},{"line_number":434,"context_line":"        self.assert_valid_body(body, \u0027volume\u0027)"},{"line_number":435,"context_line":""},{"line_number":436,"context_line":"        volume \u003d body[\u0027volume\u0027]"},{"line_number":437,"context_line":"        update_dict \u003d {}"}],"source_content_type":"text/x-python","patch_set":1,"id":"dae33548_1eaad3d4","line":434,"updated":"2016-02-15 06:50:48.000000000","message":"assert_valid_body() checks a bit more than existing code, for example:\n\n    body[\u0027volume\u0027] \u003d \u0027string\u0027\n\nwas a valid input (won\u0027t raise BadRequest exeception), but this patch changes the behavior.","commit_id":"5b001f44eeca0445c91310c58d2258389194362b"},{"author":{"_account_id":9003,"name":"Tom Barron","email":"tpb@dyncloud.net","username":"tbarron"},"change_message_id":"995d0f9993d1458b1759a7cabd889fb148c14a66","unresolved":false,"context_lines":[{"line_number":431,"context_line":"        if not body:"},{"line_number":432,"context_line":"            msg \u003d _(\"Missing request body\")"},{"line_number":433,"context_line":"            raise exc.HTTPBadRequest(explanation\u003dmsg)"},{"line_number":434,"context_line":"        self.assert_valid_body(body, \u0027volume\u0027)"},{"line_number":435,"context_line":""},{"line_number":436,"context_line":"        volume \u003d body[\u0027volume\u0027]"},{"line_number":437,"context_line":"        update_dict \u003d {}"}],"source_content_type":"text/x-python","patch_set":1,"id":"dae33548_4c882ce8","line":434,"in_reply_to":"dae33548_1eaad3d4","updated":"2016-02-15 10:14:33.000000000","message":"But was the old check not strict enough?  If so, then I think this would be the right thing to do (and update the commit message to indicate that the check has been improved).","commit_id":"5b001f44eeca0445c91310c58d2258389194362b"}]}