)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":37203,"name":"Bertrand Lanson","display_name":"Bertrand Lanson","email":"bertrand.lanson@infomaniak.com","username":"lanson","status":"Infomaniak Network SA"},"change_message_id":"594ab339d4ff3f37ee3ce5474c2bfbf30b660491","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"61ba5899_6ff5ef9d","updated":"2026-01-26 16:32:59.000000000","message":"recheck weekend chaos maybe it works better now","commit_id":"d46dd522892bb5c694db1c6cb8afbfcff1d4ba67"},{"author":{"_account_id":37203,"name":"Bertrand Lanson","display_name":"Bertrand Lanson","email":"bertrand.lanson@infomaniak.com","username":"lanson","status":"Infomaniak Network SA"},"change_message_id":"bcf41d0adb5647721d32ca2030fefc658d7b2e2f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":18,"id":"9cb6594d_ac70d534","updated":"2026-02-03 20:24:39.000000000","message":"recheck unrelated, make it green pretty please","commit_id":"b339c46be1a1e8f14a7232ca67f66a7bd58ea0fb"},{"author":{"_account_id":37203,"name":"Bertrand Lanson","display_name":"Bertrand Lanson","email":"bertrand.lanson@infomaniak.com","username":"lanson","status":"Infomaniak Network SA"},"change_message_id":"d8413a1a584d212eb699c56b3c1116e0b56d52c2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":25,"id":"3000a7a8_e66dce85","updated":"2026-02-12 08:44:10.000000000","message":"This has been fixed in https://review.opendev.org/c/openstack/kolla-ansible/+/976262","commit_id":"0d587eaf113c04c570af1ae734eba6beb4bd2b67"}],"ansible/roles/service-check-containers/tasks/iterated.yml":[{"author":{"_account_id":17669,"name":"Doug Szumski","email":"doug@stackhpc.com","username":"DougSzumski"},"change_message_id":"0b43ab1c707bd569529d6ba28c57aa6ca6551ce6","unresolved":true,"context_lines":[{"line_number":23,"context_line":"    command: \"{{ service.command | default(omit) }}\""},{"line_number":24,"context_line":"    cgroupns_mode: \"{{ service.cgroupns_mode | default(omit) }}\""},{"line_number":25,"context_line":"  loop:"},{"line_number":26,"context_line":"    - \"{{ range(1, (service.iterate_var | int) + 1) | list }}\""},{"line_number":27,"context_line":"  register: container_check"},{"line_number":28,"context_line":""},{"line_number":29,"context_line":"# NOTE(yoctozepto): Must be a separate task because one cannot see the whole"}],"source_content_type":"text/x-yaml","patch_set":14,"id":"8cb2a386_075d55b0","side":"PARENT","line":26,"range":{"start_line":26,"start_character":4,"end_line":26,"end_character":7},"updated":"2026-02-02 18:04:57.000000000","message":"good spot there - otherwise it iterates over a single item, rather than the items in the list","commit_id":"77e9d23a20fa39fe49f03210b6a20bc463c31078"},{"author":{"_account_id":37203,"name":"Bertrand Lanson","display_name":"Bertrand Lanson","email":"bertrand.lanson@infomaniak.com","username":"lanson","status":"Infomaniak Network SA"},"change_message_id":"5b879b3edcc65c16890b0259f7af5e67501dab2b","unresolved":false,"context_lines":[{"line_number":23,"context_line":"    command: \"{{ service.command | default(omit) }}\""},{"line_number":24,"context_line":"    cgroupns_mode: \"{{ service.cgroupns_mode | default(omit) }}\""},{"line_number":25,"context_line":"  loop:"},{"line_number":26,"context_line":"    - \"{{ range(1, (service.iterate_var | int) + 1) | list }}\""},{"line_number":27,"context_line":"  register: container_check"},{"line_number":28,"context_line":""},{"line_number":29,"context_line":"# NOTE(yoctozepto): Must be a separate task because one cannot see the whole"}],"source_content_type":"text/x-yaml","patch_set":14,"id":"03a36d8d_449ecc2e","side":"PARENT","line":26,"range":{"start_line":26,"start_character":4,"end_line":26,"end_character":7},"in_reply_to":"4c135b4f_9a3f91c5","updated":"2026-02-05 10:57:19.000000000","message":"Done","commit_id":"77e9d23a20fa39fe49f03210b6a20bc463c31078"},{"author":{"_account_id":37203,"name":"Bertrand Lanson","display_name":"Bertrand Lanson","email":"bertrand.lanson@infomaniak.com","username":"lanson","status":"Infomaniak Network SA"},"change_message_id":"fe525632dc85be8e5eab0362450a91384b2be131","unresolved":true,"context_lines":[{"line_number":23,"context_line":"    command: \"{{ service.command | default(omit) }}\""},{"line_number":24,"context_line":"    cgroupns_mode: \"{{ service.cgroupns_mode | default(omit) }}\""},{"line_number":25,"context_line":"  loop:"},{"line_number":26,"context_line":"    - \"{{ range(1, (service.iterate_var | int) + 1) | list }}\""},{"line_number":27,"context_line":"  register: container_check"},{"line_number":28,"context_line":""},{"line_number":29,"context_line":"# NOTE(yoctozepto): Must be a separate task because one cannot see the whole"}],"source_content_type":"text/x-yaml","patch_set":14,"id":"4c135b4f_9a3f91c5","side":"PARENT","line":26,"range":{"start_line":26,"start_character":4,"end_line":26,"end_character":7},"in_reply_to":"8cb2a386_075d55b0","updated":"2026-02-03 20:25:07.000000000","message":"yh that was my first issue and I\u0027m not gonna lie it took me a while to get it","commit_id":"77e9d23a20fa39fe49f03210b6a20bc463c31078"},{"author":{"_account_id":17669,"name":"Doug Szumski","email":"doug@stackhpc.com","username":"DougSzumski"},"change_message_id":"59daff1e4ca4f3f625ecf7306291ab2197ab1c57","unresolved":true,"context_lines":[{"line_number":46,"context_line":"    handler: \"Restart {{ service_name }} container\""},{"line_number":47,"context_line":"  debug:"},{"line_number":48,"context_line":"    msg: \"{{ (\u0027Notifying handler: \u0027 + handler) if container_check is changed else \u0027Nothing changed - not notifying handlers\u0027 }}\""},{"line_number":49,"context_line":"  changed_when: container_check is changed"},{"line_number":50,"context_line":"  notify: \"{{ handler }}\""}],"source_content_type":"text/x-yaml","patch_set":21,"id":"e1a400e8_1f21098a","line":49,"updated":"2026-02-05 11:10:23.000000000","message":"I think we need to loop over ``container_check.results`` here and then adjust this to:\n``changed_when: item is changed`` ?\nSame for the debug msg.","commit_id":"fd7ac6b8bbe56dc895be725da4e8d252ab33c16c"},{"author":{"_account_id":37203,"name":"Bertrand Lanson","display_name":"Bertrand Lanson","email":"bertrand.lanson@infomaniak.com","username":"lanson","status":"Infomaniak Network SA"},"change_message_id":"290e3863834f914b746d2f241bd05158070d1d62","unresolved":false,"context_lines":[{"line_number":46,"context_line":"    handler: \"Restart {{ service_name }} container\""},{"line_number":47,"context_line":"  debug:"},{"line_number":48,"context_line":"    msg: \"{{ (\u0027Notifying handler: \u0027 + handler) if container_check is changed else \u0027Nothing changed - not notifying handlers\u0027 }}\""},{"line_number":49,"context_line":"  changed_when: container_check is changed"},{"line_number":50,"context_line":"  notify: \"{{ handler }}\""}],"source_content_type":"text/x-yaml","patch_set":21,"id":"329342d4_f923f50c","line":49,"in_reply_to":"7e38e76f_13ea00a1","updated":"2026-02-05 11:18:25.000000000","message":"Done","commit_id":"fd7ac6b8bbe56dc895be725da4e8d252ab33c16c"},{"author":{"_account_id":17669,"name":"Doug Szumski","email":"doug@stackhpc.com","username":"DougSzumski"},"change_message_id":"505da3955e7a558a78f523527c695cfb670c5ab5","unresolved":true,"context_lines":[{"line_number":46,"context_line":"    handler: \"Restart {{ service_name }} container\""},{"line_number":47,"context_line":"  debug:"},{"line_number":48,"context_line":"    msg: \"{{ (\u0027Notifying handler: \u0027 + handler) if container_check is changed else \u0027Nothing changed - not notifying handlers\u0027 }}\""},{"line_number":49,"context_line":"  changed_when: container_check is changed"},{"line_number":50,"context_line":"  notify: \"{{ handler }}\""}],"source_content_type":"text/x-yaml","patch_set":21,"id":"7e38e76f_13ea00a1","line":49,"in_reply_to":"e1a400e8_1f21098a","updated":"2026-02-05 11:11:21.000000000","message":"ah - I guess that is what you are experimenting with above :D","commit_id":"fd7ac6b8bbe56dc895be725da4e8d252ab33c16c"}]}