)]}'
{"nova/api/openstack/compute/contrib/quotas.py":[{"author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"change_message_id":"237fa25e92e892a25f6d2fda43d4f32ac074c61e","unresolved":false,"context_lines":[{"line_number":90,"context_line":"        try:"},{"line_number":91,"context_line":"            nova.context.authorize_project_context(context, id)"},{"line_number":92,"context_line":"            QUOTAS.destroy_all_by_project(context, id)"},{"line_number":93,"context_line":"            return webob.Response(status_int\u003d200)"},{"line_number":94,"context_line":"        except exception.NotAuthorized:"},{"line_number":95,"context_line":"            raise webob.exc.HTTPForbidden()"},{"line_number":96,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAPH%2F%2F2uA%3D","line":93,"updated":"2013-05-21 16:09:17.000000000","message":"200 is the wrong response code.  I believe you want 202 here.  (Additionally, I think you should be able to do that with @wsgi.response()…)","commit_id":"baaa60a5614756addf45ee60961a940cf4227b6c"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"57789f853c242a5ed9f0e9d3d78f8c2abf0d8b40","unresolved":false,"context_lines":[{"line_number":141,"context_line":""},{"line_number":142,"context_line":"    name \u003d \"Quotas\""},{"line_number":143,"context_line":"    alias \u003d \"os-quota-sets\""},{"line_number":144,"context_line":"    namespace \u003d \"http://docs.openstack.org/compute/ext/quotas-sets/api/v1.1\""},{"line_number":145,"context_line":"    updated \u003d \"2011-08-08T00:00:00+00:00\""},{"line_number":146,"context_line":""},{"line_number":147,"context_line":"    def get_resources(self):"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAPH%2F%2F0Qc%3D","line":144,"updated":"2013-05-22 12:26:23.000000000","message":"need to at least update this","commit_id":"baaa60a5614756addf45ee60961a940cf4227b6c"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"79a300abcb253ccc570909e7668284ca6bdd86f6","unresolved":false,"context_lines":[{"line_number":138,"context_line":"                return webob.Response(status_int\u003d202)"},{"line_number":139,"context_line":"            except exception.NotAuthorized:"},{"line_number":140,"context_line":"                raise webob.exc.HTTPForbidden()"},{"line_number":141,"context_line":"        raise webob.exc.HTTPNotFound()"},{"line_number":142,"context_line":""},{"line_number":143,"context_line":""},{"line_number":144,"context_line":"class Quotas(extensions.ExtensionDescriptor):"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAPH%2F%2FxjU%3D","line":141,"updated":"2013-05-28 10:08:35.000000000","message":"very minor nit, but I would try avoid this nesting with inverting the if.","commit_id":"cdd998c418edc42a6a828ddef144e8408c206b7c"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"79a300abcb253ccc570909e7668284ca6bdd86f6","unresolved":false,"context_lines":[{"line_number":154,"context_line":""},{"line_number":155,"context_line":"        res \u003d extensions.ResourceExtension(\u0027os-quota-sets\u0027,"},{"line_number":156,"context_line":"                                            QuotaSetsController(self.ext_mgr),"},{"line_number":157,"context_line":"                                            member_actions\u003d{\u0027defaults\u0027: \u0027GET\u0027})"},{"line_number":158,"context_line":"        resources.append(res)"},{"line_number":159,"context_line":""},{"line_number":160,"context_line":"        return resources"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAPH%2F%2FxjE%3D","line":157,"updated":"2013-05-28 10:08:35.000000000","message":"Should some of these really be POST since they modify state?","commit_id":"cdd998c418edc42a6a828ddef144e8408c206b7c"}]}
