)]}'
{"tripleo_ansible/roles/tripleo_ssh_known_hosts/molecule/default/molecule.yml":[{"author":{"_account_id":6816,"name":"Jesse Pretorius","email":"jesse@odyssey4.me","username":"jesse-pretorius"},"change_message_id":"bca3a37613ae5091207486d91e21d3161cb077bc","unresolved":true,"context_lines":[{"line_number":35,"context_line":"          allovercloud:"},{"line_number":36,"context_line":"            hosts:"},{"line_number":37,"context_line":"              ubi8:"},{"line_number":38,"context_line":"                ansible_facts:"},{"line_number":39,"context_line":"                  ssh_host_key_rsa_public: AAAATEST"},{"line_number":40,"context_line":"                ctlplane_ip: 10.0.0.1"},{"line_number":41,"context_line":"                ctlplane_hostname: ubi8.ctlplane.localdomain"},{"line_number":42,"context_line":"                internal_api_ip: 10.0.1.1"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"7a667105_bc7beda1","line":39,"range":{"start_line":38,"start_character":0,"end_line":39,"end_character":51},"updated":"2021-04-14 10:00:55.000000000","message":"The precedence of inventory variables is too low. Gathered facts will override them. See https://docs.ansible.com/ansible/latest/user_guide/playbooks_variables.html#variable-precedence-where-should-i-put-a-variable\n\nThis would need to be added via role, play or task var - or a set_fact. I guess that https://review.opendev.org/c/openstack/tripleo-heat-templates/+/786089/2/common/deploy-steps-playbooks-common.yaml is the only way.\n\nFor testing, we could perhaps set_fact, then gather facts so that the two values combine.","commit_id":"595fd74bc14b37347be273f525be5e8a2b2495d1"},{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"ec0331dcb25e5f7dd2241b1badf4b24f43894fca","unresolved":true,"context_lines":[{"line_number":35,"context_line":"          allovercloud:"},{"line_number":36,"context_line":"            hosts:"},{"line_number":37,"context_line":"              ubi8:"},{"line_number":38,"context_line":"                ansible_facts:"},{"line_number":39,"context_line":"                  ssh_host_key_rsa_public: AAAATEST"},{"line_number":40,"context_line":"                ctlplane_ip: 10.0.0.1"},{"line_number":41,"context_line":"                ctlplane_hostname: ubi8.ctlplane.localdomain"},{"line_number":42,"context_line":"                internal_api_ip: 10.0.1.1"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"dcb6c5fc_c33e6f3b","line":39,"range":{"start_line":38,"start_character":0,"end_line":39,"end_character":51},"in_reply_to":"7a667105_bc7beda1","updated":"2021-04-14 11:21:32.000000000","message":"Updating with merging ansible_facts with test value.","commit_id":"595fd74bc14b37347be273f525be5e8a2b2495d1"}],"tripleo_ansible/roles/tripleo_ssh_known_hosts/tasks/main.yml":[{"author":{"_account_id":6926,"name":"Bogdan Dobrelya","email":"bdobreli@redhat.com","username":"bogdando"},"change_message_id":"d70c07ad71099b9464021863e38ddb79c9dce65c","unresolved":true,"context_lines":[{"line_number":49,"context_line":"        ssh_known_hosts_lines: |-"},{"line_number":50,"context_line":"          {% for host in groups[\u0027allovercloud\u0027] | intersect(play_hosts) %}"},{"line_number":51,"context_line":"          {%   set hostdata \u003d hostvars[host] %}"},{"line_number":52,"context_line":"          {%   if \u0027ssh_host_key_rsa_public\u0027 in hostdata[\u0027ansible_facts\u0027] %}"},{"line_number":53,"context_line":"          {%     set entries \u003d [] %}"},{"line_number":54,"context_line":"          {%     set enabled_host_networks \u003d hostdata[\u0027tripleo_role_networks\u0027]|default([]) %}"},{"line_number":55,"context_line":"          {%     for network in enabled_host_networks %}"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"c3c205c5_fbd5cf8e","line":52,"updated":"2021-04-14 09:30:48.000000000","message":"unsure if this should be replaced or complemented to use both methods (new plus legacy facts), see also https://review.opendev.org/c/openstack/tripleo-heat-templates/+/786187","commit_id":"595fd74bc14b37347be273f525be5e8a2b2495d1"},{"author":{"_account_id":6816,"name":"Jesse Pretorius","email":"jesse@odyssey4.me","username":"jesse-pretorius"},"change_message_id":"bca3a37613ae5091207486d91e21d3161cb077bc","unresolved":true,"context_lines":[{"line_number":49,"context_line":"        ssh_known_hosts_lines: |-"},{"line_number":50,"context_line":"          {% for host in groups[\u0027allovercloud\u0027] | intersect(play_hosts) %}"},{"line_number":51,"context_line":"          {%   set hostdata \u003d hostvars[host] %}"},{"line_number":52,"context_line":"          {%   if \u0027ssh_host_key_rsa_public\u0027 in hostdata[\u0027ansible_facts\u0027] %}"},{"line_number":53,"context_line":"          {%     set entries \u003d [] %}"},{"line_number":54,"context_line":"          {%     set enabled_host_networks \u003d hostdata[\u0027tripleo_role_networks\u0027]|default([]) %}"},{"line_number":55,"context_line":"          {%     for network in enabled_host_networks %}"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"fb629a65_23c38f15","line":52,"in_reply_to":"c3c205c5_fbd5cf8e","updated":"2021-04-14 10:00:55.000000000","message":"We may need to do both, but only because we need to use the legacy one in testing.","commit_id":"595fd74bc14b37347be273f525be5e8a2b2495d1"},{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"ec0331dcb25e5f7dd2241b1badf4b24f43894fca","unresolved":true,"context_lines":[{"line_number":49,"context_line":"        ssh_known_hosts_lines: |-"},{"line_number":50,"context_line":"          {% for host in groups[\u0027allovercloud\u0027] | intersect(play_hosts) %}"},{"line_number":51,"context_line":"          {%   set hostdata \u003d hostvars[host] %}"},{"line_number":52,"context_line":"          {%   if \u0027ssh_host_key_rsa_public\u0027 in hostdata[\u0027ansible_facts\u0027] %}"},{"line_number":53,"context_line":"          {%     set entries \u003d [] %}"},{"line_number":54,"context_line":"          {%     set enabled_host_networks \u003d hostdata[\u0027tripleo_role_networks\u0027]|default([]) %}"},{"line_number":55,"context_line":"          {%     for network in enabled_host_networks %}"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"65aadce8_092bc798","line":52,"in_reply_to":"fb629a65_23c38f15","updated":"2021-04-14 11:21:32.000000000","message":"Shouldn\u0027t be needed for testing with latest patch, also not sure if legacy needs to be added here as i don\u0027t find handled for other updated ansible_* vars except a few in skelton role.","commit_id":"595fd74bc14b37347be273f525be5e8a2b2495d1"}]}