)]}'
{"tempest/api/compute/v3/admin/test_servers.py":[{"author":{"_account_id":5586,"name":"Ivan-Zhu","email":"zhubo198511@gmail.com","username":"ivan-zhu"},"change_message_id":"93e7e2dac157443930f36c51806bba065c5d9d92","unresolved":false,"context_lines":[{"line_number":32,"context_line":"    @classmethod"},{"line_number":33,"context_line":"    def setUpClass(cls):"},{"line_number":34,"context_line":"        super(ServersAdminV3TestJSON, cls).setUpClass()"},{"line_number":35,"context_line":"        cls.client \u003d cls.os_adm.servers_client"},{"line_number":36,"context_line":"        cls.non_admin_client \u003d cls.servers_client"},{"line_number":37,"context_line":"        cls.flavors_client \u003d cls.os_adm.flavors_client"},{"line_number":38,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAATn%2F9V7I%3D","line":35,"updated":"2013-12-12 13:36:37.000000000","message":"We should use the cls.servers_admin_client. should not access the os_adm directly, Sorry. I should update the os_adm to _os_adm.\nThis client use the v2 client by mistake.","commit_id":"d684a78a3918d71702926ef8d0942b5165abb7e8"},{"author":{"_account_id":7139,"name":"Zhi Kun Liu","username":"zhikunliu","inactive":true},"change_message_id":"29b8349f2984045d135e5be161e6a294b6e7bc7c","unresolved":false,"context_lines":[{"line_number":32,"context_line":"    @classmethod"},{"line_number":33,"context_line":"    def setUpClass(cls):"},{"line_number":34,"context_line":"        super(ServersAdminV3TestJSON, cls).setUpClass()"},{"line_number":35,"context_line":"        cls.client \u003d cls.os_adm.servers_client"},{"line_number":36,"context_line":"        cls.non_admin_client \u003d cls.servers_client"},{"line_number":37,"context_line":"        cls.flavors_client \u003d cls.os_adm.flavors_client"},{"line_number":38,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAATn%2F9V4A%3D","line":35,"in_reply_to":"AAAATn%2F9V7I%3D","updated":"2013-12-12 13:47:11.000000000","message":"Ivan, thank you very much! I\u0027ll fix it in next patch set.","commit_id":"d684a78a3918d71702926ef8d0942b5165abb7e8"},{"author":{"_account_id":5586,"name":"Ivan-Zhu","email":"zhubo198511@gmail.com","username":"ivan-zhu"},"change_message_id":"08a9ebca70b280d23fb64e71ae306c7b602a5433","unresolved":false,"context_lines":[{"line_number":32,"context_line":"    @classmethod"},{"line_number":33,"context_line":"    def setUpClass(cls):"},{"line_number":34,"context_line":"        super(ServersAdminV3TestJSON, cls).setUpClass()"},{"line_number":35,"context_line":"        cls.client \u003d cls.os_adm.servers_client"},{"line_number":36,"context_line":"        cls.non_admin_client \u003d cls.servers_client"},{"line_number":37,"context_line":"        cls.flavors_client \u003d cls.os_adm.flavors_client"},{"line_number":38,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAATn%2F9V5o%3D","line":35,"in_reply_to":"AAAATn%2F9V7I%3D","updated":"2013-12-12 13:38:38.000000000","message":"the v3 client is cls.os_adm.servers_v3_client.","commit_id":"d684a78a3918d71702926ef8d0942b5165abb7e8"},{"author":{"_account_id":5586,"name":"Ivan-Zhu","email":"zhubo198511@gmail.com","username":"ivan-zhu"},"change_message_id":"93e7e2dac157443930f36c51806bba065c5d9d92","unresolved":false,"context_lines":[{"line_number":34,"context_line":"        super(ServersAdminV3TestJSON, cls).setUpClass()"},{"line_number":35,"context_line":"        cls.client \u003d cls.os_adm.servers_client"},{"line_number":36,"context_line":"        cls.non_admin_client \u003d cls.servers_client"},{"line_number":37,"context_line":"        cls.flavors_client \u003d cls.os_adm.flavors_client"},{"line_number":38,"context_line":""},{"line_number":39,"context_line":"        cls.s1_name \u003d data_utils.rand_name(\u0027server\u0027)"},{"line_number":40,"context_line":"        resp, server \u003d cls.create_test_server(name\u003dcls.s1_name,"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAATn%2F9V7A%3D","line":37,"updated":"2013-12-12 13:36:37.000000000","message":"ditto","commit_id":"d684a78a3918d71702926ef8d0942b5165abb7e8"},{"author":{"_account_id":7139,"name":"Zhi Kun Liu","username":"zhikunliu","inactive":true},"change_message_id":"66a35d242181817f361bea01b97a530af8ea6903","unresolved":false,"context_lines":[{"line_number":34,"context_line":"        super(ServersAdminV3TestJSON, cls).setUpClass()"},{"line_number":35,"context_line":"        cls.client \u003d cls.os_adm.servers_client"},{"line_number":36,"context_line":"        cls.non_admin_client \u003d cls.servers_client"},{"line_number":37,"context_line":"        cls.flavors_client \u003d cls.os_adm.flavors_client"},{"line_number":38,"context_line":""},{"line_number":39,"context_line":"        cls.s1_name \u003d data_utils.rand_name(\u0027server\u0027)"},{"line_number":40,"context_line":"        resp, server \u003d cls.create_test_server(name\u003dcls.s1_name,"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAATn%2F9Vvk%3D","line":37,"in_reply_to":"AAAATn%2F9V7A%3D","updated":"2013-12-12 14:08:27.000000000","message":"there\u0027s no admin flavors client in BaseV3ComputeAdminTest. now, in other admin test files also use os_adm to get some admin client. So we could fix this problem in another patch together. is it fine for you?","commit_id":"d684a78a3918d71702926ef8d0942b5165abb7e8"}],"tempest/api/compute/v3/admin/test_servers_negative.py":[{"author":{"_account_id":5803,"name":"Attila Fazekas","email":"afazekas@redhat.com","username":"afazekas"},"change_message_id":"e926fd1d53ecb1669051f05ec934dd498eb54206","unresolved":false,"context_lines":[{"line_number":33,"context_line":"    @classmethod"},{"line_number":34,"context_line":"    def setUpClass(cls):"},{"line_number":35,"context_line":"        super(ServersAdminNegativeV3TestJSON, cls).setUpClass()"},{"line_number":36,"context_line":"        cls.client \u003d cls.servers_admin_client"},{"line_number":37,"context_line":"        cls.non_adm_client \u003d cls.servers_client"},{"line_number":38,"context_line":"        cls.flavors_client \u003d cls.flavors_admin_client"},{"line_number":39,"context_line":"        cls.identity_client \u003d cls._get_identity_admin_client()"}],"source_content_type":"text/x-python","patch_set":10,"id":"AAAAUH%2F%2F8aw%3D","line":36,"updated":"2014-01-06 12:50:14.000000000","message":"We keep too many variables on cls, I would prefer to use just os and os_adm or another variable with another user if necessary.","commit_id":"de89272a1c4fbac9bafbb90c3a381e691b5544fd"}]}