)]}'
{"tripleoclient/v1/overcloud_node.py":[{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"602f199b7861ef1c08d02ec1589c20e3c29880f9","unresolved":false,"context_lines":[{"line_number":82,"context_line":""},{"line_number":83,"context_line":"        if not parsed_args.yes:"},{"line_number":84,"context_line":"            confirm \u003d oooutils.prompt_user_for_confirmation("},{"line_number":85,"context_line":"                message\u003d_(\"Are you sure you want to delete these overcloud \""},{"line_number":86,"context_line":"                          \"nodes [y/N]? \"),"},{"line_number":87,"context_line":"                logger\u003dself.log)"},{"line_number":88,"context_line":"            if not confirm:"}],"source_content_type":"text/x-python","patch_set":3,"id":"bfb3d3c7_0c0bc5d5","line":85,"range":{"start_line":85,"start_character":59,"end_line":85,"end_character":64},"updated":"2019-05-30 07:45:47.000000000","message":"would be useful to identify the nodes since we\u0027re asking for confirmation - you have args.nodes","commit_id":"2b3b7f1e1f4b7875ff62a6dca8862678f4589c89"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"74a72c84e53d3af4dea838fed0e3534a50ffaa78","unresolved":false,"context_lines":[{"line_number":82,"context_line":""},{"line_number":83,"context_line":"        if not parsed_args.yes:"},{"line_number":84,"context_line":"            confirm \u003d oooutils.prompt_user_for_confirmation("},{"line_number":85,"context_line":"                message\u003d_(\"Are you sure you want to delete these overcloud \""},{"line_number":86,"context_line":"                          \"nodes [y/N]? \"),"},{"line_number":87,"context_line":"                logger\u003dself.log)"},{"line_number":88,"context_line":"            if not confirm:"}],"source_content_type":"text/x-python","patch_set":3,"id":"bfb3d3c7_4cfc5db4","line":85,"range":{"start_line":85,"start_character":59,"end_line":85,"end_character":64},"in_reply_to":"bfb3d3c7_0c0bc5d5","updated":"2019-05-30 07:48:05.000000000","message":"well its just echo-ing back what they passed so maybe not that useful (i.e. you haven\u0027t yet confirmed there is a node with that ID yet or any other kind of operation) - I still think it would be good to have in the message.","commit_id":"2b3b7f1e1f4b7875ff62a6dca8862678f4589c89"},{"author":{"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},"change_message_id":"dd1c2f8769a156e006d819405581d84fd44b2a9a","unresolved":false,"context_lines":[{"line_number":82,"context_line":""},{"line_number":83,"context_line":"        if not parsed_args.yes:"},{"line_number":84,"context_line":"            confirm \u003d oooutils.prompt_user_for_confirmation("},{"line_number":85,"context_line":"                message\u003d_(\"Are you sure you want to delete these overcloud \""},{"line_number":86,"context_line":"                          \"nodes [y/N]? \"),"},{"line_number":87,"context_line":"                logger\u003dself.log)"},{"line_number":88,"context_line":"            if not confirm:"}],"source_content_type":"text/x-python","patch_set":3,"id":"bfb3d3c7_0dec1c28","line":85,"range":{"start_line":85,"start_character":59,"end_line":85,"end_character":64},"in_reply_to":"bfb3d3c7_4cfc5db4","updated":"2019-05-30 13:55:33.000000000","message":"I think this would be useful if we added uuid -\u003e hostname lookup or something. Though the main goal is primarily to make sure people don\u0027t accidently execute this command early","commit_id":"2b3b7f1e1f4b7875ff62a6dca8862678f4589c89"}]}