)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":28619,"name":"Dmitriy Rabotyagov","email":"noonedeadpunk@gmail.com","username":"noonedeadpunk"},"change_message_id":"64fcb9d5665e8798279b95b4d0b437e015bf2c4b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"e3c2a826_23cab1b1","updated":"2025-07-17 21:03:53.000000000","message":"recheck - molecule delete intermittent error","commit_id":"1e170dfa1b54d543a0ff77cc809c4adb205a29de"}],"tasks/systemd_mounts.yml":[{"author":{"_account_id":25023,"name":"Jonathan Rosser","email":"jonathan.rosser@rd.bbc.co.uk","username":"jrosser"},"change_message_id":"6c4c67f4f59b46faa1258fc2fa2f1239a53b4c75","unresolved":true,"context_lines":[{"line_number":26,"context_line":""},{"line_number":27,"context_line":"- name: Define mount service name"},{"line_number":28,"context_line":"  ansible.builtin.set_fact:"},{"line_number":29,"context_line":"    mount_service_name: \"{{ mount_service_name_escaped.stdout | default([systemd_mount_item, systemd_mount_suffix] | join(\u0027.\u0027)) }}\""},{"line_number":30,"context_line":""},{"line_number":31,"context_line":"# NOTE(noonedeadpunk): with s3fs IO error would raise on attempt to change permissions."},{"line_number":32,"context_line":"- name: Create mount target(s)"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"bd5131c6_e42ee796","line":29,"range":{"start_line":29,"start_character":4,"end_line":29,"end_character":22},"updated":"2025-06-05 21:46:01.000000000","message":"this could just be a lazy evaluated var rather than a set_fact to save a task execution","commit_id":"1e170dfa1b54d543a0ff77cc809c4adb205a29de"},{"author":{"_account_id":28619,"name":"Dmitriy Rabotyagov","email":"noonedeadpunk@gmail.com","username":"noonedeadpunk"},"change_message_id":"6e8bbb258d4e11b9bfab22034a5146d647e16233","unresolved":true,"context_lines":[{"line_number":26,"context_line":""},{"line_number":27,"context_line":"- name: Define mount service name"},{"line_number":28,"context_line":"  ansible.builtin.set_fact:"},{"line_number":29,"context_line":"    mount_service_name: \"{{ mount_service_name_escaped.stdout | default([systemd_mount_item, systemd_mount_suffix] | join(\u0027.\u0027)) }}\""},{"line_number":30,"context_line":""},{"line_number":31,"context_line":"# NOTE(noonedeadpunk): with s3fs IO error would raise on attempt to change permissions."},{"line_number":32,"context_line":"- name: Create mount target(s)"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"fae767b2_7c64e0f8","line":29,"range":{"start_line":29,"start_character":4,"end_line":29,"end_character":22},"in_reply_to":"bd5131c6_e42ee796","updated":"2025-06-24 13:45:41.000000000","message":"Um, this var is used 6  more times down the role. Then it would be evaluated 6 times rather then set_fact once?\n\nAs then I guess saving on set_fact is arguable, also considering that logic needs to be repeated for each following task.","commit_id":"1e170dfa1b54d543a0ff77cc809c4adb205a29de"}]}