)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":25023,"name":"Jonathan Rosser","email":"jonathan.rosser@rd.bbc.co.uk","username":"jrosser"},"change_message_id":"e297ad193e301951aa21de587539c32e49d95c1b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"e485bf18_ddd65aac","updated":"2022-06-09 21:08:50.000000000","message":"recheck - centos9s repo error, should be hopefully fixed now we use the infra mirror, but should check if dnf uses this during creating the chroot","commit_id":"f40abfa2f413476802c50bd502e9694640ca2550"}],"playbooks/os-glance-install.yml":[{"author":{"_account_id":25023,"name":"Jonathan Rosser","email":"jonathan.rosser@rd.bbc.co.uk","username":"jrosser"},"change_message_id":"dbc6b466d6c1cc8973cab07709f04368ac15e93a","unresolved":true,"context_lines":[{"line_number":54,"context_line":"      when:"},{"line_number":55,"context_line":"        - not is_metal"},{"line_number":56,"context_line":"        - glance_default_store \u003d\u003d \"file\""},{"line_number":57,"context_line":"        - (glance_remote_client is not defined) or (glance_remote_client | length \u003d\u003d 0)"},{"line_number":58,"context_line":""},{"line_number":59,"context_line":"    - name: Configure container (nfs)"},{"line_number":60,"context_line":"      include_tasks: \"common-tasks/os-{{ container_tech | default(\u0027lxc\u0027) }}-container-setup.yml\""}],"source_content_type":"text/x-yaml","patch_set":1,"id":"cac7bffb_f4aa3568","line":57,"range":{"start_line":57,"start_character":11,"end_line":57,"end_character":31},"updated":"2022-04-27 15:48:21.000000000","message":"just checking this makes sense, as it\u0027s a glance role default which defaults to defined and has a value of \"{{ glance_nfs_client | default([]) }}\" but only inside the glance role.","commit_id":"582e81c2b59f7d30b36f0e4a78935f7722f719cd"},{"author":{"_account_id":28619,"name":"Dmitriy Rabotyagov","email":"noonedeadpunk@gmail.com","username":"noonedeadpunk"},"change_message_id":"ada1335f0ca1b265ea6524230cda757f8f85fae6","unresolved":true,"context_lines":[{"line_number":54,"context_line":"      when:"},{"line_number":55,"context_line":"        - not is_metal"},{"line_number":56,"context_line":"        - glance_default_store \u003d\u003d \"file\""},{"line_number":57,"context_line":"        - (glance_remote_client is not defined) or (glance_remote_client | length \u003d\u003d 0)"},{"line_number":58,"context_line":""},{"line_number":59,"context_line":"    - name: Configure container (nfs)"},{"line_number":60,"context_line":"      include_tasks: \"common-tasks/os-{{ container_tech | default(\u0027lxc\u0027) }}-container-setup.yml\""}],"source_content_type":"text/x-yaml","patch_set":1,"id":"d99c204b_2a178b30","line":57,"range":{"start_line":57,"start_character":11,"end_line":57,"end_character":31},"in_reply_to":"cac7bffb_f4aa3568","updated":"2022-05-20 15:24:24.000000000","message":"Well, that\u0027s good question. I guess intention here was to catch user-defined variable and act upon that. as here we should not have role vars being scoped.","commit_id":"582e81c2b59f7d30b36f0e4a78935f7722f719cd"}]}