)]}'
{"tempest/api/orchestration/stacks/test_neutron_resources.py":[{"author":{"_account_id":20190,"name":"zhufl","email":"zhu.fanglei@zte.com.cn","username":"zhufl"},"change_message_id":"8d4e327d85f98c4ad406e03a7994e67b103a0643","unresolved":false,"context_lines":[{"line_number":85,"context_line":"                body \u003d cls.client.show_resource(cls.stack_identifier,"},{"line_number":86,"context_line":"                                                \u0027Server\u0027)"},{"line_number":87,"context_line":"                if \u0027physical_resource_id\u0027 not in body:"},{"line_number":88,"context_line":"                    LOG.error(\"Strange resource record %s\", str(body))"},{"line_number":89,"context_line":"                    raise  # original exception"},{"line_number":90,"context_line":"                server_id \u003d body[\u0027physical_resource_id\u0027]"},{"line_number":91,"context_line":"                LOG.debug(\u0027Console output for %s\u0027, server_id)"}],"source_content_type":"text/x-python","patch_set":1,"id":"da36d5c6_0559f88d","line":88,"range":{"start_line":88,"start_character":31,"end_line":88,"end_character":69},"updated":"2017-02-27 08:10:37.000000000","message":"Since the reason why \u0027physical_resource_id\u0027 is not in body is that the stack may not been created successfully(that\u0027s not strange),\nso the message \"Strange resource record %s\" may look a bit strange. shall it be something like \"Stack resource is: %s\"?","commit_id":"fdc78228b1eb6acf9b276a83278e69aef7a59738"},{"author":{"_account_id":7350,"name":"Jordan Pittier","email":"jordan.pittier@gmail.com","username":"JordanP"},"change_message_id":"1ae42ebb32647d21bc9b12d985a7b1c635cc1d5f","unresolved":false,"context_lines":[{"line_number":85,"context_line":"                body \u003d cls.client.show_resource(cls.stack_identifier,"},{"line_number":86,"context_line":"                                                \u0027Server\u0027)"},{"line_number":87,"context_line":"                if \u0027physical_resource_id\u0027 not in body:"},{"line_number":88,"context_line":"                    LOG.error(\"Strange resource record %s\", str(body))"},{"line_number":89,"context_line":"                    raise  # original exception"},{"line_number":90,"context_line":"                server_id \u003d body[\u0027physical_resource_id\u0027]"},{"line_number":91,"context_line":"                LOG.debug(\u0027Console output for %s\u0027, server_id)"}],"source_content_type":"text/x-python","patch_set":1,"id":"da36d5c6_79a8ca77","line":88,"updated":"2017-02-24 15:46:06.000000000","message":"That log message doesn\u0027t really help...\n\nI think this needs a bug report or more explanation because I don\"t understand this change...","commit_id":"fdc78228b1eb6acf9b276a83278e69aef7a59738"},{"author":{"_account_id":5803,"name":"Attila Fazekas","email":"afazekas@redhat.com","username":"afazekas"},"change_message_id":"417a99894f0563c94eef9a41bfd2835625eb7b53","unresolved":false,"context_lines":[{"line_number":85,"context_line":"                body \u003d cls.client.show_resource(cls.stack_identifier,"},{"line_number":86,"context_line":"                                                \u0027Server\u0027)"},{"line_number":87,"context_line":"                if \u0027physical_resource_id\u0027 not in body:"},{"line_number":88,"context_line":"                    LOG.error(\"Strange resource record %s\", str(body))"},{"line_number":89,"context_line":"                    raise  # original exception"},{"line_number":90,"context_line":"                server_id \u003d body[\u0027physical_resource_id\u0027]"},{"line_number":91,"context_line":"                LOG.debug(\u0027Console output for %s\u0027, server_id)"}],"source_content_type":"text/x-python","patch_set":1,"id":"da36d5c6_e0b3084b","line":88,"in_reply_to":"da36d5c6_79a8ca77","updated":"2017-02-25 06:36:28.000000000","message":"What you want to see here is the timeout exception,\nthis log message is not really important, unless you miss console output from your log file.","commit_id":"fdc78228b1eb6acf9b276a83278e69aef7a59738"},{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"f0342c50dc5308be9b05453a88fdaadd106d4f03","unresolved":false,"context_lines":[{"line_number":84,"context_line":"                # to heat."},{"line_number":85,"context_line":"                body \u003d cls.client.show_resource(cls.stack_identifier,"},{"line_number":86,"context_line":"                                                \u0027Server\u0027)"},{"line_number":87,"context_line":"                if \u0027physical_resource_id\u0027 not in body:"},{"line_number":88,"context_line":"                    LOG.error(\"Server resource is: %s\", str(body))"},{"line_number":89,"context_line":"                    raise  # original exception"},{"line_number":90,"context_line":"                server_id \u003d body[\u0027physical_resource_id\u0027]"},{"line_number":91,"context_line":"                LOG.debug(\u0027Console output for %s\u0027, server_id)"},{"line_number":92,"context_line":"                output \u003d cls.servers_client.get_console_output("},{"line_number":93,"context_line":"                    server_id)[\u0027output\u0027]"},{"line_number":94,"context_line":"                LOG.debug(output)"}],"source_content_type":"text/x-python","patch_set":2,"id":"ba2be162_f4dc7d48","line":91,"range":{"start_line":87,"start_character":0,"end_line":91,"end_character":61},"updated":"2017-02-28 18:29:13.000000000","message":"How about\n\n server_id \u003d body.get(\u0027physical_resource_id\u0027)\n if server_id:\n     LOG.debug(\u0027Console output for %s\u0027, server_id)\n     output \u003d cls.servers_client.get_console_output(\n          server_id)[\u0027output\u0027]\n     LOG.debug(output)\n else:\n     LOG.debug(\u0027Server resource is %s\u0027, body)\n\n?","commit_id":"4ebbb6303a33b0fc07f3fded86f5b3904836e39b"},{"author":{"_account_id":20190,"name":"zhufl","email":"zhu.fanglei@zte.com.cn","username":"zhufl"},"change_message_id":"59ad50bf2cfa4f0074b684f36742dda9c78fe84f","unresolved":false,"context_lines":[{"line_number":84,"context_line":"                # to heat."},{"line_number":85,"context_line":"                body \u003d cls.client.show_resource(cls.stack_identifier,"},{"line_number":86,"context_line":"                                                \u0027Server\u0027)"},{"line_number":87,"context_line":"                if \u0027physical_resource_id\u0027 not in body:"},{"line_number":88,"context_line":"                    LOG.error(\"Server resource is: %s\", str(body))"},{"line_number":89,"context_line":"                    raise  # original exception"},{"line_number":90,"context_line":"                server_id \u003d body[\u0027physical_resource_id\u0027]"},{"line_number":91,"context_line":"                LOG.debug(\u0027Console output for %s\u0027, server_id)"},{"line_number":92,"context_line":"                output \u003d cls.servers_client.get_console_output("},{"line_number":93,"context_line":"                    server_id)[\u0027output\u0027]"},{"line_number":94,"context_line":"                LOG.debug(output)"}],"source_content_type":"text/x-python","patch_set":2,"id":"ba2be162_6a41770b","line":91,"range":{"start_line":87,"start_character":0,"end_line":91,"end_character":61},"in_reply_to":"ba2be162_f4dc7d48","updated":"2017-03-01 05:35:40.000000000","message":"yea, looks clearer","commit_id":"4ebbb6303a33b0fc07f3fded86f5b3904836e39b"},{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"c42606990f7f7f4ad96fb0c29c27327bed0c5524","unresolved":false,"context_lines":[{"line_number":92,"context_line":"                    LOG.debug(output)"},{"line_number":93,"context_line":"                else:"},{"line_number":94,"context_line":"                    LOG.debug(\u0027Server resource is %s\u0027, body)"},{"line_number":95,"context_line":"            raise  # original exception"},{"line_number":96,"context_line":""},{"line_number":97,"context_line":"        cls.test_resources \u003d {}"},{"line_number":98,"context_line":"        for resource in resources:"}],"source_content_type":"text/x-python","patch_set":3,"id":"ba2be162_122b6c95","line":95,"range":{"start_line":95,"start_character":17,"end_line":95,"end_character":39},"updated":"2017-03-02 18:13:59.000000000","message":"nit: Maybe this comment is moved from the previous PS which raised another exception. On current patch, this comment is not mandatory..","commit_id":"35901c4f0e2a9916581a19acc66f3a9d6ba76b90"}]}