)]}'
{"tempest/api/compute/admin/test_quotas.py":[{"author":{"_account_id":8846,"name":"Hao Wang","display_name":"Hao Wang","email":"sxmatch1986@gmail.com","username":"sxmatch"},"change_message_id":"ba7a2da3828d003c5c572a60b700359d24a1fae4","unresolved":false,"context_lines":[{"line_number":191,"context_line":"        # Create a server"},{"line_number":192,"context_line":"        resp, server \u003d self.create_server(wait_until\u003d\u0027ACTIVE\u0027)"},{"line_number":193,"context_line":"        self.assertEqual(202, resp.status)"},{"line_number":194,"context_line":""},{"line_number":195,"context_line":"        self.assertRaises(exceptions.BadRequest,"},{"line_number":196,"context_line":"                          self.adm_client.update_quota_set,"},{"line_number":197,"context_line":"                          self.demo_tenant_id,"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAATH%2F%2F2L0%3D","line":194,"updated":"2013-10-09 02:11:31.000000000","message":"This server should be deleted when this case is over.","commit_id":"e3965917e4086869cc283268d81fe06158ecf561"},{"author":{"_account_id":8846,"name":"Hao Wang","display_name":"Hao Wang","email":"sxmatch1986@gmail.com","username":"sxmatch"},"change_message_id":"3511e0a540bf5bc380bd3e2bbb7dc51b413a3601","unresolved":false,"context_lines":[{"line_number":191,"context_line":"        # Create a server"},{"line_number":192,"context_line":"        resp, server \u003d self.create_server(wait_until\u003d\u0027ACTIVE\u0027)"},{"line_number":193,"context_line":"        self.assertEqual(202, resp.status)"},{"line_number":194,"context_line":""},{"line_number":195,"context_line":"        self.assertRaises(exceptions.BadRequest,"},{"line_number":196,"context_line":"                          self.adm_client.update_quota_set,"},{"line_number":197,"context_line":"                          self.demo_tenant_id,"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAATH%2F%2F1co%3D","line":194,"in_reply_to":"AAAATH%2F%2F2HE%3D","updated":"2013-10-09 07:29:30.000000000","message":"well, i figure out.","commit_id":"e3965917e4086869cc283268d81fe06158ecf561"},{"author":{"_account_id":6732,"name":"Lingxian Kong","email":"anlin.kong@gmail.com","username":"kong"},"change_message_id":"c26eb315b8c3a0f9208817f73b444fafdc680afa","unresolved":false,"context_lines":[{"line_number":191,"context_line":"        # Create a server"},{"line_number":192,"context_line":"        resp, server \u003d self.create_server(wait_until\u003d\u0027ACTIVE\u0027)"},{"line_number":193,"context_line":"        self.assertEqual(202, resp.status)"},{"line_number":194,"context_line":""},{"line_number":195,"context_line":"        self.assertRaises(exceptions.BadRequest,"},{"line_number":196,"context_line":"                          self.adm_client.update_quota_set,"},{"line_number":197,"context_line":"                          self.demo_tenant_id,"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAATH%2F%2F2HE%3D","line":194,"in_reply_to":"AAAATH%2F%2F2L0%3D","updated":"2013-10-09 02:54:25.000000000","message":"Hi wanghao:\nThe deleting action is made in \u0027tearDownClass\u0027 function, there is no need to perform deleting in every test case. make sense?","commit_id":"e3965917e4086869cc283268d81fe06158ecf561"},{"author":{"_account_id":5586,"name":"Ivan-Zhu","email":"zhubo198511@gmail.com","username":"ivan-zhu"},"change_message_id":"c0eec113c1a697a7b74e0f63a990a064f1e9456f","unresolved":false,"context_lines":[{"line_number":166,"context_line":"                        tenant_id)"},{"line_number":167,"context_line":""},{"line_number":168,"context_line":"        resp, body \u003d self.adm_client.delete_quota_set(tenant_id)"},{"line_number":169,"context_line":"        self.assertEqual(202, resp.status)"},{"line_number":170,"context_line":""},{"line_number":171,"context_line":""},{"line_number":172,"context_line":"class QuotasAdminTestXML(QuotasAdminTestJSON):"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAATH%2F%2FaZ8%3D","line":169,"updated":"2013-10-16 14:40:57.000000000","message":"Can we check it has been really deleted using get_quota_set? Maybe the response will depend on the QuotaDriver, can you check it?","commit_id":"2ffbd06af108363cdbcca787d471f537f093bbcc"},{"author":{"_account_id":6732,"name":"Lingxian Kong","email":"anlin.kong@gmail.com","username":"kong"},"change_message_id":"fcf30c6ed283498bc81b7dd1ed92e6eeda24b93f","unresolved":false,"context_lines":[{"line_number":166,"context_line":"                        tenant_id)"},{"line_number":167,"context_line":""},{"line_number":168,"context_line":"        resp, body \u003d self.adm_client.delete_quota_set(tenant_id)"},{"line_number":169,"context_line":"        self.assertEqual(202, resp.status)"},{"line_number":170,"context_line":""},{"line_number":171,"context_line":""},{"line_number":172,"context_line":"class QuotasAdminTestXML(QuotasAdminTestJSON):"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAATH%2F%2FTUk%3D","line":169,"in_reply_to":"AAAATH%2F%2FaW8%3D","updated":"2013-10-18 04:11:47.000000000","message":"@lvan, your idea? need your +1.","commit_id":"2ffbd06af108363cdbcca787d471f537f093bbcc"},{"author":{"_account_id":6732,"name":"Lingxian Kong","email":"anlin.kong@gmail.com","username":"kong"},"change_message_id":"60f72373a6acb236b06919a8084ec07a2c3ad39c","unresolved":false,"context_lines":[{"line_number":166,"context_line":"                        tenant_id)"},{"line_number":167,"context_line":""},{"line_number":168,"context_line":"        resp, body \u003d self.adm_client.delete_quota_set(tenant_id)"},{"line_number":169,"context_line":"        self.assertEqual(202, resp.status)"},{"line_number":170,"context_line":""},{"line_number":171,"context_line":""},{"line_number":172,"context_line":"class QuotasAdminTestXML(QuotasAdminTestJSON):"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAATH%2F%2FaW8%3D","line":169,"in_reply_to":"AAAATH%2F%2FaZ8%3D","updated":"2013-10-16 14:51:26.000000000","message":"hi, lvan, please see https://github.com/openstack/nova/blob/master/nova/api/openstack/compute/contrib/quotas.py#L228","commit_id":"2ffbd06af108363cdbcca787d471f537f093bbcc"}],"tempest/services/compute/json/quotas_client.py":[{"author":{"_account_id":5586,"name":"Ivan-Zhu","email":"zhubo198511@gmail.com","username":"ivan-zhu"},"change_message_id":"c0eec113c1a697a7b74e0f63a990a064f1e9456f","unresolved":false,"context_lines":[{"line_number":29,"context_line":""},{"line_number":30,"context_line":"    def get_quota_set(self, tenant_id):"},{"line_number":31,"context_line":"        \"\"\"List the quota set for a tenant.\"\"\""},{"line_number":32,"context_line":""},{"line_number":33,"context_line":"        url \u003d \u0027os-quota-sets/%s\u0027 % str(tenant_id)"},{"line_number":34,"context_line":"        resp, body \u003d self.get(url)"},{"line_number":35,"context_line":"        body \u003d json.loads(body)"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAATH%2F%2FaZs%3D","line":32,"updated":"2013-10-16 14:40:57.000000000","message":"QUERY_STRING can also be here, we can a list of resources, retrieve the quotas for the given user and project. maybe we should test it.","commit_id":"2ffbd06af108363cdbcca787d471f537f093bbcc"},{"author":{"_account_id":5586,"name":"Ivan-Zhu","email":"zhubo198511@gmail.com","username":"ivan-zhu"},"change_message_id":"c0eec113c1a697a7b74e0f63a990a064f1e9456f","unresolved":false,"context_lines":[{"line_number":105,"context_line":"    def delete_quota_set(self, tenant_id):"},{"line_number":106,"context_line":"        \"\"\"Delete the quota set for a tenant.\"\"\""},{"line_number":107,"context_line":""},{"line_number":108,"context_line":"        url \u003d \u0027os-quota-sets/%s\u0027 % str(tenant_id)"},{"line_number":109,"context_line":"        return self.delete(url)"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAATH%2F%2FaZQ%3D","line":108,"updated":"2013-10-16 14:40:57.000000000","message":"If the extension os-user-quotas is loaded, we can destroy all quotas, usages, and reservations associated with a project and user. so maybe we need add QUERY_STRING in the url and test that case.","commit_id":"2ffbd06af108363cdbcca787d471f537f093bbcc"},{"author":{"_account_id":6732,"name":"Lingxian Kong","email":"anlin.kong@gmail.com","username":"kong"},"change_message_id":"60f72373a6acb236b06919a8084ec07a2c3ad39c","unresolved":false,"context_lines":[{"line_number":105,"context_line":"    def delete_quota_set(self, tenant_id):"},{"line_number":106,"context_line":"        \"\"\"Delete the quota set for a tenant.\"\"\""},{"line_number":107,"context_line":""},{"line_number":108,"context_line":"        url \u003d \u0027os-quota-sets/%s\u0027 % str(tenant_id)"},{"line_number":109,"context_line":"        return self.delete(url)"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAATH%2F%2FaVQ%3D","line":108,"in_reply_to":"AAAATH%2F%2FaZQ%3D","updated":"2013-10-16 14:51:26.000000000","message":"hi lvan, you\u0027r totally right, but I prefer that we do it in another patch for optimization, right?","commit_id":"2ffbd06af108363cdbcca787d471f537f093bbcc"}]}