)]}'
{"tempest/api/compute/admin/test_servers.py":[{"author":{"_account_id":6983,"name":"YangLei","email":"iamhappg@aliyun.com","username":"YangLei"},"change_message_id":"d45d486fe8318e408bd2f873dd37e59ea9acda29","unresolved":false,"context_lines":[{"line_number":180,"context_line":"                          self.non_adm_client.get_server_diagnostics,"},{"line_number":181,"context_line":"                          self.s1_id)"},{"line_number":182,"context_line":""},{"line_number":183,"context_line":"    @attr(type\u003d[\u0027negative\u0027, \u0027gate\u0027])"},{"line_number":184,"context_line":"    def test_migrate_non_existent_server(self):"},{"line_number":185,"context_line":"        # migrate a non existent server"},{"line_number":186,"context_line":"        self.assertRaises(exceptions.NotFound,"}],"source_content_type":"text/x-python","patch_set":6,"id":"AAAATn%2F%2F1bA%3D","line":183,"updated":"2013-10-28 09:07:22.000000000","message":"We need to add negative testcase to the negative file.\nPlease follow the guild http://docs.openstack.org/developer/tempest/HACKING.html#negative-tests","commit_id":"92607957fa9c0b9c25670708615826dd8110b2b5"},{"author":{"_account_id":8289,"name":"huangtianhua","email":"tiantian223@163.com","username":"huangtianhua"},"change_message_id":"b99b36135b10ebcc8ff8211678844499632c5c44","unresolved":false,"context_lines":[{"line_number":180,"context_line":"                          self.non_adm_client.get_server_diagnostics,"},{"line_number":181,"context_line":"                          self.s1_id)"},{"line_number":182,"context_line":""},{"line_number":183,"context_line":"    @attr(type\u003d[\u0027negative\u0027, \u0027gate\u0027])"},{"line_number":184,"context_line":"    def test_migrate_non_existent_server(self):"},{"line_number":185,"context_line":"        # migrate a non existent server"},{"line_number":186,"context_line":"        self.assertRaises(exceptions.NotFound,"}],"source_content_type":"text/x-python","patch_set":6,"id":"AAAATn%2F%2Fxz8%3D","line":183,"in_reply_to":"AAAATn%2F%2F1bA%3D","updated":"2013-10-29 07:36:03.000000000","message":"I will modify it.","commit_id":"92607957fa9c0b9c25670708615826dd8110b2b5"},{"author":{"_account_id":6983,"name":"YangLei","email":"iamhappg@aliyun.com","username":"YangLei"},"change_message_id":"d45d486fe8318e408bd2f873dd37e59ea9acda29","unresolved":false,"context_lines":[{"line_number":187,"context_line":"                          self.client.migrate_server,"},{"line_number":188,"context_line":"                          str(uuid.uuid4()))"},{"line_number":189,"context_line":""},{"line_number":190,"context_line":"    @attr(type\u003d[\u0027negative\u0027, \u0027gate\u0027])"},{"line_number":191,"context_line":"    def test_migrate_server_invalid_state(self):"},{"line_number":192,"context_line":"        # create server."},{"line_number":193,"context_line":"        resp, server \u003d self.create_server(wait_until\u003d\u0027ACTIVE\u0027)"}],"source_content_type":"text/x-python","patch_set":6,"id":"AAAATn%2F%2F1a0%3D","line":190,"updated":"2013-10-28 09:07:22.000000000","message":"ditto","commit_id":"92607957fa9c0b9c25670708615826dd8110b2b5"},{"author":{"_account_id":8289,"name":"huangtianhua","email":"tiantian223@163.com","username":"huangtianhua"},"change_message_id":"bd22839a670f4134a43fe37d10819d28d5b4f3cc","unresolved":false,"context_lines":[{"line_number":187,"context_line":"                          self.client.migrate_server,"},{"line_number":188,"context_line":"                          str(uuid.uuid4()))"},{"line_number":189,"context_line":""},{"line_number":190,"context_line":"    @attr(type\u003d[\u0027negative\u0027, \u0027gate\u0027])"},{"line_number":191,"context_line":"    def test_migrate_server_invalid_state(self):"},{"line_number":192,"context_line":"        # create server."},{"line_number":193,"context_line":"        resp, server \u003d self.create_server(wait_until\u003d\u0027ACTIVE\u0027)"}],"source_content_type":"text/x-python","patch_set":6,"id":"AAAATn%2F%2F1JE%3D","line":190,"in_reply_to":"AAAATn%2F%2F1a0%3D","updated":"2013-10-28 10:26:12.000000000","message":"ditto","commit_id":"92607957fa9c0b9c25670708615826dd8110b2b5"}],"tempest/api/compute/servers/test_servers_negative.py":[{"author":{"_account_id":6983,"name":"YangLei","email":"iamhappg@aliyun.com","username":"YangLei"},"change_message_id":"d45d486fe8318e408bd2f873dd37e59ea9acda29","unresolved":false,"context_lines":[{"line_number":21,"context_line":""},{"line_number":22,"context_line":"from tempest.api.compute import base"},{"line_number":23,"context_line":"from tempest import clients"},{"line_number":24,"context_line":"from tempest.common.utils import data_utils"},{"line_number":25,"context_line":"from tempest import exceptions"},{"line_number":26,"context_line":"from tempest.test import attr"},{"line_number":27,"context_line":""}],"source_content_type":"text/x-python","patch_set":6,"id":"AAAATn%2F%2F1aM%3D","line":24,"updated":"2013-10-28 09:07:22.000000000","message":"I really do not know why do you change this file?\n\nAnd I see about 90 file use \"from tempest.common.utils.data_utils import rand_name\". If we has to change it, I think all the files need to be changed.\n\nyanglei@yanglei-ThinkCentre-M58:~/community/tempest$ grep -R \"from tempest.common.utils.data_utils import rand_name\" . | wc -l\n90\nyanglei@yanglei-ThinkCentre-M58:~/community/tempest$","commit_id":"92607957fa9c0b9c25670708615826dd8110b2b5"},{"author":{"_account_id":8289,"name":"huangtianhua","email":"tiantian223@163.com","username":"huangtianhua"},"change_message_id":"bd22839a670f4134a43fe37d10819d28d5b4f3cc","unresolved":false,"context_lines":[{"line_number":21,"context_line":""},{"line_number":22,"context_line":"from tempest.api.compute import base"},{"line_number":23,"context_line":"from tempest import clients"},{"line_number":24,"context_line":"from tempest.common.utils import data_utils"},{"line_number":25,"context_line":"from tempest import exceptions"},{"line_number":26,"context_line":"from tempest.test import attr"},{"line_number":27,"context_line":""}],"source_content_type":"text/x-python","patch_set":6,"id":"AAAATn%2F%2F1Iw%3D","line":24,"in_reply_to":"AAAATn%2F%2F1aM%3D","updated":"2013-10-28 10:26:12.000000000","message":"See the patch:https://review.openstack.org/#/c/46886/\nAlso:https://review.openstack.org/#/c/46886/9/tempest/api/compute/floating_ips/test_floating_ips_actions.py","commit_id":"92607957fa9c0b9c25670708615826dd8110b2b5"},{"author":{"_account_id":8625,"name":"Anju Tiwari","username":"Anju5","inactive":true},"change_message_id":"fdcc60bb046728a0e986198229fea979f238c9fd","unresolved":false,"context_lines":[{"line_number":21,"context_line":""},{"line_number":22,"context_line":"from tempest.api.compute import base"},{"line_number":23,"context_line":"from tempest import clients"},{"line_number":24,"context_line":"from tempest.common.utils import data_utils"},{"line_number":25,"context_line":"from tempest import exceptions"},{"line_number":26,"context_line":"from tempest.test import attr"},{"line_number":27,"context_line":""}],"source_content_type":"text/x-python","patch_set":9,"id":"AAAATn%2F%2FteE%3D","line":24,"updated":"2013-10-30 05:24:01.000000000","message":"you did not mentioned this in the commit message and I think this change should be  in a different patch.","commit_id":"5f0100b9fb2bed6734b14a42678dc5137bd9cf8d"}]}