)]}'
{"nova/objects/resource_provider.py":[{"author":{"_account_id":5754,"name":"Alex Xu","email":"hejie.xu@intel.com","username":"xuhj"},"change_message_id":"179d56f498fb099198a2659bcf0b3a46d09dd608","unresolved":false,"context_lines":[{"line_number":121,"context_line":"                _ALLOC_TBL.c.resource_class_id \u003d\u003d res_class))"},{"line_number":122,"context_line":"        allocations \u003d conn.execute(allocation_query).first()"},{"line_number":123,"context_line":"        if allocations and allocations[\u0027usage\u0027] \u003e inv_record.capacity:"},{"line_number":124,"context_line":"            raise exception.InvalidInventoryNewCapacityExceeded("},{"line_number":125,"context_line":"                resource_class\u003dfields.ResourceClass.from_index(res_class),"},{"line_number":126,"context_line":"                resource_provider\u003drp.uuid)"},{"line_number":127,"context_line":"        upd_stmt \u003d _INV_TBL.update().where(sa.and_("}],"source_content_type":"text/x-python","patch_set":4,"id":"9a89bdaa_52d7d0be","side":"PARENT","line":124,"range":{"start_line":124,"start_character":18,"end_line":124,"end_character":63},"updated":"2016-09-06 08:43:58.000000000","message":"You can remove this exception and the catching of this exception now https://github.com/openstack/nova/blob/master/nova/api/openstack/placement/handlers/inventory.py#L350","commit_id":"345febe3216a9cb3eb16194c8df981a116a4f9d8"},{"author":{"_account_id":11564,"name":"Chris Dent","email":"cdent@anticdent.org","username":"chdent"},"change_message_id":"c7375d03a0c13d11aebb59b2824828b38740369d","unresolved":false,"context_lines":[{"line_number":121,"context_line":"                _ALLOC_TBL.c.resource_class_id \u003d\u003d res_class))"},{"line_number":122,"context_line":"        allocations \u003d conn.execute(allocation_query).first()"},{"line_number":123,"context_line":"        if allocations and allocations[\u0027usage\u0027] \u003e inv_record.capacity:"},{"line_number":124,"context_line":"            raise exception.InvalidInventoryNewCapacityExceeded("},{"line_number":125,"context_line":"                resource_class\u003dfields.ResourceClass.from_index(res_class),"},{"line_number":126,"context_line":"                resource_provider\u003drp.uuid)"},{"line_number":127,"context_line":"        upd_stmt \u003d _INV_TBL.update().where(sa.and_("}],"source_content_type":"text/x-python","patch_set":4,"id":"9a89bdaa_95ae9d83","side":"PARENT","line":124,"in_reply_to":"9a89bdaa_52d7d0be","updated":"2016-09-06 10:01:21.000000000","message":"We\u0027re still waiting to decide if this is actually the change we want to do and in the style we want to do because it is a change from the original plan.\n\nBut yeah, if we do stick with this, good catch.","commit_id":"345febe3216a9cb3eb16194c8df981a116a4f9d8"}]}