)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"b53d08ec256246f132ac53da141627da26257bdd","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"b4a595d1_8d8debc1","updated":"2023-04-17 22:24:56.000000000","message":"This looks OK to me.\n\nI do wonder if we should have this as a cron job, as we may go long periods between restarts?\n\nI guess this will get logged in the docker output, which is good enough.  But any cron job should save it to a logfile so we know what we deleted.\n\nOne thought for testing is to setup a dummy set of files, and have an argument to this to work in a different directory, and just make sure it deletes what we think it does?","commit_id":"db178992ebe8e64330af07caa6788094a5b1b750"},{"author":{"_account_id":12898,"name":"Tony Breeds","email":"tony@bakeyournoodle.com","username":"tonyb"},"change_message_id":"6b0f4906d44dac91a40ab9c7b28f69f37fd9e92e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"28b33158_7f960b2a","updated":"2023-05-09 19:49:11.000000000","message":"Looks good to me.  Just one nit.  Can be fixed in a follow-up or not at all really.","commit_id":"64cc9514e5efb33b048868da7ac029125216b0f9"},{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"5d0488398b8dc262274c067ce98b43cf0894814e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"ab9ad387_35ed7a7f","updated":"2023-04-18 23:01:23.000000000","message":"This LGTM.  We discussed in the meeting today reporting this upstream, and possibly having it so that failures are separated into \"can not talk to host\" \u003d\u003d retry and \"can not push due to acl issues, etc.\" \u003d\u003d delete file.\n\nHowever, while it would be ideal for gerrit to fix, the reality is that will not happen and we need to maintain and restart what we have without thousands of errors.","commit_id":"64cc9514e5efb33b048868da7ac029125216b0f9"}],"docker/gerrit/base/Dockerfile":[{"author":{"_account_id":12898,"name":"Tony Breeds","email":"tony@bakeyournoodle.com","username":"tonyb"},"change_message_id":"6b0f4906d44dac91a40ab9c7b28f69f37fd9e92e","unresolved":true,"context_lines":[{"line_number":60,"context_line":"# the bare minimum to start a Gerrit instance."},{"line_number":61,"context_line":"#"},{"line_number":62,"context_line":"# These scripts live in"},{"line_number":63,"context_line":"# system-config/playbooks/zuul/gerrit/files/wait-for-it.sh. They are not"},{"line_number":64,"context_line":"# in the Dockerfile dir as the Dockerfile is run in the context of the"},{"line_number":65,"context_line":"# Gerrit source. Instead we copy those files with Ansible to the Gerrit"},{"line_number":66,"context_line":"# source dir."}],"source_content_type":"text/x-dockerfile","patch_set":7,"id":"9fe78fda_b955e235","line":63,"range":{"start_line":63,"start_character":44,"end_line":63,"end_character":58},"updated":"2023-05-09 19:49:11.000000000","message":"NIT: Addional filename here","commit_id":"64cc9514e5efb33b048868da7ac029125216b0f9"}]}