)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":37598,"name":"Ivan Anfimov","display_name":"Ivan Anfimov","email":"lazekteam@gmail.com","username":"anfimovir"},"change_message_id":"3a58bc7639078f118fa6b8f2193c4ce16ea881b3","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"889b485c_4f454873","updated":"2025-08-18 21:45:21.000000000","message":"recheck","commit_id":"30feae0c0eb4a440ff7c32d6c4bdde3207c6d407"}],"tasks/ironic_inspector_apparmor.yml":[{"author":{"_account_id":28619,"name":"Dmitriy Rabotyagov","email":"noonedeadpunk@gmail.com","username":"noonedeadpunk"},"change_message_id":"1cd7b58cb46d4909b4d1649429430b4cc0d8a140","unresolved":true,"context_lines":[{"line_number":26,"context_line":"  when:"},{"line_number":27,"context_line":"    - item.condition | default(true)"},{"line_number":28,"context_line":"  register: _apparmor_overrides"},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"- name: Reload apparmor"},{"line_number":31,"context_line":"  ansible.builtin.service:"},{"line_number":32,"context_line":"    name: apparmor"},{"line_number":33,"context_line":"    state: reloaded"},{"line_number":34,"context_line":"  when: _apparmor_overrides is changed # noqa: no-handler"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"c52e3284_956b3886","line":34,"range":{"start_line":29,"start_character":0,"end_line":34,"end_character":38},"updated":"2025-05-29 14:39:43.000000000","message":"any good reason for it to be not in handlers indeed? some race condition?","commit_id":"17defc737cb76b69ee4a28ddcf55c7e271c33ffd"},{"author":{"_account_id":25023,"name":"Jonathan Rosser","email":"jonathan.rosser@rd.bbc.co.uk","username":"jrosser"},"change_message_id":"2e29b611371fc73265d6997ff7a89285a90acd49","unresolved":true,"context_lines":[{"line_number":26,"context_line":"  when:"},{"line_number":27,"context_line":"    - item.condition | default(true)"},{"line_number":28,"context_line":"  register: _apparmor_overrides"},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"- name: Reload apparmor"},{"line_number":31,"context_line":"  ansible.builtin.service:"},{"line_number":32,"context_line":"    name: apparmor"},{"line_number":33,"context_line":"    state: reloaded"},{"line_number":34,"context_line":"  when: _apparmor_overrides is changed # noqa: no-handler"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"afef06f0_54e4e9fa","line":34,"range":{"start_line":29,"start_character":0,"end_line":34,"end_character":38},"in_reply_to":"c52e3284_956b3886","updated":"2025-05-29 14:51:27.000000000","message":"We install dnsmasq for inspector as part of this role, and the service won\u0027t start it it can\u0027t read the config file (prevented currently by apparmor). Hander would be too late, unless we somehow move the restart of dnsmasq to a later handler?","commit_id":"17defc737cb76b69ee4a28ddcf55c7e271c33ffd"},{"author":{"_account_id":28619,"name":"Dmitriy Rabotyagov","email":"noonedeadpunk@gmail.com","username":"noonedeadpunk"},"change_message_id":"4184638efc9c18437656d48f9849720e3ebdbfd0","unresolved":true,"context_lines":[{"line_number":25,"context_line":"    label: \"{{ item[\u0027profile\u0027] }}\""},{"line_number":26,"context_line":"  when:"},{"line_number":27,"context_line":"    - item.condition | default(true)"},{"line_number":28,"context_line":"  notify: Reload apparmor"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"4b932975_e4791396","line":28,"updated":"2025-06-02 08:47:26.000000000","message":"new line at end is missing","commit_id":"543fa760ea786f6e77bd440833c7711bb5d8c97d"},{"author":{"_account_id":28619,"name":"Dmitriy Rabotyagov","email":"noonedeadpunk@gmail.com","username":"noonedeadpunk"},"change_message_id":"6730a759a3d807e2afe01a89b52dca9762928561","unresolved":false,"context_lines":[{"line_number":25,"context_line":"    label: \"{{ item[\u0027profile\u0027] }}\""},{"line_number":26,"context_line":"  when:"},{"line_number":27,"context_line":"    - item.condition | default(true)"},{"line_number":28,"context_line":"  notify: Reload apparmor"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"0be86dcc_388c70dd","line":28,"in_reply_to":"4b932975_e4791396","updated":"2025-06-02 08:52:04.000000000","message":"Done","commit_id":"543fa760ea786f6e77bd440833c7711bb5d8c97d"}]}