)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":14200,"name":"Maksim Malchuk","email":"maksim.malchuk@gmail.com","username":"mmalchuk"},"change_message_id":"a50635f8f91f84df17c7315674e6f8fa67f812fa","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"0bc1ee5e_853076ae","updated":"2025-09-04 11:51:37.000000000","message":"recheck get fresh logs","commit_id":"4ce646cee8a714e1d89778948cfbe7f21901546d"},{"author":{"_account_id":14200,"name":"Maksim Malchuk","email":"maksim.malchuk@gmail.com","username":"mmalchuk"},"change_message_id":"4f6f16d0b553644104f5682eebce34ecd35973be","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":11,"id":"b79bbdf2_ef6d0570","updated":"2025-09-09 12:07:46.000000000","message":"failed with msg: Ansible inventory does not contain the expected group ironic-inspector for service ironic-dnsmasq in ironic.","commit_id":"59eb49f88efd6b83f7431363d31676188e49390a"},{"author":{"_account_id":14200,"name":"Maksim Malchuk","email":"maksim.malchuk@gmail.com","username":"mmalchuk"},"change_message_id":"07a5e13ce785da42aac575f2815e1915c3bb2d8e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":11,"id":"6270b8d6_65514f11","updated":"2025-09-09 14:21:34.000000000","message":"ok","commit_id":"59eb49f88efd6b83f7431363d31676188e49390a"},{"author":{"_account_id":15197,"name":"Pierre Riteau","email":"pierre@stackhpc.com","username":"priteau","status":"StackHPC"},"change_message_id":"cab1adb857cb55999a26f55bd2ef486a94183c69","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":11,"id":"bf394b7d_dd397ea1","updated":"2025-09-09 14:21:19.000000000","message":"recheck kolla-ansible dependant patch has merged","commit_id":"59eb49f88efd6b83f7431363d31676188e49390a"},{"author":{"_account_id":15197,"name":"Pierre Riteau","email":"pierre@stackhpc.com","username":"priteau","status":"StackHPC"},"change_message_id":"9f47b99ba60f4602376918218488cf7de6bafd03","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":11,"id":"649e69a6_b5deb0eb","in_reply_to":"b79bbdf2_ef6d0570","updated":"2025-09-09 12:20:33.000000000","message":"That\u0027s because the Depends-On was dropped. It should work once the kolla-ansible patch merges.","commit_id":"59eb49f88efd6b83f7431363d31676188e49390a"}],"ansible/overcloud-introspection-rules.yml":[{"author":{"_account_id":35263,"name":"Matt Crees","email":"mattc@stackhpc.com","username":"mattcrees"},"change_message_id":"82422562b6d5ed07c95441acc1784e7a1262668d","unresolved":true,"context_lines":[{"line_number":54,"context_line":"        openstack image show \u0027{{ ipa_images_ramdisk_name }}\u0027 -f value -c id"},{"line_number":55,"context_line":"      changed_when: False"},{"line_number":56,"context_line":"      register: ipa_ramdisk_id"},{"line_number":57,"context_line":"      environment: \"{{ openstack_auth_env }}\""}],"source_content_type":"text/x-yaml","patch_set":10,"id":"7c4c5e0a_6d805b90","line":57,"updated":"2025-09-09 09:12:24.000000000","message":"Given we\u0027re dropping creating the introspection rules, it looks to me we can just drop this whole playbook. The remaining tasks only gather image uuids that go unused.","commit_id":"48892328abedeae864da1da6a3c8497403f3a780"},{"author":{"_account_id":35263,"name":"Matt Crees","email":"mattc@stackhpc.com","username":"mattcrees"},"change_message_id":"2d5a7c9ddd3b60c761b86970338d2a6c57d71738","unresolved":true,"context_lines":[{"line_number":54,"context_line":"        openstack image show \u0027{{ ipa_images_ramdisk_name }}\u0027 -f value -c id"},{"line_number":55,"context_line":"      changed_when: False"},{"line_number":56,"context_line":"      register: ipa_ramdisk_id"},{"line_number":57,"context_line":"      environment: \"{{ openstack_auth_env }}\""}],"source_content_type":"text/x-yaml","patch_set":10,"id":"181ea70f_7196aaae","line":57,"in_reply_to":"168791e6_8c7668e6","updated":"2025-09-09 09:20:26.000000000","message":"I see. In that case, I think it would be better to just comment out lines 58+ and leave a comment pointing to that patch.","commit_id":"48892328abedeae864da1da6a3c8497403f3a780"},{"author":{"_account_id":35263,"name":"Matt Crees","email":"mattc@stackhpc.com","username":"mattcrees"},"change_message_id":"6893d2c30d28a7d2c549b37b19b8c363fd06ac54","unresolved":false,"context_lines":[{"line_number":54,"context_line":"        openstack image show \u0027{{ ipa_images_ramdisk_name }}\u0027 -f value -c id"},{"line_number":55,"context_line":"      changed_when: False"},{"line_number":56,"context_line":"      register: ipa_ramdisk_id"},{"line_number":57,"context_line":"      environment: \"{{ openstack_auth_env }}\""}],"source_content_type":"text/x-yaml","patch_set":10,"id":"b302cd48_03f1cc30","line":57,"in_reply_to":"181ea70f_7196aaae","updated":"2025-09-09 09:38:29.000000000","message":"Happy to stick with this so we don\u0027t need to send CI round again, given we\u0027ll be aiming to merge the follow-up soon anyway.","commit_id":"48892328abedeae864da1da6a3c8497403f3a780"},{"author":{"_account_id":15197,"name":"Pierre Riteau","email":"pierre@stackhpc.com","username":"priteau","status":"StackHPC"},"change_message_id":"0e7e65c97c764b6b3062490f1d3f744533697139","unresolved":true,"context_lines":[{"line_number":54,"context_line":"        openstack image show \u0027{{ ipa_images_ramdisk_name }}\u0027 -f value -c id"},{"line_number":55,"context_line":"      changed_when: False"},{"line_number":56,"context_line":"      register: ipa_ramdisk_id"},{"line_number":57,"context_line":"      environment: \"{{ openstack_auth_env }}\""}],"source_content_type":"text/x-yaml","patch_set":10,"id":"168791e6_8c7668e6","line":57,"in_reply_to":"7c4c5e0a_6d805b90","updated":"2025-09-09 09:19:21.000000000","message":"I have kept as much code as possible because I thought we would still support the rules through the new built-in inspector. See https://review.opendev.org/c/openstack/kayobe/+/939147","commit_id":"48892328abedeae864da1da6a3c8497403f3a780"}]}
