)]}'
{"ansible/roles/heat/tasks/main.yml":[{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"27238fa97634c2633899526770c3ddda9a431c1f","unresolved":false,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"- include: \"{{ action }}.yml\""},{"line_number":3,"context_line":""},{"line_number":4,"context_line":"- include: check.yml"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"9ad45d7e_c6ea8ada","line":4,"range":{"start_line":4,"start_character":2,"end_line":4,"end_character":20},"updated":"2016-08-11 16:04:44.000000000","message":"this file does not exist.\ndid you skip adding it in the review.","commit_id":"c24e19ac7347e48a0ee744da5d1e36067b6bdfe6"},{"author":{"_account_id":7488,"name":"Jeffrey Zhang","email":"zhang.lei.fly@gmail.com","username":"Jeffrey4l"},"change_message_id":"0b108fc541994dd1861039d28b618e1fad4f2701","unresolved":false,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"- include: \"{{ action }}.yml\""},{"line_number":3,"context_line":""},{"line_number":4,"context_line":"- include: check.yml"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"3ac371cc_614b91b1","line":4,"updated":"2016-08-14 16:04:23.000000000","message":"this should be wrong","commit_id":"c24e19ac7347e48a0ee744da5d1e36067b6bdfe6"},{"author":{"_account_id":22165,"name":"caoyuan","email":"cao.yingjunz@gmail.com","username":"caoyuan"},"change_message_id":"66eb672fd2e7cccb7dd4ce9faaff1b224abf2c9e","unresolved":false,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"- include: \"{{ action }}.yml\""},{"line_number":3,"context_line":""},{"line_number":4,"context_line":"- include: check.yml"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"7a77a97e_6d031943","line":4,"in_reply_to":"3ac371cc_614b91b1","updated":"2016-11-16 02:09:03.000000000","message":"agree ，please remove it","commit_id":"c24e19ac7347e48a0ee744da5d1e36067b6bdfe6"}],"docker/kolla-toolbox/kolla_sanity.py":[{"author":{"_account_id":16233,"name":"Mauricio Lima","email":"mauriciolimab@gmail.com","username":"limamauricio"},"change_message_id":"d593af69c6197946e61a4f7cbcc3c6d83496017a","unresolved":false,"context_lines":[{"line_number":56,"context_line":"    @staticmethod"},{"line_number":57,"context_line":"    def heat(cloud):"},{"line_number":58,"context_line":"        [stack for stack in cloud.heat_client.stacks.list()]"},{"line_number":59,"context_line":""},{"line_number":60,"context_line":"def main():"},{"line_number":61,"context_line":"    module \u003d AnsibleModule("},{"line_number":62,"context_line":"        argument_spec\u003dopenstack_full_argument_spec("}],"source_content_type":"text/x-python","patch_set":1,"id":"bacf61ea_da769805","line":59,"updated":"2016-08-02 19:20:31.000000000","message":"expected 2 blank lines","commit_id":"c40dce3998bbc3217aecbfe5701ff34d8542dd64"},{"author":{"_account_id":167,"name":"Christian Berendt","email":"berendt@osism.tech","username":"berendt"},"change_message_id":"7d43cff7e4f8134fe6dd74036d3871ad1e0f5366","unresolved":false,"context_lines":[{"line_number":55,"context_line":""},{"line_number":56,"context_line":"    @staticmethod"},{"line_number":57,"context_line":"    def heat(cloud):"},{"line_number":58,"context_line":"        [stack for stack in cloud.heat_client.stacks.list()]"},{"line_number":59,"context_line":""},{"line_number":60,"context_line":""},{"line_number":61,"context_line":"def main():"}],"source_content_type":"text/x-python","patch_set":3,"id":"9ad45d7e_c0133b5b","line":58,"updated":"2016-08-08 23:36:02.000000000","message":"I do not unterstand the use of a loop here. Is it not sufficient to use cloud.heat_client.stacks.list() here?","commit_id":"5929b229a33442bdca4187f95d30ceaf53f176d7"},{"author":{"_account_id":7488,"name":"Jeffrey Zhang","email":"zhang.lei.fly@gmail.com","username":"Jeffrey4l"},"change_message_id":"639bcd953b7bae94bfb2d1a59ebe5423baa68006","unresolved":false,"context_lines":[{"line_number":55,"context_line":""},{"line_number":56,"context_line":"    @staticmethod"},{"line_number":57,"context_line":"    def heat(cloud):"},{"line_number":58,"context_line":"        [stack for stack in cloud.heat_client.stacks.list()]"},{"line_number":59,"context_line":""},{"line_number":60,"context_line":""},{"line_number":61,"context_line":"def main():"}],"source_content_type":"text/x-python","patch_set":3,"id":"9ad45d7e_0c36e5e5","line":58,"updated":"2016-08-10 05:33:16.000000000","message":"agree. the list() should be enough, it will send the request to the heat-api.\n\nI also saw the same logical for other service.","commit_id":"5929b229a33442bdca4187f95d30ceaf53f176d7"}]}