)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":32553,"name":"Sven Kieske","email":"sven_oss@posteo.de","username":"skieske"},"change_message_id":"2949bbbef4bca3a53bc84089adc7a4cf6ed9e4da","unresolved":true,"context_lines":[{"line_number":12,"context_line":"changed command names (mysql* commands have been removed and only"},{"line_number":13,"context_line":"available in -compat package which we are not installing)"},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"[1]: https://mariadb.org/11-4-lts/"},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"Depends-On: https://review.opendev.org/c/openstack/kolla-ansible/+/927096"},{"line_number":18,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":14,"id":"22554dfd_724f15d8","line":15,"range":{"start_line":15,"start_character":0,"end_line":15,"end_character":2},"updated":"2024-09-02 08:48:36.000000000","message":"slightly off topic:\n\nlooking at that blog post I read:\n\n\u003e Early February, we announced that we were adjusting the MariaDB Server release model. As part of that, we already announced our move towards a yearly LTS cycle.\n\nIf they do \"a yearly LTS cycle\", don\u0027t we also need slurp upgrade jobs for mariadb then, depending on when they exactly release?\n\nReading the blog post it\u0027s imho entirely unclear though, how long an LTS release will be supported, at least I couldn\u0027t find that information there.","commit_id":"ab86c55b19b4e62e7b78cc4af7e4f2f8c4f18395"},{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"aaed1a46c5c17606684a00d6c19ccdc578129a5b","unresolved":false,"context_lines":[{"line_number":12,"context_line":"changed command names (mysql* commands have been removed and only"},{"line_number":13,"context_line":"available in -compat package which we are not installing)"},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"[1]: https://mariadb.org/11-4-lts/"},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"Depends-On: https://review.opendev.org/c/openstack/kolla-ansible/+/927096"},{"line_number":18,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":14,"id":"7b2aa589_50141398","line":15,"range":{"start_line":15,"start_character":0,"end_line":15,"end_character":2},"in_reply_to":"22554dfd_724f15d8","updated":"2025-10-23 08:15:33.000000000","message":"Acknowledged","commit_id":"ab86c55b19b4e62e7b78cc4af7e4f2f8c4f18395"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"fb873cb05b3e2e3ba3cb885fb2221f31ed897eee","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"e10c93e9_5b4eff12","updated":"2024-08-23 20:17:32.000000000","message":"recheck (depends on changed)","commit_id":"e57ee2d60b5fc5916afd4dd11a8f4f0eaa9029e5"},{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"369832752e9f8a0fdfb66a155aee3429e941d3d8","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":11,"id":"928f2598_8fbc9f60","updated":"2024-08-26 06:49:38.000000000","message":"recheck (depends on changed)","commit_id":"eabd54150be43dc089a553217ed2dacd3bc198bc"},{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"9be7a04adcf764061e4df38a9f517a48628ec62a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":12,"id":"be767514_c7203df5","updated":"2024-08-26 14:05:09.000000000","message":"recheck (depends on changed)","commit_id":"6e6aaa179ab2c4b263bf8888954bf3c39474dc67"},{"author":{"_account_id":32553,"name":"Sven Kieske","email":"sven_oss@posteo.de","username":"skieske"},"change_message_id":"2949bbbef4bca3a53bc84089adc7a4cf6ed9e4da","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":14,"id":"0347dfce_4e86b423","updated":"2024-09-02 08:48:36.000000000","message":"code itself looks good to me. apart from the missing licence header.\nI also found a somewhat less outdated clustercheck script, but I didn\u0027t run it myself, it\u0027s in python instead of bash which might, or might not be considered an advantage:\nhttps://github.com/matthanley/haproxy-galera\n\nmy favorite solution would still be to get proxysql running by default though. ;)","commit_id":"ab86c55b19b4e62e7b78cc4af7e4f2f8c4f18395"},{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"23fca452a044631f4780708d5a32a3dc83f0f1ed","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":14,"id":"90fdd18b_21a748d8","updated":"2024-08-27 18:04:09.000000000","message":"recheck (depends on changed)","commit_id":"ab86c55b19b4e62e7b78cc4af7e4f2f8c4f18395"},{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"bf586bba5f444adb534b00cab5594252d916bebb","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":14,"id":"fb8f0f9c_ad342f5c","updated":"2024-08-27 14:47:38.000000000","message":"recheck (depends on changed)","commit_id":"ab86c55b19b4e62e7b78cc4af7e4f2f8c4f18395"},{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"03078bb249e83d080d87ce9ef163b82910b4b788","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":14,"id":"06cb5dd5_47886f94","updated":"2024-08-26 19:42:21.000000000","message":"recheck depends on changed","commit_id":"ab86c55b19b4e62e7b78cc4af7e4f2f8c4f18395"},{"author":{"_account_id":36765,"name":"Seunghun Lee","display_name":"Seunghun Lee","email":"seunghun@stackhpc.com","username":"seunghun1ee","status":"Cloud Engineer at StackHPC"},"change_message_id":"e347bdba787201e9c646e0f1f9a6df95180fd139","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":14,"id":"fdf4037b_3448bc7c","in_reply_to":"0347dfce_4e86b423","updated":"2025-10-01 09:12:34.000000000","message":"There\u0027s a patch I made to make ProxySQL default DB LB and drop clustercheck+haproxy\nhttps://review.opendev.org/c/openstack/kolla-ansible/+/962479","commit_id":"ab86c55b19b4e62e7b78cc4af7e4f2f8c4f18395"},{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"4457eb390a864e7e78fc776c0d16267d8d64b08b","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":15,"id":"dd75e0aa_75eb2167","updated":"2024-09-09 14:50:17.000000000","message":"multimode upgrades do not work due to a bug in mariadb WSREP https://jira.mariadb.org/browse/MDEV-18931 - let\u0027s wait for a fix","commit_id":"d71465adf7a104cd1aca9cf25c846874fd529286"},{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"04e2ba310ce8006ab9a73d3607ce63be5b97e509","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":15,"id":"6c8a0be8_d59f330c","updated":"2024-09-09 07:13:29.000000000","message":"recheck (depends on changed)","commit_id":"d71465adf7a104cd1aca9cf25c846874fd529286"},{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"fbdc94a1baae0bb7bafeb1d5dac59e5caa19e341","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":15,"id":"7d5161a4_976cd5a4","updated":"2024-09-09 08:15:22.000000000","message":"recheck (depends on changed)","commit_id":"d71465adf7a104cd1aca9cf25c846874fd529286"},{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"6107d27468a8f799050ee467dd1bb8691e623233","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":15,"id":"843816db_828e0b24","updated":"2024-09-09 09:57:53.000000000","message":"recheck (depends on changed)","commit_id":"d71465adf7a104cd1aca9cf25c846874fd529286"},{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"43e24d18efba3506dde69aa84c8b433279884953","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":15,"id":"93af3887_56013914","updated":"2024-09-08 18:54:29.000000000","message":"recheck (depends on changed)","commit_id":"d71465adf7a104cd1aca9cf25c846874fd529286"},{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"a857be2f57db81505fb40bdcc049c4239deec55f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":15,"id":"a4be7fdb_96eb5e02","updated":"2024-09-09 05:33:50.000000000","message":"recheck (depends on changed)","commit_id":"d71465adf7a104cd1aca9cf25c846874fd529286"},{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"ef17af9cf490126c14230de746c06cfc8c7f80c8","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":15,"id":"46567f9d_1a1ad2c7","in_reply_to":"a16eafc8_b0469762","updated":"2025-09-19 12:49:18.000000000","message":"I assume they had to fix this at some point, because the patch doesn\u0027t fail now.","commit_id":"d71465adf7a104cd1aca9cf25c846874fd529286"},{"author":{"_account_id":36765,"name":"Seunghun Lee","display_name":"Seunghun Lee","email":"seunghun@stackhpc.com","username":"seunghun1ee","status":"Cloud Engineer at StackHPC"},"change_message_id":"818932d13b0d657e2b79f96b9d877f1fb478a8cf","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":15,"id":"a16eafc8_b0469762","in_reply_to":"dd75e0aa_75eb2167","updated":"2025-09-19 12:46:38.000000000","message":"Tested restoring 10.11 backup to 11.4 MariaDB cluster without renaming the prefixes.\nThe restore was successful and the cloud functions well.\nWe don\u0027t need to worry about this","commit_id":"d71465adf7a104cd1aca9cf25c846874fd529286"},{"author":{"_account_id":17669,"name":"Doug Szumski","email":"doug@stackhpc.com","username":"DougSzumski"},"change_message_id":"dc4c453f9086df09079c5ce3fe7de8c33a3fcdd7","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":20,"id":"adc9a433_a0c114dd","updated":"2025-09-24 11:09:20.000000000","message":"It looks good generally, just some minor comments.","commit_id":"8e2cf9b988cb1a71dbf1f4e17339e33740f6fe72"},{"author":{"_account_id":36765,"name":"Seunghun Lee","display_name":"Seunghun Lee","email":"seunghun@stackhpc.com","username":"seunghun1ee","status":"Cloud Engineer at StackHPC"},"change_message_id":"3d2283465e1e5658c38a718e770dcbe6daa5f0d3","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":20,"id":"71eabb93_bda0fd1f","updated":"2025-09-09 18:09:54.000000000","message":"recheck","commit_id":"8e2cf9b988cb1a71dbf1f4e17339e33740f6fe72"},{"author":{"_account_id":36765,"name":"Seunghun Lee","display_name":"Seunghun Lee","email":"seunghun@stackhpc.com","username":"seunghun1ee","status":"Cloud Engineer at StackHPC"},"change_message_id":"ed10a216dcf99b4381d9978e40fcc3ae85857eaa","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":20,"id":"052ec19d_680c0cdc","updated":"2025-09-19 09:43:53.000000000","message":"recheck dependency had an update","commit_id":"8e2cf9b988cb1a71dbf1f4e17339e33740f6fe72"},{"author":{"_account_id":36765,"name":"Seunghun Lee","display_name":"Seunghun Lee","email":"seunghun@stackhpc.com","username":"seunghun1ee","status":"Cloud Engineer at StackHPC"},"change_message_id":"3ba6cf3f7c2f58b9bd3a045533005e893b144c70","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":26,"id":"5f518f32_4176222c","updated":"2025-10-01 10:30:06.000000000","message":"Patchset 26 is identical to patchset 21","commit_id":"97b1d42e545494f84cdda7f145c4c0cad306c564"},{"author":{"_account_id":36765,"name":"Seunghun Lee","display_name":"Seunghun Lee","email":"seunghun@stackhpc.com","username":"seunghun1ee","status":"Cloud Engineer at StackHPC"},"change_message_id":"ff2de2bb5424996e748534a8ae38d4e8a87d5936","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":26,"id":"48d5c4a5_ef70a879","updated":"2025-10-01 11:15:18.000000000","message":"The patch for adding healthcheck.sh was split to https://review.opendev.org/c/openstack/kolla/+/962657\n\nThe patch for removing clustercheck.sh was split to https://review.opendev.org/c/openstack/kolla/+/962659","commit_id":"97b1d42e545494f84cdda7f145c4c0cad306c564"},{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"ec830da55afa6a06d7c07b3744eb3f75f2ab8cea","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":28,"id":"2ab9e300_d7c52074","updated":"2025-10-23 08:15:48.000000000","message":"recheck - let\u0027s see some fresh results","commit_id":"c24528917661a9dc6f0fb1379cc74c544c690a42"}],"docker/mariadb/mariadb-base/Dockerfile.j2":[{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"c49cbc19b0c7298c7d72c7a6d3cf1928f5d9f4c5","unresolved":true,"context_lines":[{"line_number":24,"context_line":""},{"line_number":25,"context_line":"{{ macros.install_packages(mariadb_base_packages | customizable(\"packages\")) }}"},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"{% block mariadb_healthcheck_version %}"},{"line_number":28,"context_line":"ARG mariadb_healthcheck_version\u003d11.4"},{"line_number":29,"context_line":"ARG mariadb_healthcheck_url\u003dhttps://raw.githubusercontent.com/MariaDB/mariadb-docker/refs/heads/master/${mariadb_healthcheck_version}/healthcheck.sh"},{"line_number":30,"context_line":"{% endblock %}"}],"source_content_type":"text/x-jinja2","patch_set":25,"id":"d2f516fa_a7525302","line":27,"updated":"2025-10-01 09:58:47.000000000","message":"Split it out to separate patch please","commit_id":"816a21d7193c326ab75e73b3185517c3ec9642dd"},{"author":{"_account_id":36765,"name":"Seunghun Lee","display_name":"Seunghun Lee","email":"seunghun@stackhpc.com","username":"seunghun1ee","status":"Cloud Engineer at StackHPC"},"change_message_id":"8c2289d5c8f1f13830802dd6d053983f06d7c58c","unresolved":false,"context_lines":[{"line_number":24,"context_line":""},{"line_number":25,"context_line":"{{ macros.install_packages(mariadb_base_packages | customizable(\"packages\")) }}"},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"{% block mariadb_healthcheck_version %}"},{"line_number":28,"context_line":"ARG mariadb_healthcheck_version\u003d11.4"},{"line_number":29,"context_line":"ARG mariadb_healthcheck_url\u003dhttps://raw.githubusercontent.com/MariaDB/mariadb-docker/refs/heads/master/${mariadb_healthcheck_version}/healthcheck.sh"},{"line_number":30,"context_line":"{% endblock %}"}],"source_content_type":"text/x-jinja2","patch_set":25,"id":"d0385823_fa317831","line":27,"in_reply_to":"d2f516fa_a7525302","updated":"2025-10-01 10:15:30.000000000","message":"Acknowledged","commit_id":"816a21d7193c326ab75e73b3185517c3ec9642dd"}],"docker/mariadb/mariadb-base/clustercheck.sh":[{"author":{"_account_id":32553,"name":"Sven Kieske","email":"sven_oss@posteo.de","username":"skieske"},"change_message_id":"2949bbbef4bca3a53bc84089adc7a4cf6ed9e4da","unresolved":true,"context_lines":[{"line_number":1,"context_line":"#!/bin/bash"},{"line_number":2,"context_line":"#"},{"line_number":3,"context_line":"# Script to make a proxy (ie HAProxy) capable of monitoring Galera cluster nodes properly"},{"line_number":4,"context_line":"#"},{"line_number":5,"context_line":"# Author: Olaf van Zandwijk \u003colaf.vanzandwijk@nedap.com\u003e"},{"line_number":6,"context_line":"# Author: Raghavendra Prabhu \u003craghavendra.prabhu@percona.com\u003e"},{"line_number":7,"context_line":"# Author: Ryan O\u0027Hara \u003crohara@redhat.com\u003e"},{"line_number":8,"context_line":"#"},{"line_number":9,"context_line":"# Documentation and download: https://github.com/olafz/percona-clustercheck"},{"line_number":10,"context_line":"#"},{"line_number":11,"context_line":"# Based on the original script from Unai Rodriguez"},{"line_number":12,"context_line":"#"}],"source_content_type":"text/x-sh","patch_set":14,"id":"93271885_02939496","line":9,"range":{"start_line":1,"start_character":0,"end_line":9,"end_character":75},"updated":"2024-09-02 08:48:36.000000000","message":"I think we must add the LICENCE header to this file, no?\n\nIt\u0027s BSD-3 clause, see:\n\nhttps://github.com/olafz/percona-clustercheck/blob/master/LICENSE","commit_id":"ab86c55b19b4e62e7b78cc4af7e4f2f8c4f18395"},{"author":{"_account_id":36765,"name":"Seunghun Lee","display_name":"Seunghun Lee","email":"seunghun@stackhpc.com","username":"seunghun1ee","status":"Cloud Engineer at StackHPC"},"change_message_id":"b54b5a75c703b1298778ac240b73039e684d3bf4","unresolved":false,"context_lines":[{"line_number":1,"context_line":"#!/bin/bash"},{"line_number":2,"context_line":"#"},{"line_number":3,"context_line":"# Script to make a proxy (ie HAProxy) capable of monitoring Galera cluster nodes properly"},{"line_number":4,"context_line":"#"},{"line_number":5,"context_line":"# Author: Olaf van Zandwijk \u003colaf.vanzandwijk@nedap.com\u003e"},{"line_number":6,"context_line":"# Author: Raghavendra Prabhu \u003craghavendra.prabhu@percona.com\u003e"},{"line_number":7,"context_line":"# Author: Ryan O\u0027Hara \u003crohara@redhat.com\u003e"},{"line_number":8,"context_line":"#"},{"line_number":9,"context_line":"# Documentation and download: https://github.com/olafz/percona-clustercheck"},{"line_number":10,"context_line":"#"},{"line_number":11,"context_line":"# Based on the original script from Unai Rodriguez"},{"line_number":12,"context_line":"#"}],"source_content_type":"text/x-sh","patch_set":14,"id":"61ced012_73b65ab6","line":9,"range":{"start_line":1,"start_character":0,"end_line":9,"end_character":75},"in_reply_to":"93271885_02939496","updated":"2025-09-09 15:03:56.000000000","message":"Done","commit_id":"ab86c55b19b4e62e7b78cc4af7e4f2f8c4f18395"},{"author":{"_account_id":17669,"name":"Doug Szumski","email":"doug@stackhpc.com","username":"DougSzumski"},"change_message_id":"dc4c453f9086df09079c5ce3fe7de8c33a3fcdd7","unresolved":true,"context_lines":[{"line_number":6,"context_line":"# Author: Raghavendra Prabhu \u003craghavendra.prabhu@percona.com\u003e"},{"line_number":7,"context_line":"# Author: Ryan O\u0027Hara \u003crohara@redhat.com\u003e"},{"line_number":8,"context_line":"#"},{"line_number":9,"context_line":"# Documentation and download: https://github.com/olafz/percona-clustercheck"},{"line_number":10,"context_line":"#"},{"line_number":11,"context_line":"# Based on the original script from Unai Rodriguez"},{"line_number":12,"context_line":"#"}],"source_content_type":"text/x-sh","patch_set":20,"id":"704b6bd2_5d126d76","line":9,"updated":"2025-09-24 11:09:20.000000000","message":"This is no longer correct (it is a much older version). It\u0027s no longer obvious where the script has been copied from.\n\nTo help with future maintenance, please can we retain the original approach of fetching the check script, and then apply a patch to it to update the commands?","commit_id":"8e2cf9b988cb1a71dbf1f4e17339e33740f6fe72"},{"author":{"_account_id":17669,"name":"Doug Szumski","email":"doug@stackhpc.com","username":"DougSzumski"},"change_message_id":"dc4c453f9086df09079c5ce3fe7de8c33a3fcdd7","unresolved":true,"context_lines":[{"line_number":6,"context_line":"# Author: Raghavendra Prabhu \u003craghavendra.prabhu@percona.com\u003e"},{"line_number":7,"context_line":"# Author: Ryan O\u0027Hara \u003crohara@redhat.com\u003e"},{"line_number":8,"context_line":"#"},{"line_number":9,"context_line":"# Documentation and download: https://github.com/olafz/percona-clustercheck"},{"line_number":10,"context_line":"#"},{"line_number":11,"context_line":"# Based on the original script from Unai Rodriguez"},{"line_number":12,"context_line":"#"}],"source_content_type":"text/x-sh","patch_set":20,"id":"8b008b1c_95f37fb4","line":9,"updated":"2025-09-24 11:09:20.000000000","message":"This version of the script is not from this link.\n\nIt would be easier to maintain if we could continue fetching the script, rather than copying it here, and instead patch it with the updated command.","commit_id":"8e2cf9b988cb1a71dbf1f4e17339e33740f6fe72"},{"author":{"_account_id":36765,"name":"Seunghun Lee","display_name":"Seunghun Lee","email":"seunghun@stackhpc.com","username":"seunghun1ee","status":"Cloud Engineer at StackHPC"},"change_message_id":"9659b2c0553427523c282099207ca19ecc8dec16","unresolved":true,"context_lines":[{"line_number":6,"context_line":"# Author: Raghavendra Prabhu \u003craghavendra.prabhu@percona.com\u003e"},{"line_number":7,"context_line":"# Author: Ryan O\u0027Hara \u003crohara@redhat.com\u003e"},{"line_number":8,"context_line":"#"},{"line_number":9,"context_line":"# Documentation and download: https://github.com/olafz/percona-clustercheck"},{"line_number":10,"context_line":"#"},{"line_number":11,"context_line":"# Based on the original script from Unai Rodriguez"},{"line_number":12,"context_line":"#"}],"source_content_type":"text/x-sh","patch_set":20,"id":"ef9317d6_db58f622","line":9,"in_reply_to":"3005d505_649579b7","updated":"2025-09-25 09:19:33.000000000","message":"What should we do then? Force users to always use proxysql on K-A?","commit_id":"8e2cf9b988cb1a71dbf1f4e17339e33740f6fe72"},{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"a2da231ea0328da6346639b272d21dcd294d6f40","unresolved":true,"context_lines":[{"line_number":6,"context_line":"# Author: Raghavendra Prabhu \u003craghavendra.prabhu@percona.com\u003e"},{"line_number":7,"context_line":"# Author: Ryan O\u0027Hara \u003crohara@redhat.com\u003e"},{"line_number":8,"context_line":"#"},{"line_number":9,"context_line":"# Documentation and download: https://github.com/olafz/percona-clustercheck"},{"line_number":10,"context_line":"#"},{"line_number":11,"context_line":"# Based on the original script from Unai Rodriguez"},{"line_number":12,"context_line":"#"}],"source_content_type":"text/x-sh","patch_set":20,"id":"83aed26a_a8b2589c","line":9,"in_reply_to":"33fd4aca_696730eb","updated":"2025-09-24 14:14:07.000000000","message":"I don\u0027t foresee future maintenance - we should get rid of clustercheck at all - proxysql has been working nicely as a default.\nActually we could get rid of it now stating it\u0027s not supported since the upgrade to 11.4.","commit_id":"8e2cf9b988cb1a71dbf1f4e17339e33740f6fe72"},{"author":{"_account_id":36765,"name":"Seunghun Lee","display_name":"Seunghun Lee","email":"seunghun@stackhpc.com","username":"seunghun1ee","status":"Cloud Engineer at StackHPC"},"change_message_id":"e347bdba787201e9c646e0f1f9a6df95180fd139","unresolved":false,"context_lines":[{"line_number":6,"context_line":"# Author: Raghavendra Prabhu \u003craghavendra.prabhu@percona.com\u003e"},{"line_number":7,"context_line":"# Author: Ryan O\u0027Hara \u003crohara@redhat.com\u003e"},{"line_number":8,"context_line":"#"},{"line_number":9,"context_line":"# Documentation and download: https://github.com/olafz/percona-clustercheck"},{"line_number":10,"context_line":"#"},{"line_number":11,"context_line":"# Based on the original script from Unai Rodriguez"},{"line_number":12,"context_line":"#"}],"source_content_type":"text/x-sh","patch_set":20,"id":"7715c500_45bf705b","line":9,"in_reply_to":"5cd4659d_0160acfe","updated":"2025-10-01 09:12:34.000000000","message":"clustercheck got replaced with official mariadb docker image\u0027s healthcheck.sh\nSee https://mariadb.com/docs/server/server-management/install-and-upgrade-mariadb/installing-mariadb/binary-packages/automated-mariadb-deployment-and-administration/docker-and-mariadb/using-healthcheck-sh","commit_id":"8e2cf9b988cb1a71dbf1f4e17339e33740f6fe72"},{"author":{"_account_id":36765,"name":"Seunghun Lee","display_name":"Seunghun Lee","email":"seunghun@stackhpc.com","username":"seunghun1ee","status":"Cloud Engineer at StackHPC"},"change_message_id":"fd5b3902d28cf652d551137903aa7083c2de1db8","unresolved":true,"context_lines":[{"line_number":6,"context_line":"# Author: Raghavendra Prabhu \u003craghavendra.prabhu@percona.com\u003e"},{"line_number":7,"context_line":"# Author: Ryan O\u0027Hara \u003crohara@redhat.com\u003e"},{"line_number":8,"context_line":"#"},{"line_number":9,"context_line":"# Documentation and download: https://github.com/olafz/percona-clustercheck"},{"line_number":10,"context_line":"#"},{"line_number":11,"context_line":"# Based on the original script from Unai Rodriguez"},{"line_number":12,"context_line":"#"}],"source_content_type":"text/x-sh","patch_set":20,"id":"33fd4aca_696730eb","line":9,"in_reply_to":"704b6bd2_5d126d76","updated":"2025-09-24 14:05:30.000000000","message":"I agree that the original approach is better for the maintenance but the main reason of bringing script to local was to change ``mysql`` commands to ``mariadb`` commands.\n``mysql`` commands are no longer supported","commit_id":"8e2cf9b988cb1a71dbf1f4e17339e33740f6fe72"},{"author":{"_account_id":17669,"name":"Doug Szumski","email":"doug@stackhpc.com","username":"DougSzumski"},"change_message_id":"270d4b8cee4e471bc6233f3d7429a5ef61daba25","unresolved":true,"context_lines":[{"line_number":6,"context_line":"# Author: Raghavendra Prabhu \u003craghavendra.prabhu@percona.com\u003e"},{"line_number":7,"context_line":"# Author: Ryan O\u0027Hara \u003crohara@redhat.com\u003e"},{"line_number":8,"context_line":"#"},{"line_number":9,"context_line":"# Documentation and download: https://github.com/olafz/percona-clustercheck"},{"line_number":10,"context_line":"#"},{"line_number":11,"context_line":"# Based on the original script from Unai Rodriguez"},{"line_number":12,"context_line":"#"}],"source_content_type":"text/x-sh","patch_set":20,"id":"f482e5fc_c9ddcb32","line":9,"in_reply_to":"83aed26a_a8b2589c","updated":"2025-09-24 14:33:19.000000000","message":"@Seunghun Lee what I meant was 1) Fetch raw script as we used to 2) Apply a local patch to update the commands in the script, instead of copying the whole thing. This would keep the original file / source intact, and make it clear which changes we have applied to the script.\n\nNot to worry now as it sounds like we can just remove the script in this change.","commit_id":"8e2cf9b988cb1a71dbf1f4e17339e33740f6fe72"},{"author":{"_account_id":17669,"name":"Doug Szumski","email":"doug@stackhpc.com","username":"DougSzumski"},"change_message_id":"455736551291b1f9a5626876c675858476978b24","unresolved":false,"context_lines":[{"line_number":6,"context_line":"# Author: Raghavendra Prabhu \u003craghavendra.prabhu@percona.com\u003e"},{"line_number":7,"context_line":"# Author: Ryan O\u0027Hara \u003crohara@redhat.com\u003e"},{"line_number":8,"context_line":"#"},{"line_number":9,"context_line":"# Documentation and download: https://github.com/olafz/percona-clustercheck"},{"line_number":10,"context_line":"#"},{"line_number":11,"context_line":"# Based on the original script from Unai Rodriguez"},{"line_number":12,"context_line":"#"}],"source_content_type":"text/x-sh","patch_set":20,"id":"97f5aeeb_999389f6","line":9,"in_reply_to":"8b008b1c_95f37fb4","updated":"2025-09-24 11:11:23.000000000","message":"Sorry, duplicate comment.","commit_id":"8e2cf9b988cb1a71dbf1f4e17339e33740f6fe72"},{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"4bd607e77229327dd18d9ed69f0103557a7ba842","unresolved":true,"context_lines":[{"line_number":6,"context_line":"# Author: Raghavendra Prabhu \u003craghavendra.prabhu@percona.com\u003e"},{"line_number":7,"context_line":"# Author: Ryan O\u0027Hara \u003crohara@redhat.com\u003e"},{"line_number":8,"context_line":"#"},{"line_number":9,"context_line":"# Documentation and download: https://github.com/olafz/percona-clustercheck"},{"line_number":10,"context_line":"#"},{"line_number":11,"context_line":"# Based on the original script from Unai Rodriguez"},{"line_number":12,"context_line":"#"}],"source_content_type":"text/x-sh","patch_set":20,"id":"3005d505_649579b7","line":9,"in_reply_to":"8b32f7f7_bc7ac5e9","updated":"2025-09-24 14:46:18.000000000","message":"There is larger impact in kolla-ansible - we use clustercheck.sh for healthchecks (and we use it when proxysql is disabled).","commit_id":"8e2cf9b988cb1a71dbf1f4e17339e33740f6fe72"},{"author":{"_account_id":36765,"name":"Seunghun Lee","display_name":"Seunghun Lee","email":"seunghun@stackhpc.com","username":"seunghun1ee","status":"Cloud Engineer at StackHPC"},"change_message_id":"4201573acf30efb8457c8242c36b13ca61650115","unresolved":true,"context_lines":[{"line_number":6,"context_line":"# Author: Raghavendra Prabhu \u003craghavendra.prabhu@percona.com\u003e"},{"line_number":7,"context_line":"# Author: Ryan O\u0027Hara \u003crohara@redhat.com\u003e"},{"line_number":8,"context_line":"#"},{"line_number":9,"context_line":"# Documentation and download: https://github.com/olafz/percona-clustercheck"},{"line_number":10,"context_line":"#"},{"line_number":11,"context_line":"# Based on the original script from Unai Rodriguez"},{"line_number":12,"context_line":"#"}],"source_content_type":"text/x-sh","patch_set":20,"id":"5cd4659d_0160acfe","line":9,"in_reply_to":"ef9317d6_db58f622","updated":"2025-09-26 13:34:08.000000000","message":"The clustercheck.sh itself is simple. What about write a new cluster check script that uses proxysql (or monitor) DB user?","commit_id":"8e2cf9b988cb1a71dbf1f4e17339e33740f6fe72"},{"author":{"_account_id":36765,"name":"Seunghun Lee","display_name":"Seunghun Lee","email":"seunghun@stackhpc.com","username":"seunghun1ee","status":"Cloud Engineer at StackHPC"},"change_message_id":"68bfb8d5db8b046f351a96e64510ab00c064decb","unresolved":true,"context_lines":[{"line_number":6,"context_line":"# Author: Raghavendra Prabhu \u003craghavendra.prabhu@percona.com\u003e"},{"line_number":7,"context_line":"# Author: Ryan O\u0027Hara \u003crohara@redhat.com\u003e"},{"line_number":8,"context_line":"#"},{"line_number":9,"context_line":"# Documentation and download: https://github.com/olafz/percona-clustercheck"},{"line_number":10,"context_line":"#"},{"line_number":11,"context_line":"# Based on the original script from Unai Rodriguez"},{"line_number":12,"context_line":"#"}],"source_content_type":"text/x-sh","patch_set":20,"id":"8b32f7f7_bc7ac5e9","line":9,"in_reply_to":"f482e5fc_c9ddcb32","updated":"2025-09-24 14:40:33.000000000","message":"@doug@stackhpc.com Sorry I misread your first comment. @mnasiadka@gmail.com I\u0027ll remove the file and write this change on release note","commit_id":"8e2cf9b988cb1a71dbf1f4e17339e33740f6fe72"}],"docker/mariadb/mariadb-clustercheck/Dockerfile.j2":[{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"c49cbc19b0c7298c7d72c7a6d3cf1928f5d9f4c5","unresolved":true,"context_lines":[{"line_number":1,"context_line":"FROM {{ namespace }}/{{ image_prefix }}mariadb-base:{{ tag }}"},{"line_number":2,"context_line":"{% block labels %}"},{"line_number":3,"context_line":"LABEL maintainer\u003d\"{{ maintainer }}\" name\u003d\"{{ image_name }}\" build-date\u003d\"{{ build_date }}\""},{"line_number":4,"context_line":"{% endblock %}"}],"source_content_type":"text/x-jinja2","patch_set":25,"id":"4b27c883_18b5c9ad","side":"PARENT","line":1,"updated":"2025-10-01 09:58:47.000000000","message":"Split it out to separate patch please","commit_id":"dca8462bc491a93c6a6ae71e86d59a2dcb804ff5"},{"author":{"_account_id":36765,"name":"Seunghun Lee","display_name":"Seunghun Lee","email":"seunghun@stackhpc.com","username":"seunghun1ee","status":"Cloud Engineer at StackHPC"},"change_message_id":"8c2289d5c8f1f13830802dd6d053983f06d7c58c","unresolved":false,"context_lines":[{"line_number":1,"context_line":"FROM {{ namespace }}/{{ image_prefix }}mariadb-base:{{ tag }}"},{"line_number":2,"context_line":"{% block labels %}"},{"line_number":3,"context_line":"LABEL maintainer\u003d\"{{ maintainer }}\" name\u003d\"{{ image_name }}\" build-date\u003d\"{{ build_date }}\""},{"line_number":4,"context_line":"{% endblock %}"}],"source_content_type":"text/x-jinja2","patch_set":25,"id":"c303425c_19c01f2b","side":"PARENT","line":1,"in_reply_to":"4b27c883_18b5c9ad","updated":"2025-10-01 10:15:30.000000000","message":"Acknowledged","commit_id":"dca8462bc491a93c6a6ae71e86d59a2dcb804ff5"}],"docker/mariadb/mariadb-server/extend_start.sh":[{"author":{"_account_id":17669,"name":"Doug Szumski","email":"doug@stackhpc.com","username":"DougSzumski"},"change_message_id":"dc4c453f9086df09079c5ce3fe7de8c33a3fcdd7","unresolved":true,"context_lines":[{"line_number":45,"context_line":""},{"line_number":46,"context_line":"# This catches all cases of the KOLLA_UPGRADE variable being set, including empty"},{"line_number":47,"context_line":"if [[ \"${!KOLLA_UPGRADE[@]}\" ]]; then"},{"line_number":48,"context_line":"    # The mysql_upgrade command treats any directories under /var/lib/mysql as"},{"line_number":49,"context_line":"    # databases. Somehow we can end up with a .pki directory, which causes the"},{"line_number":50,"context_line":"    # command to fail with this error:"},{"line_number":51,"context_line":"    # Incorrect database name \u0027#mysql50#.pki\u0027 when selecting the database"}],"source_content_type":"text/x-sh","patch_set":20,"id":"75dddb0e_97e1cee2","line":48,"updated":"2025-09-24 11:09:20.000000000","message":"nit: `mariadb_upgrade` command","commit_id":"8e2cf9b988cb1a71dbf1f4e17339e33740f6fe72"},{"author":{"_account_id":36765,"name":"Seunghun Lee","display_name":"Seunghun Lee","email":"seunghun@stackhpc.com","username":"seunghun1ee","status":"Cloud Engineer at StackHPC"},"change_message_id":"80cb7f57dd641e09fb000b8089ca1d86052a76c3","unresolved":false,"context_lines":[{"line_number":45,"context_line":""},{"line_number":46,"context_line":"# This catches all cases of the KOLLA_UPGRADE variable being set, including empty"},{"line_number":47,"context_line":"if [[ \"${!KOLLA_UPGRADE[@]}\" ]]; then"},{"line_number":48,"context_line":"    # The mysql_upgrade command treats any directories under /var/lib/mysql as"},{"line_number":49,"context_line":"    # databases. Somehow we can end up with a .pki directory, which causes the"},{"line_number":50,"context_line":"    # command to fail with this error:"},{"line_number":51,"context_line":"    # Incorrect database name \u0027#mysql50#.pki\u0027 when selecting the database"}],"source_content_type":"text/x-sh","patch_set":20,"id":"51fa655b_1b3d26bd","line":48,"in_reply_to":"75dddb0e_97e1cee2","updated":"2025-09-24 13:45:02.000000000","message":"Acknowledged","commit_id":"8e2cf9b988cb1a71dbf1f4e17339e33740f6fe72"}],"releasenotes/notes/mariadb-11.4-b66b5baf9f5e6cd0.yaml":[{"author":{"_account_id":17669,"name":"Doug Szumski","email":"doug@stackhpc.com","username":"DougSzumski"},"change_message_id":"dc4c453f9086df09079c5ce3fe7de8c33a3fcdd7","unresolved":true,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"features:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    ``MariaDB`` version has been updated to ``11.4``."}],"source_content_type":"text/x-yaml","patch_set":20,"id":"ff24ef5b_d5ea5305","line":4,"updated":"2025-09-24 11:09:20.000000000","message":"nit: It would be informative to mention that this is an LTS version, supported until `29 May 2029`.\n\nhttps://mariadb.org/about/#maintenance-policy","commit_id":"8e2cf9b988cb1a71dbf1f4e17339e33740f6fe72"},{"author":{"_account_id":17669,"name":"Doug Szumski","email":"doug@stackhpc.com","username":"DougSzumski"},"change_message_id":"dc4c453f9086df09079c5ce3fe7de8c33a3fcdd7","unresolved":true,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"features:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    ``MariaDB`` version has been updated to ``11.4``."}],"source_content_type":"text/x-yaml","patch_set":20,"id":"2c97dd26_d4f58c80","line":4,"updated":"2025-09-24 11:09:20.000000000","message":"nit: You co","commit_id":"8e2cf9b988cb1a71dbf1f4e17339e33740f6fe72"},{"author":{"_account_id":17669,"name":"Doug Szumski","email":"doug@stackhpc.com","username":"DougSzumski"},"change_message_id":"2f8769ec425bbd5198ad957006ed67be87b3add5","unresolved":false,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"features:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    ``MariaDB`` version has been updated to ``11.4``."}],"source_content_type":"text/x-yaml","patch_set":20,"id":"ea2ab736_f8390eae","line":4,"in_reply_to":"2c97dd26_d4f58c80","updated":"2025-09-24 11:12:02.000000000","message":"ignore","commit_id":"8e2cf9b988cb1a71dbf1f4e17339e33740f6fe72"},{"author":{"_account_id":36765,"name":"Seunghun Lee","display_name":"Seunghun Lee","email":"seunghun@stackhpc.com","username":"seunghun1ee","status":"Cloud Engineer at StackHPC"},"change_message_id":"80cb7f57dd641e09fb000b8089ca1d86052a76c3","unresolved":false,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"features:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    ``MariaDB`` version has been updated to ``11.4``."}],"source_content_type":"text/x-yaml","patch_set":20,"id":"c2600259_e24e7aab","line":4,"in_reply_to":"ff24ef5b_d5ea5305","updated":"2025-09-24 13:45:02.000000000","message":"Acknowledged","commit_id":"8e2cf9b988cb1a71dbf1f4e17339e33740f6fe72"}]}