)]}'
{"tempest/api/compute/admin/test_servers.py":[{"author":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"change_message_id":"62641c35eb996bc7b0026a4ddb1c4ffed29e9fc2","unresolved":false,"context_lines":[{"line_number":106,"context_line":"        self.assertNotIn(self.s1_name, servers_name)"},{"line_number":107,"context_line":"        self.assertNotIn(self.s2_name, servers_name)"},{"line_number":108,"context_line":""},{"line_number":109,"context_line":"        params \u003d {\u0027all_tenants\u0027: \u0027\u0027, \u0027tenant_id\u0027: tenant_id}"},{"line_number":110,"context_line":"        body \u003d self.client.list_servers(detail\u003dTrue, **params)"},{"line_number":111,"context_line":"        servers \u003d body[\u0027servers\u0027]"},{"line_number":112,"context_line":"        servers_name \u003d map(lambda x: x[\u0027name\u0027], servers)"}],"source_content_type":"text/x-python","patch_set":1,"id":"3a98d1f4_20c22634","line":109,"updated":"2016-09-28 21:16:56.000000000","message":"What does this add? we do the same thing below just with a different tenant id. I\u0027m not sure we gain anything by doing this twice with different tenant ids","commit_id":"93625b92ac8bba12e5afd2f496a32f3779f7353d"},{"author":{"_account_id":22692,"name":"guo yunxian","email":"yunxian.guo@easystack.cn","username":"guoyunxian"},"change_message_id":"1b0444f764bdf34a40ee9a5c8f1b2362eb84c614","unresolved":false,"context_lines":[{"line_number":106,"context_line":"        self.assertNotIn(self.s1_name, servers_name)"},{"line_number":107,"context_line":"        self.assertNotIn(self.s2_name, servers_name)"},{"line_number":108,"context_line":""},{"line_number":109,"context_line":"        params \u003d {\u0027all_tenants\u0027: \u0027\u0027, \u0027tenant_id\u0027: tenant_id}"},{"line_number":110,"context_line":"        body \u003d self.client.list_servers(detail\u003dTrue, **params)"},{"line_number":111,"context_line":"        servers \u003d body[\u0027servers\u0027]"},{"line_number":112,"context_line":"        servers_name \u003d map(lambda x: x[\u0027name\u0027], servers)"}],"source_content_type":"text/x-python","patch_set":1,"id":"3a98d1f4_2a183c84","line":109,"in_reply_to":"3a98d1f4_20c22634","updated":"2016-09-29 03:13:09.000000000","message":"s1 and s2 belong to primary tenant. We have tests that list the primary tenant but get nothing due to odd specified behavior and tests that can not list other tenants. But we do not test that we can list primary tenant with all_tenants specified. So I add this.","commit_id":"93625b92ac8bba12e5afd2f496a32f3779f7353d"},{"author":{"_account_id":7350,"name":"Jordan Pittier","email":"jordan.pittier@gmail.com","username":"JordanP"},"change_message_id":"a469bbaacd8b388ae0a096bc79effc9c7a30605e","unresolved":false,"context_lines":[{"line_number":106,"context_line":"        self.assertNotIn(self.s1_name, servers_name)"},{"line_number":107,"context_line":"        self.assertNotIn(self.s2_name, servers_name)"},{"line_number":108,"context_line":""},{"line_number":109,"context_line":"        # List the primary tenant with with all_tenants is specified"},{"line_number":110,"context_line":"        params \u003d {\u0027all_tenants\u0027: \u0027\u0027, \u0027tenant_id\u0027: tenant_id}"},{"line_number":111,"context_line":"        body \u003d self.client.list_servers(detail\u003dTrue, **params)"},{"line_number":112,"context_line":"        servers \u003d body[\u0027servers\u0027]"}],"source_content_type":"text/x-python","patch_set":2,"id":"3a98d1f4_416d6297","line":109,"updated":"2016-09-29 08:34:59.000000000","message":"\"with with all_tenants\" ? Looks like a typo.","commit_id":"42c282fa0a5763b1669158323eb3fc04e090a834"}]}
