)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"c3c6c18563585bbd1ccd64af225a2ba7ad0f53cf","unresolved":false,"context_lines":[{"line_number":14,"context_line":"* Remove private volume type access from project"},{"line_number":15,"context_line":"  POST /v3/{project_id}/types/{volume_type}/action"},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"Made changes to unit tests to pass body as keyword argument as wsgi"},{"line_number":18,"context_line":"calls action method [1] and passes body as keyword argument."},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"[1] https://github.com/openstack/cinder/blob/master/cinder/api/openstack/wsgi.py#L997"},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"Partial-Implements: bp json-schema-validation"},{"line_number":23,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"7f96bb07_f00c8f43","line":20,"range":{"start_line":17,"start_character":0,"end_line":20,"end_character":85},"updated":"2018-01-23 13:22:28.000000000","message":"nit: Usually it is not necessary to add this to the commit message, you can submit your patch and add it as the first comment on the review.","commit_id":"1a29898d981fdb06da283b0b767a9f178438608d"}],"cinder/api/schemas/volume_type_access.py":[{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"c3c6c18563585bbd1ccd64af225a2ba7ad0f53cf","unresolved":false,"context_lines":[{"line_number":53,"context_line":"    },"},{"line_number":54,"context_line":"    \u0027required\u0027: [\u0027removeProjectAccess\u0027],"},{"line_number":55,"context_line":"    \u0027additionalProperties\u0027: False,"},{"line_number":56,"context_line":"}"}],"source_content_type":"text/x-python","patch_set":1,"id":"7f96bb07_503d43db","line":56,"updated":"2018-01-23 13:22:28.000000000","message":"nit: I would use the same validation for both, and if we need to differentiate them in the future we can split them then. We could call it add_remove_project_access","commit_id":"1a29898d981fdb06da283b0b767a9f178438608d"}]}