)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":23084,"name":"Bartosz Bezak","email":"bartosz@stackhpc.com","username":"b.bezak"},"change_message_id":"6ac9b7ffd46ac419e539b1ecc77d00ba1a68e555","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"5a8444be_b401f3ab","updated":"2026-08-25 09:54:22.000000000","message":"Verified on 11.4.13 via the kolla-side runs, which build images in-run:\nhttps://zuul.opendev.org/t/openstack/build/ec0b2c3ee4e448a9b1528c1c507176ce  (rocky-10)\nhttps://zuul.opendev.org/t/openstack/build/9a64103764fe4d4e85c760e2db7c4398  (ubuntu-noble)\nBoth render host \u003d 192.0.2.1 in mariabackup/my.cnf and pass. This patch\u0027s own\nmariadb jobs pull an 11.4.12 image from quay.io, so they pass either way and\ndo not prove anything yet (tomorrow it will be new mariadb built in periodic jobs)","commit_id":"d3583e0be9b9dc4c9786d0ded1734501e4ec3296"},{"author":{"_account_id":23084,"name":"Bartosz Bezak","email":"bartosz@stackhpc.com","username":"b.bezak"},"change_message_id":"ee22712f862f898f81e25fbb6ea8752564549bfb","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":3,"id":"66ba91a7_e3c1f35f","updated":"2026-08-25 09:53:41.000000000","message":"potentiall follow-ups needed:\n- kolla: drop backup_replica.sh one cycle after this releases\n- mariadb_backup_target is now unused, but keep it so this can be backported,\n  remove it later","commit_id":"d3583e0be9b9dc4c9786d0ded1734501e4ec3296"},{"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":"e5b5965cc36c2fffc15744bd47998bed82ab7da3","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"a350d6f3_e707c8c0","updated":"2026-08-25 11:24:42.000000000","message":"LGTM, just the dangling variable to address","commit_id":"f1f2702bbe812e285f0766a5787159ba92fdce8b"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"4dfb663e4db8acc49f03bc1355a3bd783e99a5ef","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"44198a3d_79275eb3","updated":"2026-08-25 11:20:38.000000000","message":"re-using previous +2 to fix gate","commit_id":"f1f2702bbe812e285f0766a5787159ba92fdce8b"}],"ansible/roles/mariadb/tasks/backup.yml":[{"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":"e5b5965cc36c2fffc15744bd47998bed82ab7da3","unresolved":true,"context_lines":[{"line_number":11,"context_line":""},{"line_number":12,"context_line":"- name: Taking database backup via Mariabackup - backup type {{ mariadb_backup_type }}"},{"line_number":13,"context_line":"  vars:"},{"line_number":14,"context_line":"    cmd: \"{{ \u0027kolla_mariadb_backup.sh\u0027 if mariadb_backup_target \u003d\u003d \u0027active\u0027 else \u0027kolla_mariadb_backup_replica.sh\u0027 }}\""},{"line_number":15,"context_line":"  become: true"},{"line_number":16,"context_line":"  kolla_container:"},{"line_number":17,"context_line":"    action: \"start_container\""}],"source_content_type":"text/x-yaml","patch_set":4,"id":"83ff4654_1c8cb1fc","side":"PARENT","line":14,"range":{"start_line":14,"start_character":42,"end_line":14,"end_character":63},"updated":"2026-08-25 11:24:42.000000000","message":"we can probbly remove this variable from `group_vars/all/mariadb.yml` since it\u0027s unused now ?\n\nOr mark it for removal if we want a deprecation cycle for it ?","commit_id":"2d8d9e2af0d8a3477a8931a6b4fff91c748940bf"},{"author":{"_account_id":23084,"name":"Bartosz Bezak","email":"bartosz@stackhpc.com","username":"b.bezak"},"change_message_id":"51484710840001bf37dcff3ee410b3f86baafca1","unresolved":false,"context_lines":[{"line_number":11,"context_line":""},{"line_number":12,"context_line":"- name: Taking database backup via Mariabackup - backup type {{ mariadb_backup_type }}"},{"line_number":13,"context_line":"  vars:"},{"line_number":14,"context_line":"    cmd: \"{{ \u0027kolla_mariadb_backup.sh\u0027 if mariadb_backup_target \u003d\u003d \u0027active\u0027 else \u0027kolla_mariadb_backup_replica.sh\u0027 }}\""},{"line_number":15,"context_line":"  become: true"},{"line_number":16,"context_line":"  kolla_container:"},{"line_number":17,"context_line":"    action: \"start_container\""}],"source_content_type":"text/x-yaml","patch_set":4,"id":"3f1c1806_5024abe6","side":"PARENT","line":14,"range":{"start_line":14,"start_character":42,"end_line":14,"end_character":63},"in_reply_to":"83ff4654_1c8cb1fc","updated":"2026-08-25 11:26:32.000000000","message":"yes, good idea. I planned to do this as a follow up - check previous comment","commit_id":"2d8d9e2af0d8a3477a8931a6b4fff91c748940bf"}]}
