)]}'
{"playbooks/common-tasks/dynamic-address-fact.yml":[{"author":{"_account_id":28619,"name":"Dmitriy Rabotyagov","email":"noonedeadpunk@gmail.com","username":"noonedeadpunk"},"change_message_id":"1c94720cf515ab0ee6489001faa20d23eb04d99a","unresolved":true,"context_lines":[{"line_number":19,"context_line":"    filter: \"{{ dynamic_address_gather_filter | default((\u0027ansible_\u0027 ~ container_networks[network_address][\u0027bridge\u0027]) | replace(\u0027-\u0027,\u0027_\u0027)) }}\""},{"line_number":20,"context_line":"  when:"},{"line_number":21,"context_line":"    - is_metal"},{"line_number":22,"context_line":"    - (dynamic_address_gather_filter is defined) or (container_networks[network_address] is defined and container_networks[network_address][\u0027bridge\u0027] is defined)"},{"line_number":23,"context_line":""},{"line_number":24,"context_line":"- name: Set IP to use for {{ network_address }}"},{"line_number":25,"context_line":"  set_fact: \"{{ network_address }}\u003d{{ hostvars[inventory_hostname] | json_query(query) }}\""}],"source_content_type":"text/x-yaml","patch_set":2,"id":"c1adedf3_6cd74373","side":"PARENT","line":22,"range":{"start_line":22,"start_character":53,"end_line":22,"end_character":99},"updated":"2021-10-21 08:49:51.000000000","message":"For some reason I thought that this part should cover cases where network_address is not defined. Maybe it would be simpler just to change to `network_address in container_networks`?","commit_id":"78d9a8596ad14e1caa483887545abb587c6c9fd9"},{"author":{"_account_id":31749,"name":"James Gibson","email":"james.gibson@bbc.co.uk","username":"jamesgibo"},"change_message_id":"6ba7c574f6e381f8c9e778a436021b9895fbb3c7","unresolved":false,"context_lines":[{"line_number":19,"context_line":"    filter: \"{{ dynamic_address_gather_filter | default((\u0027ansible_\u0027 ~ container_networks[network_address][\u0027bridge\u0027]) | replace(\u0027-\u0027,\u0027_\u0027)) }}\""},{"line_number":20,"context_line":"  when:"},{"line_number":21,"context_line":"    - is_metal"},{"line_number":22,"context_line":"    - (dynamic_address_gather_filter is defined) or (container_networks[network_address] is defined and container_networks[network_address][\u0027bridge\u0027] is defined)"},{"line_number":23,"context_line":""},{"line_number":24,"context_line":"- name: Set IP to use for {{ network_address }}"},{"line_number":25,"context_line":"  set_fact: \"{{ network_address }}\u003d{{ hostvars[inventory_hostname] | json_query(query) }}\""}],"source_content_type":"text/x-yaml","patch_set":2,"id":"f9893a36_8593c04d","side":"PARENT","line":22,"range":{"start_line":22,"start_character":53,"end_line":22,"end_character":99},"in_reply_to":"c1adedf3_6cd74373","updated":"2021-11-11 14:51:35.000000000","message":"It does, however in the case when you have `dynamic_address_gather_filter` is defined, the task is executed and even though the jinja filter default is not required, it still tries to evaluate `container_networks[network_address][\u0027bridge\u0027]` which is not defined.\n\n`{{ dynamic_address_gather_filter | default((\u0027ansible_\u0027 ~ container_networks[network_address][\u0027bridge\u0027]) | replace(\u0027-\u0027,\u0027_\u0027)) }}\"`","commit_id":"78d9a8596ad14e1caa483887545abb587c6c9fd9"},{"author":{"_account_id":28619,"name":"Dmitriy Rabotyagov","email":"noonedeadpunk@gmail.com","username":"noonedeadpunk"},"change_message_id":"7c08899a60020704369e0313cb4a70d25ae9ee04","unresolved":false,"context_lines":[{"line_number":19,"context_line":"    filter: \"{{ dynamic_address_gather_filter | default((\u0027ansible_\u0027 ~ container_networks[network_address][\u0027bridge\u0027]) | replace(\u0027-\u0027,\u0027_\u0027)) }}\""},{"line_number":20,"context_line":"  when:"},{"line_number":21,"context_line":"    - is_metal"},{"line_number":22,"context_line":"    - (dynamic_address_gather_filter is defined) or (container_networks[network_address] is defined and container_networks[network_address][\u0027bridge\u0027] is defined)"},{"line_number":23,"context_line":""},{"line_number":24,"context_line":"- name: Set IP to use for {{ network_address }}"},{"line_number":25,"context_line":"  set_fact: \"{{ network_address }}\u003d{{ hostvars[inventory_hostname] | json_query(query) }}\""}],"source_content_type":"text/x-yaml","patch_set":2,"id":"18b0b504_04e455db","side":"PARENT","line":22,"range":{"start_line":22,"start_character":53,"end_line":22,"end_character":99},"in_reply_to":"f9893a36_8593c04d","updated":"2021-11-11 14:56:58.000000000","message":"ah, that is fair","commit_id":"78d9a8596ad14e1caa483887545abb587c6c9fd9"}]}