)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":39327,"name":"Rowan Johns","display_name":"Rowan Johns","email":"rowan@stackhpc.com","username":"rowanjohns"},"change_message_id":"954946f1dd061a6c221ab3fcb1daf02627ad29e4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"c09094ad_cbe9b438","updated":"2026-09-11 15:12:35.000000000","message":"kolla-packages hosts still need resolving, just pushing for progress check","commit_id":"97322255311e29eb48bed3da8cbb0aabd0d63fc9"},{"author":{"_account_id":28048,"name":"Will Szumski","email":"will@stackhpc.com","username":"jovial"},"change_message_id":"0eac5827c2c3102593249a4bfd263623ed0883ee","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":12,"id":"39a1e86a_bd34c991","updated":"2026-09-14 12:59:19.000000000","message":"If CI is happy, I\u0027m happy","commit_id":"75795cb2bbf6ea8dcdf723b34dac4be716e620f6"},{"author":{"_account_id":14200,"name":"Maksim Malchuk","email":"maksim.malchuk@gmail.com","username":"mmalchuk"},"change_message_id":"473ddc6d958f9d74996eab80f1778a875ee3b712","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":20,"id":"135d8264_29f8afe2","updated":"2026-09-17 19:09:13.000000000","message":"recheck POST_FAILURE","commit_id":"17494b93dff05b3b48fd780bce42d3394cce5144"}],"ansible/install-packages.yml":[{"author":{"_account_id":28048,"name":"Will Szumski","email":"will@stackhpc.com","username":"jovial"},"change_message_id":"0758f84b4320f0851d16a3105be35455d7c7b88a","unresolved":true,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"- name: Install apt"},{"line_number":3,"context_line":"  ansible.builtin.import_playbook: apt.yml"},{"line_number":4,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":4,"id":"4f9d5d73_cef94053","line":1,"updated":"2026-09-11 14:03:15.000000000","message":"Don\u0027t forget kolla-packages.yml","commit_id":"4e9b7126e1be577404a39948b8137c133a00304d"},{"author":{"_account_id":39327,"name":"Rowan Johns","display_name":"Rowan Johns","email":"rowan@stackhpc.com","username":"rowanjohns"},"change_message_id":"954946f1dd061a6c221ab3fcb1daf02627ad29e4","unresolved":false,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"- name: Install apt"},{"line_number":3,"context_line":"  ansible.builtin.import_playbook: apt.yml"},{"line_number":4,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":4,"id":"e51d7c87_1197c0ab","line":1,"in_reply_to":"4f9d5d73_cef94053","updated":"2026-09-11 15:12:35.000000000","message":"Done","commit_id":"4e9b7126e1be577404a39948b8137c133a00304d"},{"author":{"_account_id":28048,"name":"Will Szumski","email":"will@stackhpc.com","username":"jovial"},"change_message_id":"0758f84b4320f0851d16a3105be35455d7c7b88a","unresolved":true,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"- name: Install apt"},{"line_number":3,"context_line":"  ansible.builtin.import_playbook: apt.yml"},{"line_number":4,"context_line":""},{"line_number":5,"context_line":"- name: Install dnf"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"bf5e6209_fdf7692f","line":2,"updated":"2026-09-11 14:03:15.000000000","message":"I\u0027d keep these separate as they are configuring the repos.","commit_id":"4e9b7126e1be577404a39948b8137c133a00304d"},{"author":{"_account_id":39327,"name":"Rowan Johns","display_name":"Rowan Johns","email":"rowan@stackhpc.com","username":"rowanjohns"},"change_message_id":"954946f1dd061a6c221ab3fcb1daf02627ad29e4","unresolved":false,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"- name: Install apt"},{"line_number":3,"context_line":"  ansible.builtin.import_playbook: apt.yml"},{"line_number":4,"context_line":""},{"line_number":5,"context_line":"- name: Install dnf"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"08bdd1c9_f3ce5d09","line":2,"in_reply_to":"bf5e6209_fdf7692f","updated":"2026-09-11 15:12:35.000000000","message":"Done","commit_id":"4e9b7126e1be577404a39948b8137c133a00304d"}],"ansible/roles/dev-tools/defaults/main.yml":[{"author":{"_account_id":35263,"name":"Matt Crees","email":"mattc@stackhpc.com","username":"mattcrees"},"change_message_id":"3f5949e176ecb7b9373be3938db2b42aec0765cf","unresolved":true,"context_lines":[{"line_number":15,"context_line":"  - acl"},{"line_number":16,"context_line":"  # NOTE(rowanjohns): kernel-modules-extra-matched needs to be installed on RedHat"},{"line_number":17,"context_line":"  # hosts so that iptables can be used."},{"line_number":18,"context_line":"  - \"{{ \u0027kernel-modules-extra-matched\u0027 if ansible_facts.os_family \u003d\u003d \u0027RedHat\u0027 else \u0027\u0027 }}\""},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"# List of packages to install."},{"line_number":21,"context_line":"dev_tools_packages: \"{{ (dev_tools_packages_default + dev_tools_packages_extra + dev_tools_packages_system) | select | list }}\""}],"source_content_type":"text/x-yaml","patch_set":13,"id":"71183383_0f51da74","line":18,"updated":"2026-09-15 14:19:08.000000000","message":"Combining into `install-packages.yml` means we now install kolla packages onto every host. As such, we don\u0027t need to add kernel-modules-extra-matched to dev_tools, because it is installed in kolla-packages: https://github.com/openstack/ansible-collection-kolla/blob/master/roles/packages/defaults/main.yml#L19","commit_id":"bd631dfa9df08ddec259ebac51e5f4920e75d243"},{"author":{"_account_id":35263,"name":"Matt Crees","email":"mattc@stackhpc.com","username":"mattcrees"},"change_message_id":"25fb120e9bfdcfb07cfe9ce201d3bcd4ee8efae5","unresolved":false,"context_lines":[{"line_number":15,"context_line":"  - acl"},{"line_number":16,"context_line":"  # NOTE(rowanjohns): kernel-modules-extra-matched needs to be installed on RedHat"},{"line_number":17,"context_line":"  # hosts so that iptables can be used."},{"line_number":18,"context_line":"  - \"{{ \u0027kernel-modules-extra-matched\u0027 if ansible_facts.os_family \u003d\u003d \u0027RedHat\u0027 else \u0027\u0027 }}\""},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"# List of packages to install."},{"line_number":21,"context_line":"dev_tools_packages: \"{{ (dev_tools_packages_default + dev_tools_packages_extra + dev_tools_packages_system) | select | list }}\""}],"source_content_type":"text/x-yaml","patch_set":13,"id":"3484f48d_fa8c9ab2","line":18,"in_reply_to":"06049ef1_4f45e25c","updated":"2026-09-15 14:49:01.000000000","message":"Ah I didn\u0027t notice that, yes that would mean it gets skipped. Ignore my suggestion :)","commit_id":"bd631dfa9df08ddec259ebac51e5f4920e75d243"},{"author":{"_account_id":39327,"name":"Rowan Johns","display_name":"Rowan Johns","email":"rowan@stackhpc.com","username":"rowanjohns"},"change_message_id":"78b9e904fbc960b663aacf2e6985be3db208d72f","unresolved":true,"context_lines":[{"line_number":15,"context_line":"  - acl"},{"line_number":16,"context_line":"  # NOTE(rowanjohns): kernel-modules-extra-matched needs to be installed on RedHat"},{"line_number":17,"context_line":"  # hosts so that iptables can be used."},{"line_number":18,"context_line":"  - \"{{ \u0027kernel-modules-extra-matched\u0027 if ansible_facts.os_family \u003d\u003d \u0027RedHat\u0027 else \u0027\u0027 }}\""},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"# List of packages to install."},{"line_number":21,"context_line":"dev_tools_packages: \"{{ (dev_tools_packages_default + dev_tools_packages_extra + dev_tools_packages_system) | select | list }}\""}],"source_content_type":"text/x-yaml","patch_set":13,"id":"06049ef1_4f45e25c","line":18,"in_reply_to":"71183383_0f51da74","updated":"2026-09-15 14:32:07.000000000","message":"The kolla-packages.yml playbook only has \"overcloud\" listed for hosts. Wouldn\u0027t this mean that the play gets skipped on any other hosts?","commit_id":"bd631dfa9df08ddec259ebac51e5f4920e75d243"}]}