)]}'
{"tempest/api/compute/servers/test_delete_server.py":[{"author":{"_account_id":97,"name":"Daryl Walleck","email":"daryl.walleck@rackspace.com","username":"dwalleck"},"change_message_id":"0adae2d7dc9950231ba8d95270601cdaae1c9963","unresolved":false,"context_lines":[{"line_number":85,"context_line":"    @testtools.skipIf(not CONF.compute_feature_enabled.resize,"},{"line_number":86,"context_line":"                      \u0027Resize not available.\u0027)"},{"line_number":87,"context_line":"    @test.attr(type\u003d\u0027gate\u0027)"},{"line_number":88,"context_line":"    def test_delete_server_while_in_verify_resize_state(self):"},{"line_number":89,"context_line":"        # Delete a server while it\u0027s VM state is VERIFY_RESIZE"},{"line_number":90,"context_line":"        resp, server \u003d self.create_test_server(wait_until\u003d\u0027ACTIVE\u0027)"},{"line_number":91,"context_line":"        resp, body \u003d self.client.resize(server[\u0027id\u0027], self.flavor_ref_alt)"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAVX%2F%2FqOI%3D","line":88,"updated":"2014-03-01 17:58:26.000000000","message":"This test only proves deletion by the initial delete response code. Wouldn\u0027t you also want to check either by listing or performing a get on the server that the server was actually deleted?","commit_id":"f9a85758458a184447aa703633be56717faf080d"},{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"37c135262c416bea32a709b3384a3a2613e7dee7","unresolved":false,"context_lines":[{"line_number":85,"context_line":"    @testtools.skipIf(not CONF.compute_feature_enabled.resize,"},{"line_number":86,"context_line":"                      \u0027Resize not available.\u0027)"},{"line_number":87,"context_line":"    @test.attr(type\u003d\u0027gate\u0027)"},{"line_number":88,"context_line":"    def test_delete_server_while_in_verify_resize_state(self):"},{"line_number":89,"context_line":"        # Delete a server while it\u0027s VM state is VERIFY_RESIZE"},{"line_number":90,"context_line":"        resp, server \u003d self.create_test_server(wait_until\u003d\u0027ACTIVE\u0027)"},{"line_number":91,"context_line":"        resp, body \u003d self.client.resize(server[\u0027id\u0027], self.flavor_ref_alt)"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAVn%2F%2F%2B4M%3D","line":88,"in_reply_to":"AAAAVX%2F%2FqLc%3D","updated":"2014-03-03 00:34:11.000000000","message":"@Daryl\n\nThanks for good point. Now Tempest checks that the server was actually deleted in the latest code, so I will add wait_for_resource_deletion() as Matthew said.","commit_id":"f9a85758458a184447aa703633be56717faf080d"},{"author":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"change_message_id":"c257e38984104a0afbb2008903a073901f5171f4","unresolved":false,"context_lines":[{"line_number":85,"context_line":"    @testtools.skipIf(not CONF.compute_feature_enabled.resize,"},{"line_number":86,"context_line":"                      \u0027Resize not available.\u0027)"},{"line_number":87,"context_line":"    @test.attr(type\u003d\u0027gate\u0027)"},{"line_number":88,"context_line":"    def test_delete_server_while_in_verify_resize_state(self):"},{"line_number":89,"context_line":"        # Delete a server while it\u0027s VM state is VERIFY_RESIZE"},{"line_number":90,"context_line":"        resp, server \u003d self.create_test_server(wait_until\u003d\u0027ACTIVE\u0027)"},{"line_number":91,"context_line":"        resp, body \u003d self.client.resize(server[\u0027id\u0027], self.flavor_ref_alt)"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAVX%2F%2FqLc%3D","line":88,"in_reply_to":"AAAAVX%2F%2FqOI%3D","updated":"2014-03-01 18:21:10.000000000","message":"This is consistent with the rest of the tests in this file. (I was going to leave a similar comment before I noticed that)\n\nMy guess is that the wait_for_resource_deletion() call is specifically excluded in this file because of execute time concerns. But it would definitely be a TODO moving forward to investigate if we can add that without adding too much run time to the gate.","commit_id":"f9a85758458a184447aa703633be56717faf080d"}]}