)]}'
{"tempest/api/compute/admin/test_aggregates.py":[{"author":{"_account_id":27078,"name":"Manik Bindlish","email":"manikbindlish19@gmail.com","username":"manik"},"change_message_id":"9e73c766fe9728292d3804081dd0c879990a5900","unresolved":false,"context_lines":[{"line_number":213,"context_line":"        for agg in aggregates:"},{"line_number":214,"context_line":"            if agg[\u0027availability_zone\u0027]:"},{"line_number":215,"context_line":"                hosts_in_zone.extend(agg[\u0027hosts\u0027])"},{"line_number":216,"context_line":"        hosts \u003d [v for v in self.hosts_available if v not in set(hosts_in_zone)]"},{"line_number":217,"context_line":"        if not hosts:"},{"line_number":218,"context_line":"            raise self.skipException(\"All hosts are already in other \""},{"line_number":219,"context_line":"                                     \"availability zones, so can\u0027t add \""}],"source_content_type":"text/x-python","patch_set":1,"id":"5f7c97a3_79cf6d8a","line":216,"range":{"start_line":216,"start_character":8,"end_line":216,"end_character":80},"updated":"2018-08-02 05:32:09.000000000","message":"It\u0027s exceeding 79 chars. So can we break it in two lines?\ne.g. \nhosts \u003d [v for v in self.hosts_available\n         if v not in set(hosts_in_zone)]","commit_id":"76041a7128a1e8e25389511759090b48ee2e4aaa"},{"author":{"_account_id":20190,"name":"zhufl","email":"zhu.fanglei@zte.com.cn","username":"zhufl"},"change_message_id":"54f82a0abeb9a1eecb41287a39beda91cba97e66","unresolved":false,"context_lines":[{"line_number":213,"context_line":"        for agg in aggregates:"},{"line_number":214,"context_line":"            if agg[\u0027availability_zone\u0027]:"},{"line_number":215,"context_line":"                hosts_in_zone.extend(agg[\u0027hosts\u0027])"},{"line_number":216,"context_line":"        hosts \u003d [v for v in self.hosts_available if v not in set(hosts_in_zone)]"},{"line_number":217,"context_line":"        if not hosts:"},{"line_number":218,"context_line":"            raise self.skipException(\"All hosts are already in other \""},{"line_number":219,"context_line":"                                     \"availability zones, so can\u0027t add \""}],"source_content_type":"text/x-python","patch_set":1,"id":"5f7c97a3_8c06a91b","line":216,"range":{"start_line":216,"start_character":61,"end_line":216,"end_character":65},"updated":"2018-08-03 02:38:49.000000000","message":"Maybe we can remove this \"set\"? It seems doesn\u0027t matter even if there are duplicated hosts in hosts_in_zone","commit_id":"76041a7128a1e8e25389511759090b48ee2e4aaa"},{"author":{"_account_id":23186,"name":"Felipe Monteiro","email":"felipe.carneiro.monteiro@gmail.com","username":"felipe.monteiro"},"change_message_id":"eaed6fa0c72bed1758b8be08fd9afd0a8f7c7e81","unresolved":false,"context_lines":[{"line_number":217,"context_line":"        if not hosts:"},{"line_number":218,"context_line":"            raise self.skipException(\"All hosts are already in other \""},{"line_number":219,"context_line":"                                     \"availability zones, so can\u0027t add \""},{"line_number":220,"context_line":"                                     \"host to aggregate. \\nAggregates list:\""},{"line_number":221,"context_line":"                                     \"%s\" % aggregates)"},{"line_number":222,"context_line":"        host \u003d hosts[0]"},{"line_number":223,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"3f79a3b5_47fa4075","line":220,"range":{"start_line":220,"start_character":74,"end_line":220,"end_character":75},"updated":"2018-08-22 13:28:54.000000000","message":"nit: missing space after this","commit_id":"3bdb19f4058fc70677d8f9a397a06beff12dab82"},{"author":{"_account_id":20190,"name":"zhufl","email":"zhu.fanglei@zte.com.cn","username":"zhufl"},"change_message_id":"27cd948d749e7afaae820e86183c27cd5de21bc0","unresolved":false,"context_lines":[{"line_number":217,"context_line":"        if not hosts:"},{"line_number":218,"context_line":"            raise self.skipException(\"All hosts are already in other \""},{"line_number":219,"context_line":"                                     \"availability zones, so can\u0027t add \""},{"line_number":220,"context_line":"                                     \"host to aggregate. \\nAggregates list:\""},{"line_number":221,"context_line":"                                     \"%s\" % aggregates)"},{"line_number":222,"context_line":"        host \u003d hosts[0]"},{"line_number":223,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"3f79a3b5_f32843ef","line":220,"range":{"start_line":220,"start_character":74,"end_line":220,"end_character":75},"in_reply_to":"3f79a3b5_47fa4075","updated":"2018-08-30 01:53:23.000000000","message":"Done","commit_id":"3bdb19f4058fc70677d8f9a397a06beff12dab82"}]}