)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},"change_message_id":"596eea8d46cd66d4b7a0e2b0854ddd3f39624a02","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"a4c02f39_c5173b48","updated":"2024-07-10 12:04:46.000000000","message":"Precheck working, ready for review.","commit_id":"ca9720e14f1fb52d53e3e12b7febbe9ed5f5d413"},{"author":{"_account_id":35263,"name":"Matt Crees","email":"mattc@stackhpc.com","username":"mattcrees"},"change_message_id":"98eaf58a2ab456272ce89d0a6848d81a49fc8955","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"83b87eed_106c303d","updated":"2024-09-19 15:45:07.000000000","message":"Unsure if we\u0027d still want this in Dalmatian, but it should definitely be backported to Caracal","commit_id":"ca9720e14f1fb52d53e3e12b7febbe9ed5f5d413"},{"author":{"_account_id":32553,"name":"Sven Kieske","email":"sven_oss@posteo.de","username":"skieske"},"change_message_id":"1b0d4f89595f05021a6d1b447717a0abd127c766","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"fcdfc620_8f2f9826","updated":"2024-11-27 15:19:29.000000000","message":"recheck need fresh logs.","commit_id":"ca9720e14f1fb52d53e3e12b7febbe9ed5f5d413"}],"ansible/roles/horizon/tasks/precheck.yml":[{"author":{"_account_id":32553,"name":"Sven Kieske","email":"sven_oss@posteo.de","username":"skieske"},"change_message_id":"67cf3b2c58c65c7fb35cc576f77c110a1e197a89","unresolved":true,"context_lines":[{"line_number":30,"context_line":"- name: Check for old local_settings file"},{"line_number":31,"context_line":"  assert:"},{"line_number":32,"context_line":"    that:"},{"line_number":33,"context_line":"      - \"\u0027{{ node_custom_config }}/horizon/{{ inventory_hostname }}/local_settings\u0027 is not exists\""},{"line_number":34,"context_line":"      - \"\u0027{{ node_custom_config }}/horizon/local_settings\u0027 is not exists\""},{"line_number":35,"context_line":"    fail_msg: \u003e-"},{"line_number":36,"context_line":"      Horizon configuration must now be provided using"}],"source_content_type":"text/x-yaml","patch_set":6,"id":"b56aab04_100af987","line":33,"range":{"start_line":33,"start_character":0,"end_line":33,"end_character":2},"updated":"2024-07-10 12:16:04.000000000","message":"not sure if it would be better to use `link_exists` instead?\n\nhttps://docs.ansible.com/ansible/latest/collections/ansible/builtin/link_exists_test.html","commit_id":"ca9720e14f1fb52d53e3e12b7febbe9ed5f5d413"},{"author":{"_account_id":32553,"name":"Sven Kieske","email":"sven_oss@posteo.de","username":"skieske"},"change_message_id":"5b81074bd70451e22bde855f79cedd7d86363980","unresolved":false,"context_lines":[{"line_number":30,"context_line":"- name: Check for old local_settings file"},{"line_number":31,"context_line":"  assert:"},{"line_number":32,"context_line":"    that:"},{"line_number":33,"context_line":"      - \"\u0027{{ node_custom_config }}/horizon/{{ inventory_hostname }}/local_settings\u0027 is not exists\""},{"line_number":34,"context_line":"      - \"\u0027{{ node_custom_config }}/horizon/local_settings\u0027 is not exists\""},{"line_number":35,"context_line":"    fail_msg: \u003e-"},{"line_number":36,"context_line":"      Horizon configuration must now be provided using"}],"source_content_type":"text/x-yaml","patch_set":6,"id":"71b3c8dc_2b53748c","line":33,"range":{"start_line":33,"start_character":0,"end_line":33,"end_character":2},"in_reply_to":"25d9f266_d90909c4","updated":"2024-07-16 14:20:00.000000000","message":"ah, sorry, the wording in the ansible docs i ambigous, it mixes \"links\" with \"symlinks\" and \"paths\". I thus thought this was able to also detect hardlinks.","commit_id":"ca9720e14f1fb52d53e3e12b7febbe9ed5f5d413"},{"author":{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},"change_message_id":"80cac45a492a86671e1601250d50640baefc7600","unresolved":true,"context_lines":[{"line_number":30,"context_line":"- name: Check for old local_settings file"},{"line_number":31,"context_line":"  assert:"},{"line_number":32,"context_line":"    that:"},{"line_number":33,"context_line":"      - \"\u0027{{ node_custom_config }}/horizon/{{ inventory_hostname }}/local_settings\u0027 is not exists\""},{"line_number":34,"context_line":"      - \"\u0027{{ node_custom_config }}/horizon/local_settings\u0027 is not exists\""},{"line_number":35,"context_line":"    fail_msg: \u003e-"},{"line_number":36,"context_line":"      Horizon configuration must now be provided using"}],"source_content_type":"text/x-yaml","patch_set":6,"id":"25d9f266_d90909c4","line":33,"range":{"start_line":33,"start_character":0,"end_line":33,"end_character":2},"in_reply_to":"b56aab04_100af987","updated":"2024-07-11 10:16:03.000000000","message":"Why? I wouldn\u0027t necessarily expect this to be a symlink.","commit_id":"ca9720e14f1fb52d53e3e12b7febbe9ed5f5d413"}]}