)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":5441,"name":"Andrew Laski","email":"andrew@lascii.com","username":"alaski"},"change_message_id":"59d854b28d60fe2c4e2c036afb18b4ada1db2074","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Delete a quota through admin api."},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Exposes the delete api implemented as part of blueprint admin-api-for-delete-quota"},{"line_number":10,"context_line":"Change-Id: Iab358f0fcf2dfb41bcd9a3a5b73d590d3f55cd6c"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"AAAAPX%2F%2F%2FPQ%3D","line":9,"updated":"2013-06-04 13:57:53.000000000","message":"This should be wrapped at 72 chars.  https://wiki.openstack.org/wiki/GitCommitMessages#Summary_of_GIT_commit_message_structure","commit_id":"be4da7bf17e0b4743d432030be911b21f6fcf42d"},{"author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"change_message_id":"a37304a5b43cfca18ca5a112632a6e49f5046dbc","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Delete a quota through admin api."},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Exposes the quota-delete api mplemented as part of"},{"line_number":10,"context_line":"blueprint admin-api-for-delete-quota"},{"line_number":11,"context_line":"Change-Id: Iab358f0fcf2dfb41bcd9a3a5b73d590d3f55cd6c"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"AAAAPX%2F%2F5ZA%3D","line":9,"updated":"2013-06-06 15:09:21.000000000","message":"s/mplemented/implemented/ (femtonit)","commit_id":"3247cef88aa02908fc94feb7f1c05cfd73d5c8ee"}],"novaclient/tests/v1_1/fakes.py":[{"author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"change_message_id":"a37304a5b43cfca18ca5a112632a6e49f5046dbc","unresolved":false,"context_lines":[{"line_number":1015,"context_line":"                      \u0027security_group_rules\u0027: 1}})"},{"line_number":1016,"context_line":""},{"line_number":1017,"context_line":"    def delete_os_quota_sets_test(self, **kw):"},{"line_number":1018,"context_line":"        return 202, {}, {}"},{"line_number":1019,"context_line":""},{"line_number":1020,"context_line":"    def delete_os_quota_sets_97f4c221bff44578b0300df4ef119353(self, **kw):"},{"line_number":1021,"context_line":"        return 202, {}, {}"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAPX%2F%2F5Yw%3D","line":1018,"updated":"2013-06-06 15:09:21.000000000","message":"femtonit: For consistency, it would be nice for these tuples to be enclosed in parentheses, even though that is not strictly necessary.","commit_id":"3247cef88aa02908fc94feb7f1c05cfd73d5c8ee"}],"novaclient/v1_1/shell.py":[{"author":{"_account_id":5441,"name":"Andrew Laski","email":"andrew@lascii.com","username":"alaski"},"change_message_id":"7ca13227ec1cd3013fa7496d341f6a1254cb9b4e","unresolved":false,"context_lines":[{"line_number":2916,"context_line":""},{"line_number":2917,"context_line":"@utils.arg(\u0027--tenant\u0027,"},{"line_number":2918,"context_line":"           metavar\u003d\u0027\u003ctenant-id\u003e\u0027,"},{"line_number":2919,"context_line":"           default\u003dNone,"},{"line_number":2920,"context_line":"           help\u003d\u0027ID of tenant to list the default quotas for.\u0027)"},{"line_number":2921,"context_line":"def do_quota_delete(cs, args):"},{"line_number":2922,"context_line":"    \"\"\"Delete quota for a tenant so his quota will revert back to default.\"\"\""}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAPH%2F%2Fc3w%3D","line":2919,"updated":"2013-06-03 17:08:21.000000000","message":"I don\u0027t think a default is appropriate here.  Better to make it a required argument.","commit_id":"e867f936d080deee6f30385605d35f1194573749"},{"author":{"_account_id":5441,"name":"Andrew Laski","email":"andrew@lascii.com","username":"alaski"},"change_message_id":"7ca13227ec1cd3013fa7496d341f6a1254cb9b4e","unresolved":false,"context_lines":[{"line_number":2917,"context_line":"@utils.arg(\u0027--tenant\u0027,"},{"line_number":2918,"context_line":"           metavar\u003d\u0027\u003ctenant-id\u003e\u0027,"},{"line_number":2919,"context_line":"           default\u003dNone,"},{"line_number":2920,"context_line":"           help\u003d\u0027ID of tenant to list the default quotas for.\u0027)"},{"line_number":2921,"context_line":"def do_quota_delete(cs, args):"},{"line_number":2922,"context_line":"    \"\"\"Delete quota for a tenant so his quota will revert back to default.\"\"\""},{"line_number":2923,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAPH%2F%2Fc5A%3D","line":2920,"updated":"2013-06-03 17:08:21.000000000","message":"s/list/delete/","commit_id":"e867f936d080deee6f30385605d35f1194573749"},{"author":{"_account_id":5441,"name":"Andrew Laski","email":"andrew@lascii.com","username":"alaski"},"change_message_id":"59d854b28d60fe2c4e2c036afb18b4ada1db2074","unresolved":false,"context_lines":[{"line_number":2918,"context_line":"           metavar\u003d\u0027\u003ctenant-id\u003e\u0027,"},{"line_number":2919,"context_line":"           help\u003d\u0027ID of tenant to delete the default quotas for.\u0027)"},{"line_number":2920,"context_line":"def do_quota_delete(cs, args):"},{"line_number":2921,"context_line":"    \"\"\"Delete quota for a tenant so his quota will revert back to default.\"\"\""},{"line_number":2922,"context_line":""},{"line_number":2923,"context_line":"    cs.quotas.delete(args.tenant)"},{"line_number":2924,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAPX%2F%2F%2FO0%3D","line":2921,"updated":"2013-06-04 13:57:53.000000000","message":"The use of \u0027his\u0027 really sticks out to me here.  We don\u0027t use gender specific pronouns anywhere else that I can find and I think it\u0027s better to stay that way.  Could you change this to \u0027their\u0027?","commit_id":"be4da7bf17e0b4743d432030be911b21f6fcf42d"},{"author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"change_message_id":"a37304a5b43cfca18ca5a112632a6e49f5046dbc","unresolved":false,"context_lines":[{"line_number":2916,"context_line":""},{"line_number":2917,"context_line":"@utils.arg(\u0027--tenant\u0027,"},{"line_number":2918,"context_line":"           metavar\u003d\u0027\u003ctenant-id\u003e\u0027,"},{"line_number":2919,"context_line":"           help\u003d\u0027ID of tenant to delete the default quotas for.\u0027)"},{"line_number":2920,"context_line":"def do_quota_delete(cs, args):"},{"line_number":2921,"context_line":"    \"\"\"Delete quota for a tenant so their quota will revert back to default.\"\"\""},{"line_number":2922,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAPX%2F%2F5Xw%3D","line":2919,"updated":"2013-06-06 15:09:21.000000000","message":"This help text seems to contradict the docstring for do_quota_delete(); this implies that the default quotas are being deleted, whereas the purpose of the command is to revert quotas to the defaults…","commit_id":"3247cef88aa02908fc94feb7f1c05cfd73d5c8ee"}]}