)]}'
{"tempest/api/compute/servers/test_instance_actions_negative.py":[{"author":{"_account_id":5292,"name":"Christopher Yeoh","email":"cbkyeoh@gmail.com","username":"cyeoh-0"},"change_message_id":"ddabcf88255c75799d36f6809248dd4c1bdb87d9","unresolved":false,"context_lines":[{"line_number":28,"context_line":"        super(InstanceActionsNegativeTestJSON, cls).setUpClass()"},{"line_number":29,"context_line":"        cls.client \u003d cls.servers_client"},{"line_number":30,"context_line":"        resp, server \u003d cls.create_test_server(wait_until\u003d\u0027ACTIVE\u0027)"},{"line_number":31,"context_line":"        cls.request_id \u003d resp[\u0027x-compute-request-id\u0027]"},{"line_number":32,"context_line":"        cls.server_id \u003d server[\u0027id\u0027]"},{"line_number":33,"context_line":""},{"line_number":34,"context_line":"    @test.attr(type\u003d[\u0027negative\u0027, \u0027gate\u0027])"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAT3%2F%2F6AM%3D","line":31,"updated":"2013-12-27 12:12:36.000000000","message":"request_id is not needed for these tests","commit_id":"3bb2e99d628917827828232e353f09c5bff46987"},{"author":{"_account_id":7139,"name":"Zhi Kun Liu","username":"zhikunliu","inactive":true},"change_message_id":"4c5bf967f56b38fd48e3b1c1bcd85f803857b426","unresolved":false,"context_lines":[{"line_number":28,"context_line":"        super(InstanceActionsNegativeTestJSON, cls).setUpClass()"},{"line_number":29,"context_line":"        cls.client \u003d cls.servers_client"},{"line_number":30,"context_line":"        resp, server \u003d cls.create_test_server(wait_until\u003d\u0027ACTIVE\u0027)"},{"line_number":31,"context_line":"        cls.server_id \u003d server[\u0027id\u0027]"},{"line_number":32,"context_line":""},{"line_number":33,"context_line":"    @test.attr(type\u003d[\u0027negative\u0027, \u0027gate\u0027])"},{"line_number":34,"context_line":"    def test_list_instance_actions_invalid_server(self):"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAT3%2F%2FwJY%3D","line":31,"updated":"2014-01-02 11:16:12.000000000","message":"IMO, this test don\u0027t need create test server. they are not used in negative tests.","commit_id":"377153db87e30f118f414cbc465064dd1711b4b0"},{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"e77026d2701766bc920d81a86782539eb5f1500e","unresolved":false,"context_lines":[{"line_number":28,"context_line":"        super(InstanceActionsNegativeTestJSON, cls).setUpClass()"},{"line_number":29,"context_line":"        cls.client \u003d cls.servers_client"},{"line_number":30,"context_line":"        resp, server \u003d cls.create_test_server(wait_until\u003d\u0027ACTIVE\u0027)"},{"line_number":31,"context_line":"        cls.server_id \u003d server[\u0027id\u0027]"},{"line_number":32,"context_line":""},{"line_number":33,"context_line":"    @test.attr(type\u003d[\u0027negative\u0027, \u0027gate\u0027])"},{"line_number":34,"context_line":"    def test_list_instance_actions_invalid_server(self):"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAUH%2F%2F%2BFE%3D","line":31,"in_reply_to":"AAAAT3%2F%2FwJY%3D","updated":"2014-01-06 02:43:35.000000000","message":"Agreed, better to remove this server creation.","commit_id":"377153db87e30f118f414cbc465064dd1711b4b0"},{"author":{"_account_id":7039,"name":"Hoisaleshwara Madan V S","email":"hoisalvs@in.ibm.com","username":"Madan"},"change_message_id":"490f8f44ce14650c913b2f764a4fee1f376d7017","unresolved":false,"context_lines":[{"line_number":28,"context_line":"        super(InstanceActionsNegativeTestJSON, cls).setUpClass()"},{"line_number":29,"context_line":"        cls.client \u003d cls.servers_client"},{"line_number":30,"context_line":"        resp, server \u003d cls.create_test_server(wait_until\u003d\u0027ACTIVE\u0027)"},{"line_number":31,"context_line":"        cls.server_id \u003d server[\u0027id\u0027]"},{"line_number":32,"context_line":""},{"line_number":33,"context_line":"    @test.attr(type\u003d[\u0027negative\u0027, \u0027gate\u0027])"},{"line_number":34,"context_line":"    def test_list_instance_actions_invalid_server(self):"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAUH%2F%2F4mc%3D","line":31,"in_reply_to":"AAAAUH%2F%2F%2BFE%3D","updated":"2014-01-07 09:00:49.000000000","message":"Ken\u0027ichi Ohmichi, Zhi Kun Liu, Thanks for reviewing.\nin #L42 self.assertRaises(exceptions.NotFound, self.client.get_instance_action, self.server_id, \u0027999\u0027) checks for the non-existent instance action so server creation is needed.\n\nhttps://github.com/openstack/tempest/blob/master/tempest/services/compute/json/servers_client.py#L420\nI\u0027ll be keep this on hold, waiting for further reviews on the same","commit_id":"377153db87e30f118f414cbc465064dd1711b4b0"},{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"f2eb6ed69ff1239e8aaf1450b54a40729ed657e0","unresolved":false,"context_lines":[{"line_number":28,"context_line":"        super(InstanceActionsNegativeTestJSON, cls).setUpClass()"},{"line_number":29,"context_line":"        cls.client \u003d cls.servers_client"},{"line_number":30,"context_line":"        resp, server \u003d cls.create_test_server(wait_until\u003d\u0027ACTIVE\u0027)"},{"line_number":31,"context_line":"        cls.server_id \u003d server[\u0027id\u0027]"},{"line_number":32,"context_line":""},{"line_number":33,"context_line":"    @test.attr(type\u003d[\u0027negative\u0027, \u0027gate\u0027])"},{"line_number":34,"context_line":"    def test_list_instance_actions_invalid_server(self):"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAUH%2F%2Fzmo%3D","line":31,"in_reply_to":"AAAAUH%2F%2F4mc%3D","updated":"2014-01-08 08:33:16.000000000","message":"Thanks for your explanation, I see.","commit_id":"377153db87e30f118f414cbc465064dd1711b4b0"},{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"e77026d2701766bc920d81a86782539eb5f1500e","unresolved":false,"context_lines":[{"line_number":31,"context_line":"        cls.server_id \u003d server[\u0027id\u0027]"},{"line_number":32,"context_line":""},{"line_number":33,"context_line":"    @test.attr(type\u003d[\u0027negative\u0027, \u0027gate\u0027])"},{"line_number":34,"context_line":"    def test_list_instance_actions_invalid_server(self):"},{"line_number":35,"context_line":"        # List actions of the invalid server id"},{"line_number":36,"context_line":"        self.assertRaises(exceptions.NotFound,"},{"line_number":37,"context_line":"                          self.client.list_instance_actions, \u0027server-999\u0027)"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAUH%2F%2F%2BEw%3D","line":34,"updated":"2014-01-06 02:43:35.000000000","message":"s/invalid_server/nonexistent_server/","commit_id":"377153db87e30f118f414cbc465064dd1711b4b0"},{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"e77026d2701766bc920d81a86782539eb5f1500e","unresolved":false,"context_lines":[{"line_number":32,"context_line":""},{"line_number":33,"context_line":"    @test.attr(type\u003d[\u0027negative\u0027, \u0027gate\u0027])"},{"line_number":34,"context_line":"    def test_list_instance_actions_invalid_server(self):"},{"line_number":35,"context_line":"        # List actions of the invalid server id"},{"line_number":36,"context_line":"        self.assertRaises(exceptions.NotFound,"},{"line_number":37,"context_line":"                          self.client.list_instance_actions, \u0027server-999\u0027)"},{"line_number":38,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAUH%2F%2F%2BEs%3D","line":35,"updated":"2014-01-06 02:43:35.000000000","message":"s/invalid/non-existent/","commit_id":"377153db87e30f118f414cbc465064dd1711b4b0"},{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"e77026d2701766bc920d81a86782539eb5f1500e","unresolved":false,"context_lines":[{"line_number":34,"context_line":"    def test_list_instance_actions_invalid_server(self):"},{"line_number":35,"context_line":"        # List actions of the invalid server id"},{"line_number":36,"context_line":"        self.assertRaises(exceptions.NotFound,"},{"line_number":37,"context_line":"                          self.client.list_instance_actions, \u0027server-999\u0027)"},{"line_number":38,"context_line":""},{"line_number":39,"context_line":"    @test.attr(type\u003d[\u0027negative\u0027, \u0027gate\u0027])"},{"line_number":40,"context_line":"    def test_get_instance_action_invalid_request(self):"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAUH%2F%2F%2BE4%3D","line":37,"updated":"2014-01-06 02:43:35.000000000","message":"better to use data_utils.rand_uuid() instead of static value(\u0027server-999\u0027), because of the other negative tests(test_servers_negative.py etc.) and \u0027server-999\u0027 is not server-id format. server-id is uuid now.","commit_id":"377153db87e30f118f414cbc465064dd1711b4b0"}]}