)]}'
{"tempest/common/compute.py":[{"author":{"_account_id":16425,"name":"Bin Bin Cong","email":"binbin.cong@easystack.cn","username":"stephen_cong"},"change_message_id":"6ada60441ccee5a0a922b47fd74ddd3874ddb5dd","unresolved":false,"context_lines":[{"line_number":44,"context_line":"    :param wait_until: Server status to wait for the server to reach after"},{"line_number":45,"context_line":"    its creation."},{"line_number":46,"context_line":"    :param volume_backed: Whether the instance is volume backed or not."},{"line_number":47,"context_line":"    :returns: a tuple"},{"line_number":48,"context_line":"    \"\"\""},{"line_number":49,"context_line":""},{"line_number":50,"context_line":"    # TODO(jlanoux) add support of wait_until PINGABLE/SSHABLE"}],"source_content_type":"text/x-python","patch_set":1,"id":"5aef4532_db14a145","line":47,"updated":"2016-03-09 03:00:54.000000000","message":"The parameters image, name, flavor do not have corresponding docstring, it will be nice to have them in this patch.","commit_id":"87ba2876ecad2c9b643660d116f41a61d03fbf9b"},{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"11170bef0c79f334e016cbb95140536bb0299868","unresolved":false,"context_lines":[{"line_number":44,"context_line":"    :param wait_until: Server status to wait for the server to reach after"},{"line_number":45,"context_line":"    its creation."},{"line_number":46,"context_line":"    :param volume_backed: Whether the instance is volume backed or not."},{"line_number":47,"context_line":"    :returns: a tuple"},{"line_number":48,"context_line":"    \"\"\""},{"line_number":49,"context_line":""},{"line_number":50,"context_line":"    # TODO(jlanoux) add support of wait_until PINGABLE/SSHABLE"}],"source_content_type":"text/x-python","patch_set":1,"id":"1af94dfe_7c25cca9","line":47,"in_reply_to":"3afc51ec_e9726210","updated":"2016-03-16 01:21:41.000000000","message":"Yeah, I agree with Jordan here.\n\nSometimes we find the other issues in a single review, but we should avoid extending the patch scope for a simple review.","commit_id":"87ba2876ecad2c9b643660d116f41a61d03fbf9b"},{"author":{"_account_id":7350,"name":"Jordan Pittier","email":"jordan.pittier@gmail.com","username":"JordanP"},"change_message_id":"df9c1d7ed14e47995f89c1a96ffea2898993a379","unresolved":false,"context_lines":[{"line_number":44,"context_line":"    :param wait_until: Server status to wait for the server to reach after"},{"line_number":45,"context_line":"    its creation."},{"line_number":46,"context_line":"    :param volume_backed: Whether the instance is volume backed or not."},{"line_number":47,"context_line":"    :returns: a tuple"},{"line_number":48,"context_line":"    \"\"\""},{"line_number":49,"context_line":""},{"line_number":50,"context_line":"    # TODO(jlanoux) add support of wait_until PINGABLE/SSHABLE"}],"source_content_type":"text/x-python","patch_set":1,"id":"3afc51ec_e9726210","line":47,"in_reply_to":"5aef4532_db14a145","updated":"2016-03-15 10:45:49.000000000","message":"This patch does a code cleanup. It\u0027s not about fixing docstrings. \n\nI agree it would be nice to have proper docstring, that\u0027s why you could send another patch to fix this :) thanks !\n\nYou can have a look at this really good document: https://wiki.openstack.org/wiki/GitCommitMessages","commit_id":"87ba2876ecad2c9b643660d116f41a61d03fbf9b"},{"author":{"_account_id":16986,"name":"Shuquan Huang","email":"shuquan.huang@live.com","username":"shuquan"},"change_message_id":"a34abbdad7ef099d73925511ae69af1796482ab9","unresolved":false,"context_lines":[{"line_number":120,"context_line":"    # handle the case of multiple servers"},{"line_number":121,"context_line":"    servers \u003d []"},{"line_number":122,"context_line":"    if multiple_create_request:"},{"line_number":123,"context_line":"        # Get servers created which name match with name param."},{"line_number":124,"context_line":"        body_servers \u003d clients.servers_client.list_servers()"},{"line_number":125,"context_line":"        servers \u003d \\"},{"line_number":126,"context_line":"            [s for s in body_servers[\u0027servers\u0027] if s[\u0027name\u0027].startswith(name)]"}],"source_content_type":"text/x-python","patch_set":1,"id":"5aef4532_036ec400","line":123,"range":{"start_line":123,"start_character":41,"end_line":123,"end_character":46},"updated":"2016-03-08 13:38:01.000000000","message":"s/match/matches/","commit_id":"87ba2876ecad2c9b643660d116f41a61d03fbf9b"},{"author":{"_account_id":7350,"name":"Jordan Pittier","email":"jordan.pittier@gmail.com","username":"JordanP"},"change_message_id":"df9c1d7ed14e47995f89c1a96ffea2898993a379","unresolved":false,"context_lines":[{"line_number":120,"context_line":"    # handle the case of multiple servers"},{"line_number":121,"context_line":"    servers \u003d []"},{"line_number":122,"context_line":"    if multiple_create_request:"},{"line_number":123,"context_line":"        # Get servers created which name match with name param."},{"line_number":124,"context_line":"        body_servers \u003d clients.servers_client.list_servers()"},{"line_number":125,"context_line":"        servers \u003d \\"},{"line_number":126,"context_line":"            [s for s in body_servers[\u0027servers\u0027] if s[\u0027name\u0027].startswith(name)]"}],"source_content_type":"text/x-python","patch_set":1,"id":"3afc51ec_e98ee2ce","line":123,"range":{"start_line":123,"start_character":41,"end_line":123,"end_character":46},"in_reply_to":"5aef4532_036ec400","updated":"2016-03-15 10:45:49.000000000","message":"Sure but fixing typos is not what this patch is about. \n\nYou can have a look at this really good document: https://wiki.openstack.org/wiki/GitCommitMessages","commit_id":"87ba2876ecad2c9b643660d116f41a61d03fbf9b"}]}