)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":20676,"name":"daniel.pawlik","display_name":"Daniel Pawlik","email":"dpawlik@redhat.com","username":"daniel.pawlik"},"change_message_id":"d36f81fd237f52e123e2d20206c40276e0483221","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"d4081c26_8570ad15","updated":"2022-03-15 10:32:32.000000000","message":"recheck","commit_id":"08fc0d1f3190bf65620ae2c48cf48ecb4ed45010"},{"author":{"_account_id":20676,"name":"daniel.pawlik","display_name":"Daniel Pawlik","email":"dpawlik@redhat.com","username":"daniel.pawlik"},"change_message_id":"c8b04b6400f24d869e4cf36e424b04699eeae96f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"9e4eb7a3_d741294a","updated":"2022-03-16 09:09:56.000000000","message":"rech","commit_id":"5dfb4ae1e9cb9a766173f0c5186a5dfd3e094fec"}],"logscraper/logsender.py":[{"author":{"_account_id":20676,"name":"daniel.pawlik","display_name":"Daniel Pawlik","email":"dpawlik@redhat.com","username":"daniel.pawlik"},"change_message_id":"123dfdb5d92eaee54e768bf449ad69f70317a21f","unresolved":true,"context_lines":[{"line_number":306,"context_line":"    if args.keep:"},{"line_number":307,"context_line":"        logging.info(\"Keeping file %s\" % build_dir)"},{"line_number":308,"context_line":"        return"},{"line_number":309,"context_line":""},{"line_number":310,"context_line":"    if send_status:"},{"line_number":311,"context_line":"        remove_directory(build_dir)"},{"line_number":312,"context_line":"    else:"},{"line_number":313,"context_line":"        logging.warning(\"The document was not send. Keeping log file\")"},{"line_number":314,"context_line":""},{"line_number":315,"context_line":""},{"line_number":316,"context_line":"def get_index(args):"}],"source_content_type":"text/x-python","patch_set":4,"id":"aec91e51_56307bd2","line":313,"range":{"start_line":309,"start_character":0,"end_line":313,"end_character":70},"updated":"2022-03-14 08:18:10.000000000","message":"we don\u0027t check send status, so it will always keep the files, right?","commit_id":"ef58d1cabfe8b860f5b2c975a333ef11f45fcd2e"},{"author":{"_account_id":20676,"name":"daniel.pawlik","display_name":"Daniel Pawlik","email":"dpawlik@redhat.com","username":"daniel.pawlik"},"change_message_id":"3977767233c0d6ea20d802a0250bc0cf83797b4e","unresolved":true,"context_lines":[{"line_number":306,"context_line":"    if args.keep:"},{"line_number":307,"context_line":"        logging.info(\"Keeping file %s\" % build_dir)"},{"line_number":308,"context_line":"        return"},{"line_number":309,"context_line":""},{"line_number":310,"context_line":"    if send_status:"},{"line_number":311,"context_line":"        remove_directory(build_dir)"},{"line_number":312,"context_line":"    else:"},{"line_number":313,"context_line":"        logging.warning(\"The document was not send. Keeping log file\")"},{"line_number":314,"context_line":""},{"line_number":315,"context_line":""},{"line_number":316,"context_line":"def get_index(args):"}],"source_content_type":"text/x-python","patch_set":4,"id":"82260c7e_209e05a1","line":313,"range":{"start_line":309,"start_character":0,"end_line":313,"end_character":70},"in_reply_to":"542fa4c4_3dde718d","updated":"2022-03-15 09:45:27.000000000","message":"https://review.opendev.org/c/openstack/ci-log-processing/+/831968 is fixing that issue","commit_id":"ef58d1cabfe8b860f5b2c975a333ef11f45fcd2e"},{"author":{"_account_id":9311,"name":"Tristan Cacqueray","email":"tdecacqu@redhat.com","username":"tristanC"},"change_message_id":"8578635fb90ececb7ea23973468df6a460d893b4","unresolved":true,"context_lines":[{"line_number":306,"context_line":"    if args.keep:"},{"line_number":307,"context_line":"        logging.info(\"Keeping file %s\" % build_dir)"},{"line_number":308,"context_line":"        return"},{"line_number":309,"context_line":""},{"line_number":310,"context_line":"    if send_status:"},{"line_number":311,"context_line":"        remove_directory(build_dir)"},{"line_number":312,"context_line":"    else:"},{"line_number":313,"context_line":"        logging.warning(\"The document was not send. Keeping log file\")"},{"line_number":314,"context_line":""},{"line_number":315,"context_line":""},{"line_number":316,"context_line":"def get_index(args):"}],"source_content_type":"text/x-python","patch_set":4,"id":"542fa4c4_3dde718d","line":313,"range":{"start_line":309,"start_character":0,"end_line":313,"end_character":70},"in_reply_to":"aec91e51_56307bd2","updated":"2022-03-14 12:08:50.000000000","message":"I see, we need to break when send_bulk fails.\n\nWhat is going to remove the log files?","commit_id":"ef58d1cabfe8b860f5b2c975a333ef11f45fcd2e"}]}