)]}'
{"openstack/compute/v2/limits_absolute.py":[{"author":{"_account_id":8257,"name":"Brian Curtin","email":"brian@python.org","username":"briancurtin"},"change_message_id":"e4378981a1b720c0d14d3f3e09ec8014c67676b5","unresolved":false,"context_lines":[{"line_number":16,"context_line":"from openstack import resource"},{"line_number":17,"context_line":""},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"class LimitsAbsolute(resource.Resource):"},{"line_number":20,"context_line":"    resource_key \u003d \u0027limits_absolute\u0027"},{"line_number":21,"context_line":"    resources_key \u003d \u0027limits_absolutes\u0027"},{"line_number":22,"context_line":"    base_path \u003d \u0027/limits\u0027"}],"source_content_type":"text/x-python","patch_set":1,"id":"fa98f980_8697818f","line":19,"updated":"2014-09-11 19:18:05.000000000","message":"I think we should either drop Absolute from the name, or make it AbsoluteLimits.","commit_id":"4f65a78fe8ef34d9f6da73bb0152810dbf10af55"},{"author":{"_account_id":8257,"name":"Brian Curtin","email":"brian@python.org","username":"briancurtin"},"change_message_id":"e4378981a1b720c0d14d3f3e09ec8014c67676b5","unresolved":false,"context_lines":[{"line_number":27,"context_line":""},{"line_number":28,"context_line":"    # Properties"},{"line_number":29,"context_line":"    name \u003d resource.prop(\u0027name\u0027)"},{"line_number":30,"context_line":"    value \u003d resource.prop(\u0027value\u0027)"},{"line_number":31,"context_line":""},{"line_number":32,"context_line":"    @classmethod"},{"line_number":33,"context_line":"    def list(cls, session, path_args\u003dNone, **params):"}],"source_content_type":"text/x-python","patch_set":1,"id":"fa98f980_8673017f","line":30,"updated":"2014-09-11 19:18:05.000000000","message":"Shouldn\u0027t this class account for more than just the \"absolute\" data, but also include the \"rate\" items?","commit_id":"4f65a78fe8ef34d9f6da73bb0152810dbf10af55"},{"author":{"_account_id":8736,"name":"Terry Howe","email":"terrylhowe@gmail.com","username":"terrylhowe"},"change_message_id":"0fae86e06b0d3a4cb54a53284a375611eb942c15","unresolved":false,"context_lines":[{"line_number":27,"context_line":""},{"line_number":28,"context_line":"    # Properties"},{"line_number":29,"context_line":"    name \u003d resource.prop(\u0027name\u0027)"},{"line_number":30,"context_line":"    value \u003d resource.prop(\u0027value\u0027)"},{"line_number":31,"context_line":""},{"line_number":32,"context_line":"    @classmethod"},{"line_number":33,"context_line":"    def list(cls, session, path_args\u003dNone, **params):"}],"source_content_type":"text/x-python","patch_set":1,"id":"fa98f980_e39dd1bc","line":30,"in_reply_to":"fa98f980_8673017f","updated":"2014-09-19 14:59:49.000000000","message":"Well, the /limits call returns two distinct chunks of data, so I used two resources on one URL.  The rate limits are already out there:\n\nhttps://github.com/stackforge/python-openstacksdk/blob/master/openstack/compute/v2/limits_rate.py\n\nIf we are going to kill this review, I\u0027ll rename limits_rate.py to limits.py and figure out some way to jam absolute limits in there.","commit_id":"4f65a78fe8ef34d9f6da73bb0152810dbf10af55"}]}