)]}'
{"tempest/api/compute/servers/test_list_server_filters.py":[{"author":{"_account_id":11075,"name":"Benny Kopilov","email":"bkopilov@redhat.com","username":"bkopilov"},"change_message_id":"62f56c059dcdf76c76bf5acf8667b585f18c98b8","unresolved":false,"context_lines":[{"line_number":63,"context_line":"                                        wait_until\u003d\u0027ACTIVE\u0027)"},{"line_number":64,"context_line":""},{"line_number":65,"context_line":"        waiters.wait_for_server_status(cls.client, cls.s1[\u0027id\u0027],"},{"line_number":66,"context_line":"                                       \u0027ACTIVE\u0027)"},{"line_number":67,"context_line":"        waiters.wait_for_server_status(cls.client, cls.s2[\u0027id\u0027],"},{"line_number":68,"context_line":"                                       \u0027ACTIVE\u0027)"},{"line_number":69,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"dfeb2761_bfddb715","line":66,"range":{"start_line":66,"start_character":0,"end_line":66,"end_character":39},"updated":"2017-04-02 08:34:07.000000000","message":"Hi , \nHow do you know that you should wait for s1 first , and not wait for v2 .\nIsnt better to use here threads and join at the end ?\n\nYour hook does look so nice for me :) \n\nBenny","commit_id":"9b8a367472707aa5e3ac3e7f327a142fbf356688"},{"author":{"_account_id":7350,"name":"Jordan Pittier","email":"jordan.pittier@gmail.com","username":"JordanP"},"change_message_id":"eb446c277be49778942d50397015a1dc63758ab4","unresolved":false,"context_lines":[{"line_number":63,"context_line":"                                        wait_until\u003d\u0027ACTIVE\u0027)"},{"line_number":64,"context_line":""},{"line_number":65,"context_line":"        waiters.wait_for_server_status(cls.client, cls.s1[\u0027id\u0027],"},{"line_number":66,"context_line":"                                       \u0027ACTIVE\u0027)"},{"line_number":67,"context_line":"        waiters.wait_for_server_status(cls.client, cls.s2[\u0027id\u0027],"},{"line_number":68,"context_line":"                                       \u0027ACTIVE\u0027)"},{"line_number":69,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"dfeb2761_47b6d4fc","line":66,"range":{"start_line":66,"start_character":0,"end_line":66,"end_character":39},"in_reply_to":"dfeb2761_bfddb715","updated":"2017-04-03 09:09:46.000000000","message":"It doesn\u0027t matter which one finishes first. The point of this patch is that we perform some operations before actually waiting for any server to be active. So we save maybe 1/2 seconds for free.\n\nThreads would be optimal but they are a pain to debug in case of failure, and debug-ability is super super important for Tempest.","commit_id":"9b8a367472707aa5e3ac3e7f327a142fbf356688"}]}