)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":15197,"name":"Pierre Riteau","email":"pierre@stackhpc.com","username":"priteau","status":"StackHPC"},"change_message_id":"1f1b9eb729b7265aa6e7245a787d859476e6e9b0","unresolved":true,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"[CI] Fix testing inventory template"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"For some reason, Ansible now  includes localhost"},{"line_number":10,"context_line":"in hostvars without the ansible_host variable set, which"},{"line_number":11,"context_line":"unexpectedly breaks our test inventory render."},{"line_number":12,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":12,"id":"054c5d3c_eebc4bb5","line":9,"updated":"2024-12-07 17:31:27.000000000","message":"Nit: duplicate space","commit_id":"90fca730f0c1d406394857002a257b3deff36d26"},{"author":{"_account_id":27339,"name":"Michal Arbet","email":"michal.arbet@ultimum.io","username":"michalarbet"},"change_message_id":"25ad33a3bd4f30008786a864ca313cbe57ca29e4","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"[CI] Fix testing inventory template"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"For some reason, Ansible now  includes localhost"},{"line_number":10,"context_line":"in hostvars without the ansible_host variable set, which"},{"line_number":11,"context_line":"unexpectedly breaks our test inventory render."},{"line_number":12,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":12,"id":"43e9c6fe_6297cb7c","line":9,"in_reply_to":"054c5d3c_eebc4bb5","updated":"2024-12-07 18:04:46.000000000","message":"Done","commit_id":"90fca730f0c1d406394857002a257b3deff36d26"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":27339,"name":"Michal Arbet","email":"michal.arbet@ultimum.io","username":"michalarbet"},"change_message_id":"024d3920d252b227571895af943440630380b837","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"6e069046_d6e4dcb8","updated":"2024-12-07 08:56:30.000000000","message":"recheck weird ansible_host error","commit_id":"3e2b3c01ff4aa1d46a9d564ca52cf0c2b07ed820"},{"author":{"_account_id":15197,"name":"Pierre Riteau","email":"pierre@stackhpc.com","username":"priteau","status":"StackHPC"},"change_message_id":"890ba8dd6022e9541c6fc5682b5c2f0947be34bc","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":12,"id":"c10fb83c_6e2f3659","updated":"2024-12-07 17:31:14.000000000","message":"You also need to fix tests/templates/ceph-inventory.j2 and potentially tests/templates/globals-default.j2.","commit_id":"90fca730f0c1d406394857002a257b3deff36d26"},{"author":{"_account_id":15197,"name":"Pierre Riteau","email":"pierre@stackhpc.com","username":"priteau","status":"StackHPC"},"change_message_id":"03e855281160a0bead22e32da1618e5b0991ab5c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":12,"id":"6b54c843_31fc4e75","in_reply_to":"5ffeeda3_7413f0cc","updated":"2024-12-09 12:47:31.000000000","message":"Done in https://review.opendev.org/c/openstack/kolla-ansible/+/937305","commit_id":"90fca730f0c1d406394857002a257b3deff36d26"},{"author":{"_account_id":15197,"name":"Pierre Riteau","email":"pierre@stackhpc.com","username":"priteau","status":"StackHPC"},"change_message_id":"11635f1d043cac8c15471c0ebbd8afcac47c205b","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":12,"id":"5ffeeda3_7413f0cc","in_reply_to":"c10fb83c_6e2f3659","updated":"2024-12-08 12:05:04.000000000","message":"Please fix cephadm jobs too.","commit_id":"90fca730f0c1d406394857002a257b3deff36d26"},{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"1a15dc4bfed24a2df8d6c1ce5c726be7953c15cd","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":16,"id":"fd20d5c0_055a915e","updated":"2024-12-07 19:04:37.000000000","message":"isn’t 2.6.14 old like museum old?","commit_id":"c84aebe020f90a148cbd74b924ac8c3831cb31cd"},{"author":{"_account_id":27339,"name":"Michal Arbet","email":"michal.arbet@ultimum.io","username":"michalarbet"},"change_message_id":"8c7a443e2f0e20379cc19daff10726c3f488556d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":16,"id":"91f752ee_c4ce2a8c","in_reply_to":"2d6ad1ae_37b02bd4","updated":"2024-12-07 19:48:19.000000000","message":"Done","commit_id":"c84aebe020f90a148cbd74b924ac8c3831cb31cd"},{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"2bd7905dcfb95889fd3d0260dc7c3d0008a83125","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":16,"id":"2d6ad1ae_37b02bd4","in_reply_to":"fd20d5c0_055a915e","updated":"2024-12-07 19:05:35.000000000","message":"Ah, 2.16.14 - fix the commit message please","commit_id":"c84aebe020f90a148cbd74b924ac8c3831cb31cd"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"061991de200e000db734ca3accfcbcc39aefbd79","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":17,"id":"ab1a7583_1121f116","updated":"2024-12-08 10:38:43.000000000","message":"IIUC the ansible version being used in this case is the one embedded in zuul, we have no control over that\n\nreading the CVE contents, longer term refactoring the direct use of hostvars for templating might be advisable","commit_id":"0298699a4b33bf810862bf61691f9c127a1ea3c2"},{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"830479dbe4d9d2b0802c1f70f0367c54a6503f4c","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":17,"id":"8c91e540_c68bb740","updated":"2024-12-08 06:34:45.000000000","message":"Maybe it’s easier to pin for now?","commit_id":"0298699a4b33bf810862bf61691f9c127a1ea3c2"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"deecf44b95aca170a869f68815e7e375419076fd","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":17,"id":"fca01e8a_36582842","updated":"2024-12-07 21:40:42.000000000","message":"more fixes needed like priteau mentioned, see e.g. kolla-ansible-rocky9-cephadm https://zuul.opendev.org/t/openstack/build/c00ce80a7e5d4794a9067a27f4309dc9 but let\u0027s fix the main CI jobs with this first","commit_id":"0298699a4b33bf810862bf61691f9c127a1ea3c2"},{"author":{"_account_id":27339,"name":"Michal Arbet","email":"michal.arbet@ultimum.io","username":"michalarbet"},"change_message_id":"50ef409a1b11ba5dcf7a14340fd07c978e8c7e18","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":17,"id":"49cb8a83_934fc259","updated":"2024-12-08 13:15:39.000000000","message":"recheck","commit_id":"0298699a4b33bf810862bf61691f9c127a1ea3c2"},{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"f450d0f3a142adf5fd767effabc120732ba8ae45","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":17,"id":"f902587c_0b8195ff","updated":"2024-12-08 11:52:10.000000000","message":"recheck coverage job post failure","commit_id":"0298699a4b33bf810862bf61691f9c127a1ea3c2"},{"author":{"_account_id":27339,"name":"Michal Arbet","email":"michal.arbet@ultimum.io","username":"michalarbet"},"change_message_id":"40e976ecdbf70d44b34d7052b2b952805830e094","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":17,"id":"4876408b_d9e93c41","updated":"2024-12-08 09:00:20.000000000","message":"recheck podman unrelated","commit_id":"0298699a4b33bf810862bf61691f9c127a1ea3c2"},{"author":{"_account_id":27339,"name":"Michal Arbet","email":"michal.arbet@ultimum.io","username":"michalarbet"},"change_message_id":"02af3b852ff882d2556b72919c450f563c775e01","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":17,"id":"5b4c0a8a_5d63d82a","updated":"2024-12-08 11:28:44.000000000","message":"recheck tox post-failure","commit_id":"0298699a4b33bf810862bf61691f9c127a1ea3c2"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"73fd86baf006662457ea2761eb2930f47076c1df","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":17,"id":"7b78f9a7_75d8a25c","updated":"2024-12-08 10:37:20.000000000","message":"recheck unrelated unstable test","commit_id":"0298699a4b33bf810862bf61691f9c127a1ea3c2"},{"author":{"_account_id":15197,"name":"Pierre Riteau","email":"pierre@stackhpc.com","username":"priteau","status":"StackHPC"},"change_message_id":"03e855281160a0bead22e32da1618e5b0991ab5c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":17,"id":"b6ad450e_015a645b","in_reply_to":"2c27b7d0_d9e76695","updated":"2024-12-09 12:47:31.000000000","message":"As mentioned elsewhere, this is Zuul\u0027s Ansible version that is causing issues.","commit_id":"0298699a4b33bf810862bf61691f9c127a1ea3c2"},{"author":{"_account_id":27339,"name":"Michal Arbet","email":"michal.arbet@ultimum.io","username":"michalarbet"},"change_message_id":"3e583c6dbe78b312f7afa703b06fe3b34120cd0d","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":17,"id":"2c27b7d0_d9e76695","in_reply_to":"8c91e540_c68bb740","updated":"2024-12-08 09:02:04.000000000","message":"I don\u0027t think it\u0027s necessary, and this filter is harmless anyway, because we definitely don\u0027t want localhost in inventory.","commit_id":"0298699a4b33bf810862bf61691f9c127a1ea3c2"}],"tests/templates/inventory.j2":[{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"16b1a98d97eeaec59ecf316f70b8abcb847fd0ba","unresolved":true,"context_lines":[{"line_number":7,"context_line":""},{"line_number":8,"context_line":"[control]"},{"line_number":9,"context_line":"{% if scenario \u003d\u003d \u0027masakari\u0027 %}"},{"line_number":10,"context_line":"{% for host in hostvars if host in [\u0027primary\u0027] | reject(\u0027equalto\u0027, \u0027localhost\u0027) %}"},{"line_number":11,"context_line":"{{ host }} ansible_host\u003d{{ hostvars[host][\u0027ansible_host\u0027] }}"},{"line_number":12,"context_line":"{% endfor %}"},{"line_number":13,"context_line":"{% else %}"}],"source_content_type":"text/x-jinja2","patch_set":12,"id":"ef069fea_83f92f8a","line":10,"range":{"start_line":10,"start_character":24,"end_line":10,"end_character":46},"updated":"2024-12-07 17:02:26.000000000","message":"should not be needed here with the filter in place?","commit_id":"90fca730f0c1d406394857002a257b3deff36d26"},{"author":{"_account_id":15197,"name":"Pierre Riteau","email":"pierre@stackhpc.com","username":"priteau","status":"StackHPC"},"change_message_id":"c4b64e1828d4af989de062ea0879f869a3ecb91a","unresolved":true,"context_lines":[{"line_number":7,"context_line":""},{"line_number":8,"context_line":"[control]"},{"line_number":9,"context_line":"{% if scenario \u003d\u003d \u0027masakari\u0027 %}"},{"line_number":10,"context_line":"{% for host in hostvars if host in [\u0027primary\u0027] | reject(\u0027equalto\u0027, \u0027localhost\u0027) %}"},{"line_number":11,"context_line":"{{ host }} ansible_host\u003d{{ hostvars[host][\u0027ansible_host\u0027] }}"},{"line_number":12,"context_line":"{% endfor %}"},{"line_number":13,"context_line":"{% else %}"}],"source_content_type":"text/x-jinja2","patch_set":12,"id":"fba9abd1_e003064a","line":10,"range":{"start_line":10,"start_character":24,"end_line":10,"end_character":46},"in_reply_to":"ef069fea_83f92f8a","updated":"2024-12-07 17:29:43.000000000","message":"Indeed.","commit_id":"90fca730f0c1d406394857002a257b3deff36d26"},{"author":{"_account_id":27339,"name":"Michal Arbet","email":"michal.arbet@ultimum.io","username":"michalarbet"},"change_message_id":"25ad33a3bd4f30008786a864ca313cbe57ca29e4","unresolved":false,"context_lines":[{"line_number":7,"context_line":""},{"line_number":8,"context_line":"[control]"},{"line_number":9,"context_line":"{% if scenario \u003d\u003d \u0027masakari\u0027 %}"},{"line_number":10,"context_line":"{% for host in hostvars if host in [\u0027primary\u0027] | reject(\u0027equalto\u0027, \u0027localhost\u0027) %}"},{"line_number":11,"context_line":"{{ host }} ansible_host\u003d{{ hostvars[host][\u0027ansible_host\u0027] }}"},{"line_number":12,"context_line":"{% endfor %}"},{"line_number":13,"context_line":"{% else %}"}],"source_content_type":"text/x-jinja2","patch_set":12,"id":"bc13e989_402f1bfa","line":10,"range":{"start_line":10,"start_character":24,"end_line":10,"end_character":46},"in_reply_to":"fba9abd1_e003064a","updated":"2024-12-07 18:04:46.000000000","message":"Done","commit_id":"90fca730f0c1d406394857002a257b3deff36d26"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"16b1a98d97eeaec59ecf316f70b8abcb847fd0ba","unresolved":true,"context_lines":[{"line_number":27,"context_line":""},{"line_number":28,"context_line":"[compute]"},{"line_number":29,"context_line":"{% if scenario \u003d\u003d \u0027masakari\u0027 %}"},{"line_number":30,"context_line":"{% for host in hostvars if host in [\u0027ternary1\u0027, \u0027ternary2\u0027] | reject(\u0027equalto\u0027, \u0027localhost\u0027) %}"},{"line_number":31,"context_line":"{{ host }} ansible_host\u003d{{ hostvars[host][\u0027ansible_host\u0027] }}"},{"line_number":32,"context_line":"{% endfor %}"},{"line_number":33,"context_line":"{% else %}"}],"source_content_type":"text/x-jinja2","patch_set":12,"id":"53c1492b_dc2ca196","line":30,"updated":"2024-12-07 17:02:26.000000000","message":"dito","commit_id":"90fca730f0c1d406394857002a257b3deff36d26"},{"author":{"_account_id":27339,"name":"Michal Arbet","email":"michal.arbet@ultimum.io","username":"michalarbet"},"change_message_id":"25ad33a3bd4f30008786a864ca313cbe57ca29e4","unresolved":false,"context_lines":[{"line_number":27,"context_line":""},{"line_number":28,"context_line":"[compute]"},{"line_number":29,"context_line":"{% if scenario \u003d\u003d \u0027masakari\u0027 %}"},{"line_number":30,"context_line":"{% for host in hostvars if host in [\u0027ternary1\u0027, \u0027ternary2\u0027] | reject(\u0027equalto\u0027, \u0027localhost\u0027) %}"},{"line_number":31,"context_line":"{{ host }} ansible_host\u003d{{ hostvars[host][\u0027ansible_host\u0027] }}"},{"line_number":32,"context_line":"{% endfor %}"},{"line_number":33,"context_line":"{% else %}"}],"source_content_type":"text/x-jinja2","patch_set":12,"id":"8701572e_7ff49c8b","line":30,"in_reply_to":"53c1492b_dc2ca196","updated":"2024-12-07 18:04:46.000000000","message":"Done","commit_id":"90fca730f0c1d406394857002a257b3deff36d26"}]}