)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":23851,"name":"Riccardo Pittau","email":"elfosardo@gmail.com","username":"elfosardo"},"change_message_id":"a3b8e2d1d7aea33e413b87233a19db15767b3f85","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"4b04dbae_ea73d599","updated":"2021-12-09 09:59:30.000000000","message":"recheck","commit_id":"c0783334f7d7b54433c28ccceec252138624b0f4"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"170f2a93b6db83a8b9a7437388a60b479a305c23","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"7a7ef13a_d68b6a79","updated":"2021-12-09 15:58:17.000000000","message":"recheck","commit_id":"c0783334f7d7b54433c28ccceec252138624b0f4"},{"author":{"_account_id":4571,"name":"Steve Baker","email":"sbaker@redhat.com","username":"steve-stevebaker"},"change_message_id":"64d9189d8bee697682d3b1e34307d4b50b996baa","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"e6330c64_e9014f45","updated":"2021-12-08 20:51:54.000000000","message":"recheck","commit_id":"c0783334f7d7b54433c28ccceec252138624b0f4"}],"ironic/common/pxe_utils.py":[{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"0c96edfc12185914e9c2578c30dca62238895d7f","unresolved":false,"context_lines":[{"line_number":1218,"context_line":"                              CONF.force_raw_images)"},{"line_number":1219,"context_line":"    if CONF.pxe.file_permission:"},{"line_number":1220,"context_line":"        for info in images_info:"},{"line_number":1221,"context_line":"            os.chmod(info[1], CONF.pxe.file_permission)"},{"line_number":1222,"context_line":""},{"line_number":1223,"context_line":""},{"line_number":1224,"context_line":"def clean_up_pxe_env(task, images_info, ipxe_enabled\u003dFalse):"}],"source_content_type":"text/x-python","patch_set":5,"id":"b953b609_8ecd1a34","line":1221,"updated":"2021-12-07 18:41:28.000000000","message":"nit: We should likely find a way to move this logic to the image cache itself","commit_id":"8612c8547a40766219ca1e859e30260f098e7c66"},{"author":{"_account_id":4571,"name":"Steve Baker","email":"sbaker@redhat.com","username":"steve-stevebaker"},"change_message_id":"926bcafe1bed2092f91345a21a2a3de52855376d","unresolved":false,"context_lines":[{"line_number":1218,"context_line":"                              CONF.force_raw_images)"},{"line_number":1219,"context_line":"    if CONF.pxe.file_permission:"},{"line_number":1220,"context_line":"        for info in images_info:"},{"line_number":1221,"context_line":"            os.chmod(info[1], CONF.pxe.file_permission)"},{"line_number":1222,"context_line":""},{"line_number":1223,"context_line":""},{"line_number":1224,"context_line":"def clean_up_pxe_env(task, images_info, ipxe_enabled\u003dFalse):"}],"source_content_type":"text/x-python","patch_set":5,"id":"2ae55663_93260c9f","line":1221,"in_reply_to":"b953b609_8ecd1a34","updated":"2021-12-07 20:13:54.000000000","message":"Agreed, it should be a follow-up though. There are a few different code paths in the cache implementation which write to the destination path","commit_id":"8612c8547a40766219ca1e859e30260f098e7c66"}]}