)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":31542,"name":"Andrew Bonney","email":"andrew.bonney@bbc.co.uk","username":"andrewbonney"},"change_message_id":"0effa2f246097157d055a26e276ad987ecaad45a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"dc202aa0_cc107e6d","updated":"2024-02-12 08:53:57.000000000","message":"Backport to 2023.1 suggested.","commit_id":"ded73432b857e7498b04e0b530ea1eb06ecc0a3c"},{"author":{"_account_id":20865,"name":"Gökhan","email":"gokhan.isik@tubitak.gov.tr","username":"gokhan.isik"},"change_message_id":"364401dc73aa9e0fa4c91c7ce1113b5370d853c0","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"d924999a_296d0595","updated":"2024-02-12 10:30:27.000000000","message":"I tested in my env. It works. Thanks for the patch.","commit_id":"ded73432b857e7498b04e0b530ea1eb06ecc0a3c"}],"scripts/upgrade-utilities/nova-restore-compute-id.yml":[{"author":{"_account_id":31542,"name":"Andrew Bonney","email":"andrew.bonney@bbc.co.uk","username":"andrewbonney"},"change_message_id":"4c6fa06fcc11aeb41cd340bb9f2888f9b1139c50","unresolved":true,"context_lines":[{"line_number":30,"context_line":""},{"line_number":31,"context_line":"        - name: Register hypervisors fact"},{"line_number":32,"context_line":"          ansible.builtin.set_fact:"},{"line_number":33,"context_line":"            nova_hv: \"{{ nova_hypervisors.stdout | from_json | selectattr(\u0027Hypervisor Hostname\u0027, \u0027eq\u0027, ansible_facts[\u0027nodename\u0027]) }}\""},{"line_number":34,"context_line":""},{"line_number":35,"context_line":"        - name: Place node UUID to the expected location"},{"line_number":36,"context_line":"          ansible.builtin.copy:"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"529850da_ccd94c4d","line":33,"updated":"2024-02-12 08:53:42.000000000","message":"Note I changed this to use \u0027nodename\u0027 rather than \u0027fqdn\u0027 as this doesn\u0027t work for us otherwise. Alternatively it may be possible to check the value of both for a match.","commit_id":"ded73432b857e7498b04e0b530ea1eb06ecc0a3c"},{"author":{"_account_id":28619,"name":"Dmitriy Rabotyagov","email":"noonedeadpunk@gmail.com","username":"noonedeadpunk"},"change_message_id":"7e8d3a815999160d9c0bf2e68d822c18864ce4bf","unresolved":true,"context_lines":[{"line_number":30,"context_line":""},{"line_number":31,"context_line":"        - name: Register hypervisors fact"},{"line_number":32,"context_line":"          ansible.builtin.set_fact:"},{"line_number":33,"context_line":"            nova_hv: \"{{ nova_hypervisors.stdout | from_json | selectattr(\u0027Hypervisor Hostname\u0027, \u0027eq\u0027, ansible_facts[\u0027nodename\u0027]) }}\""},{"line_number":34,"context_line":""},{"line_number":35,"context_line":"        - name: Place node UUID to the expected location"},{"line_number":36,"context_line":"          ansible.builtin.copy:"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"ad21e083_19697a4f","line":33,"in_reply_to":"529850da_ccd94c4d","updated":"2024-02-12 09:02:46.000000000","message":"makes sense for sure.\n\nthough it\u0027s weird, as `hypervisor list` output (on contrary to compute list) should be populated with FQDN by libvirt...\n\nBut nodename is good as well.","commit_id":"ded73432b857e7498b04e0b530ea1eb06ecc0a3c"},{"author":{"_account_id":31542,"name":"Andrew Bonney","email":"andrew.bonney@bbc.co.uk","username":"andrewbonney"},"change_message_id":"9159b51c579544350b5bc70ac90d86ea24e81b64","unresolved":true,"context_lines":[{"line_number":30,"context_line":""},{"line_number":31,"context_line":"        - name: Register hypervisors fact"},{"line_number":32,"context_line":"          ansible.builtin.set_fact:"},{"line_number":33,"context_line":"            nova_hv: \"{{ nova_hypervisors.stdout | from_json | selectattr(\u0027Hypervisor Hostname\u0027, \u0027eq\u0027, ansible_facts[\u0027nodename\u0027]) }}\""},{"line_number":34,"context_line":""},{"line_number":35,"context_line":"        - name: Place node UUID to the expected location"},{"line_number":36,"context_line":"          ansible.builtin.copy:"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"777db649_46e02b78","line":33,"in_reply_to":"ad21e083_19697a4f","updated":"2024-02-12 09:04:31.000000000","message":"Yes, the hypervisor list does have FQDNs in, but for us ansible_facts[\u0027fqdn\u0027] is \u0027localhost\u0027. There are a few Ansible issues suggesting this sort of thing is quite common.","commit_id":"ded73432b857e7498b04e0b530ea1eb06ecc0a3c"}]}