)]}'
{"tempest/api/compute/admin/test_quotas.py":[{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"16b2bf2e54ac5f17d459f74e63cbf86da11d11b5","unresolved":false,"context_lines":[{"line_number":149,"context_line":"    def setUpClass(cls):"},{"line_number":150,"context_line":"        # All test cases in this class need to externally lock on doing"},{"line_number":151,"context_line":"        # anything with default quota values."},{"line_number":152,"context_line":"        # TODO(mriedem): does this need to be in setUp or is setUpClass OK?"},{"line_number":153,"context_line":"        cls.useFixture(fixtures.LockFixture(\u0027compute_quotas\u0027))"},{"line_number":154,"context_line":"        super(QuotaClassesAdminTestJSON, cls).setUpClass()"},{"line_number":155,"context_line":"        cls.adm_client \u003d cls.os_adm.quota_classes_client"}],"source_content_type":"text/x-python","patch_set":1,"id":"1ae5cdf2_015cc5ec","line":152,"updated":"2014-06-16 20:27:49.000000000","message":"Needs to be in setUp.","commit_id":"34446948718f27e8dd3d81583618cab421413909"},{"author":{"_account_id":1921,"name":"Andrea Frittoli","email":"andrea.frittoli@gmail.com","username":"andrea-frittoli"},"change_message_id":"f57380dc613bedd5c3eb6ed113fe7c10110f6ee0","unresolved":false,"context_lines":[{"line_number":171,"context_line":""},{"line_number":172,"context_line":"    @test.attr(type\u003d\u0027gate\u0027)"},{"line_number":173,"context_line":"    def test_update_default_quotas(self):"},{"line_number":174,"context_line":"        LOG.debug(\"get the current \u0027default\u0027 quota class values\")"},{"line_number":175,"context_line":"        resp, body \u003d self.adm_client.get_quota_class_set(\u0027default\u0027)"},{"line_number":176,"context_line":"        self.assertEqual(200, resp.status)"},{"line_number":177,"context_line":"        self.assertIn(\u0027id\u0027, body)"}],"source_content_type":"text/x-python","patch_set":6,"id":"1ae5cdf2_66754df9","line":174,"updated":"2014-06-25 20:35:16.000000000","message":"Do we need this log message?","commit_id":"848805f21b30548832302345759b7d648e15d2be"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"36b27f60c1ca30bb572e6ffbe084728f7a0498b3","unresolved":false,"context_lines":[{"line_number":171,"context_line":""},{"line_number":172,"context_line":"    @test.attr(type\u003d\u0027gate\u0027)"},{"line_number":173,"context_line":"    def test_update_default_quotas(self):"},{"line_number":174,"context_line":"        LOG.debug(\"get the current \u0027default\u0027 quota class values\")"},{"line_number":175,"context_line":"        resp, body \u003d self.adm_client.get_quota_class_set(\u0027default\u0027)"},{"line_number":176,"context_line":"        self.assertEqual(200, resp.status)"},{"line_number":177,"context_line":"        self.assertIn(\u0027id\u0027, body)"}],"source_content_type":"text/x-python","patch_set":6,"id":"1ae5cdf2_2e868e65","line":174,"in_reply_to":"1ae5cdf2_66754df9","updated":"2014-06-26 20:41:09.000000000","message":"These were comments before, but in general I think a good trend to have in tempest is more trace logging when we have race failures.  I don\u0027t expect race failures with these tests at all, but trace logging for the tests is a behavior I\u0027m using for new code.","commit_id":"848805f21b30548832302345759b7d648e15d2be"}],"tempest/api/compute/test_quotas.py":[{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"16b2bf2e54ac5f17d459f74e63cbf86da11d11b5","unresolved":false,"context_lines":[{"line_number":22,"context_line":""},{"line_number":23,"context_line":"    @classmethod"},{"line_number":24,"context_line":"    def setUpClass(cls):"},{"line_number":25,"context_line":"        cls.useFixture(fixtures.LockFixture(\u0027compute_quotas\u0027))"},{"line_number":26,"context_line":"        super(QuotasTestJSON, cls).setUpClass()"},{"line_number":27,"context_line":"        cls.client \u003d cls.quotas_client"},{"line_number":28,"context_line":"        cls.tenant_id \u003d cls.client.tenant_id"}],"source_content_type":"text/x-python","patch_set":1,"id":"1ae5cdf2_616ae982","line":25,"updated":"2014-06-16 20:27:49.000000000","message":"Needs to be in setUp.","commit_id":"34446948718f27e8dd3d81583618cab421413909"}],"tempest/api/compute/v3/test_quotas.py":[{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"16b2bf2e54ac5f17d459f74e63cbf86da11d11b5","unresolved":false,"context_lines":[{"line_number":22,"context_line":""},{"line_number":23,"context_line":"    @classmethod"},{"line_number":24,"context_line":"    def setUpClass(cls):"},{"line_number":25,"context_line":"        cls.useFixture(fixtures.LockFixture(\u0027compute_quotas\u0027))"},{"line_number":26,"context_line":"        super(QuotasV3Test, cls).setUpClass()"},{"line_number":27,"context_line":"        cls.client \u003d cls.quotas_client"},{"line_number":28,"context_line":"        cls.tenant_id \u003d cls.client.tenant_id"}],"source_content_type":"text/x-python","patch_set":1,"id":"1ae5cdf2_a10311bd","line":25,"updated":"2014-06-16 20:27:49.000000000","message":"Needs to be in setUp.","commit_id":"34446948718f27e8dd3d81583618cab421413909"},{"author":{"_account_id":1921,"name":"Andrea Frittoli","email":"andrea.frittoli@gmail.com","username":"andrea-frittoli"},"change_message_id":"f57380dc613bedd5c3eb6ed113fe7c10110f6ee0","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"1ae5cdf2_c6d46121","updated":"2014-06-25 20:35:16.000000000","message":"Even though we are not running v3 tests anymore by default, it\u0027s still good to have this in.","commit_id":"848805f21b30548832302345759b7d648e15d2be"}],"tempest/api_schema/compute/v2/quota_classes.py":[{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"813286c106a671b710cac5e7d8a3b26bc531892b","unresolved":false,"context_lines":[{"line_number":22,"context_line":"# quota_set, so update this copy of the schema from os-quota-sets."},{"line_number":23,"context_line":"quota_set \u003d copy.deepcopy(quotas.quota_set)"},{"line_number":24,"context_line":"quota_set[\u0027response_body\u0027][\u0027properties\u0027][\u0027quota_class_set\u0027] \u003d ("},{"line_number":25,"context_line":"    quota_set[\u0027response_body\u0027][\u0027properties\u0027].pop(\u0027quota_set\u0027))"},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"quota_set_update \u003d copy.deepcopy(quotas.quota_set_update)"},{"line_number":28,"context_line":"quota_set_update[\u0027response_body\u0027][\u0027properties\u0027][\u0027quota_class_set\u0027] \u003d ("}],"source_content_type":"text/x-python","patch_set":5,"id":"1ae5cdf2_8b2530b0","line":25,"updated":"2014-06-17 14:39:49.000000000","message":"Something isn\u0027t right here, \u0027quota_set\u0027 is still listed as a required property.","commit_id":"a70bc31a33493512d21e6fe40dc00402c86ded8a"}],"tempest/services/compute/json/quotas_client.py":[{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"44639b479437d8205c479493544d154070f65d77","unresolved":false,"context_lines":[{"line_number":136,"context_line":"        resp, body \u003d self.get(url)"},{"line_number":137,"context_line":"        body \u003d json.loads(body)"},{"line_number":138,"context_line":"        # NOTE(mriedem): we can re-use the os-quota-set schema"},{"line_number":139,"context_line":"        self.validate_response(schema.quota_set, resp, body)"},{"line_number":140,"context_line":"        return resp, body[\u0027quota_class_set\u0027]"},{"line_number":141,"context_line":""},{"line_number":142,"context_line":"    def update_quota_class_set(self, quota_class_id,"}],"source_content_type":"text/x-python","patch_set":2,"id":"1ae5cdf2_dc072080","line":139,"updated":"2014-06-16 21:05:43.000000000","message":"This won\u0027t work because jsonschema is looking for \u0027quota_set\u0027 in the response, not \u0027quota_class_set\u0027.","commit_id":"69e21347b1a4a6725fb75d17b733dad9c768040d"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"44639b479437d8205c479493544d154070f65d77","unresolved":false,"context_lines":[{"line_number":196,"context_line":""},{"line_number":197,"context_line":"        body \u003d json.loads(body)"},{"line_number":198,"context_line":"        # NOTE(mriedem): we can re-use the os-quota-set schema"},{"line_number":199,"context_line":"        self.validate_response(schema.quota_set_update, resp, body)"},{"line_number":200,"context_line":"        return resp, body[\u0027quota_class_set\u0027]"}],"source_content_type":"text/x-python","patch_set":2,"id":"1ae5cdf2_5cccf061","line":199,"updated":"2014-06-16 21:05:43.000000000","message":"This won\u0027t work because jsonschema is looking for \u0027quota_set\u0027 in the response, not \u0027quota_class_set\u0027.","commit_id":"69e21347b1a4a6725fb75d17b733dad9c768040d"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"172539262e83f0e0ced92060ea2dabc8ef9ae1d3","unresolved":false,"context_lines":[{"line_number":15,"context_line":""},{"line_number":16,"context_line":"import json"},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"from tempest.api_schema.compute.v2 import quota_classes as classes_schema"},{"line_number":19,"context_line":"from tempest.api_schema.compute.v2 import quotas as schema"},{"line_number":20,"context_line":"from tempest.common import rest_client"},{"line_number":21,"context_line":"from tempest import config"}],"source_content_type":"text/x-python","patch_set":4,"id":"1ae5cdf2_f77cf2b7","line":18,"updated":"2014-06-17 00:40:02.000000000","message":"Crap, forgot to add this when committing the latest changes.","commit_id":"c710c1450084795418811396961ca963f8bb4309"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"d0a2f0d0ee0a12bf45d7d6b67340340cdbef3928","unresolved":false,"context_lines":[{"line_number":56,"context_line":"                         security_group_rules\u003dNone, injected_files\u003dNone,"},{"line_number":57,"context_line":"                         cores\u003dNone, injected_file_path_bytes\u003dNone,"},{"line_number":58,"context_line":"                         security_groups\u003dNone):"},{"line_number":59,"context_line":"        \"\"\"update_quota_class_set"},{"line_number":60,"context_line":"        Updates the tenant\u0027s quota limits for one or more resources"},{"line_number":61,"context_line":"        \"\"\""},{"line_number":62,"context_line":"        post_body \u003d {}"}],"source_content_type":"text/x-python","patch_set":4,"id":"1ae5cdf2_f2332560","line":59,"updated":"2014-06-16 21:56:35.000000000","message":"wut","commit_id":"c710c1450084795418811396961ca963f8bb4309"},{"author":{"_account_id":1849,"name":"Joe Gordon","email":"joe.gordon0@gmail.com","username":"jogo"},"change_message_id":"35e3088359f65852ebb4dfc2eba335406d18b8d6","unresolved":false,"context_lines":[{"line_number":138,"context_line":""},{"line_number":139,"context_line":"    def update_quota_class_set(self, quota_class_id, **kwargs):"},{"line_number":140,"context_line":"        \"\"\""},{"line_number":141,"context_line":"        Updates the quota class\u0027s limits for one or more resources."},{"line_number":142,"context_line":"        \"\"\""},{"line_number":143,"context_line":"        post_body \u003d json.dumps({\u0027quota_class_set\u0027: kwargs})"},{"line_number":144,"context_line":""}],"source_content_type":"text/x-python","patch_set":6,"id":"1ae5cdf2_abf747d3","line":141,"updated":"2014-06-17 16:14:02.000000000","message":"heh this doesn\u0027t pass the hacking docstring guidelines, but it is not an issue as long as the pep8 job doesn\u0027t complain.","commit_id":"848805f21b30548832302345759b7d648e15d2be"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"874bf372d0f2c7351b809c768c4609e2e2117f5e","unresolved":false,"context_lines":[{"line_number":138,"context_line":""},{"line_number":139,"context_line":"    def update_quota_class_set(self, quota_class_id, **kwargs):"},{"line_number":140,"context_line":"        \"\"\""},{"line_number":141,"context_line":"        Updates the quota class\u0027s limits for one or more resources."},{"line_number":142,"context_line":"        \"\"\""},{"line_number":143,"context_line":"        post_body \u003d json.dumps({\u0027quota_class_set\u0027: kwargs})"},{"line_number":144,"context_line":""}],"source_content_type":"text/x-python","patch_set":6,"id":"1ae5cdf2_6b6c5f09","line":141,"in_reply_to":"1ae5cdf2_abf747d3","updated":"2014-06-17 16:23:32.000000000","message":"See line 59, it started out as a copy of the existing client method.","commit_id":"848805f21b30548832302345759b7d648e15d2be"}]}