)]}'
{"tempest/api/volume/test_volumes_negative.py":[{"author":{"_account_id":7350,"name":"Jordan Pittier","email":"jordan.pittier@gmail.com","username":"JordanP"},"change_message_id":"e745be4e2539a0a4adfc137bdbb48aab55488a84","unresolved":false,"context_lines":[{"line_number":181,"context_line":"        server \u003d self.create_server("},{"line_number":182,"context_line":"            name\u003dsrv_name,"},{"line_number":183,"context_line":"            wait_until\u003d\u0027ACTIVE\u0027)"},{"line_number":184,"context_line":"        self.addCleanup(waiters.wait_for_server_termination,"},{"line_number":185,"context_line":"                        self.servers_client, server[\u0027id\u0027])"},{"line_number":186,"context_line":"        self.addCleanup(self.servers_client.delete_server, server[\u0027id\u0027])"},{"line_number":187,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"9a89bdaa_6f010205","side":"PARENT","line":184,"updated":"2016-09-07 11:01:58.000000000","message":"This line was added because of https://bugs.launchpad.net/tempest/+bug/1471712\n\nI am afraid that this bug will show up again if we remove it. I think the \"create_server\" method should take an extra argument \"wait_for_deletion\u003dFalse\" and if it set to True then we would add the call to self.addCleanup(waiters.wait_for_server_termination(...)\n\nWhat do you think ?","commit_id":"c59270760a9e44f4c1f2c37bc3a1bb6222cc19b6"},{"author":{"_account_id":19262,"name":"Liron Kuchlani","email":"lkuchlan@redhat.com","username":"lkuchlan"},"change_message_id":"a97882234b9c2f3a861d374940f58694b49be7f2","unresolved":false,"context_lines":[{"line_number":181,"context_line":"        server \u003d self.create_server("},{"line_number":182,"context_line":"            name\u003dsrv_name,"},{"line_number":183,"context_line":"            wait_until\u003d\u0027ACTIVE\u0027)"},{"line_number":184,"context_line":"        self.addCleanup(waiters.wait_for_server_termination,"},{"line_number":185,"context_line":"                        self.servers_client, server[\u0027id\u0027])"},{"line_number":186,"context_line":"        self.addCleanup(self.servers_client.delete_server, server[\u0027id\u0027])"},{"line_number":187,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"9a89bdaa_91182877","side":"PARENT","line":184,"in_reply_to":"9a89bdaa_6f010205","updated":"2016-09-07 11:55:51.000000000","message":"I agree with you, to prevent future issues, I\u0027ll add \u0027wait_for_deletion\u0027 argument to the method","commit_id":"c59270760a9e44f4c1f2c37bc3a1bb6222cc19b6"}]}