)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":37598,"name":"Ivan Anfimov","display_name":"Ivan Anfimov","email":"lazekteam@gmail.com","username":"anfimovir"},"change_message_id":"08285aa1ab484f878c19511ef75f40ff1bd350a4","unresolved":true,"context_lines":[{"line_number":4,"context_line":"Commit:     Erkin Mussurmankulov \u003cmangust404@gmail.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2026-01-29 19:11:17 +0500"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Defaults config path in MaraiDB 10.4+ for official images"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"For official mariadb docker images from dockerhub, MariaDB tests fails"},{"line_number":10,"context_line":"for versions higher than 10.4. The reason is that MariaDB doesn\u0027t use"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":6,"id":"0bbbc6eb_4c83ff76","line":7,"updated":"2026-01-30 11:26:47.000000000","message":"```suggestion\nDefaults config path in MariaDB 10.4+ for official images\n```","commit_id":"b3afd18914fa8b5177dc1187295b1a1dfa866302"},{"author":{"_account_id":36080,"name":"Erkin Mussurmankulov","display_name":"Eric","email":"mangust404@gmail.com","username":"mongoose404","status":"PS Cloud services employee"},"change_message_id":"ec4b4e0a4e757222c1b9d28e5976daba7f726103","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Erkin Mussurmankulov \u003cmangust404@gmail.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2026-01-29 19:11:17 +0500"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Defaults config path in MaraiDB 10.4+ for official images"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"For official mariadb docker images from dockerhub, MariaDB tests fails"},{"line_number":10,"context_line":"for versions higher than 10.4. The reason is that MariaDB doesn\u0027t use"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":6,"id":"d0e22995_1b85f3a3","line":7,"in_reply_to":"0bbbc6eb_4c83ff76","updated":"2026-02-02 13:38:18.000000000","message":"Fix applied.","commit_id":"b3afd18914fa8b5177dc1187295b1a1dfa866302"},{"author":{"_account_id":37598,"name":"Ivan Anfimov","display_name":"Ivan Anfimov","email":"lazekteam@gmail.com","username":"anfimovir"},"change_message_id":"1dc21ee04b713db483349101c3e594ef4636704d","unresolved":true,"context_lines":[{"line_number":4,"context_line":"Commit:     Erkin Mussurmankulov \u003cmangust404@gmail.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2026-02-03 20:38:53 +0500"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Defaults config path in MaraiDB 10.4+ for official images"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"For official mariadb docker images from dockerhub, MariaDB tests fails"},{"line_number":10,"context_line":"for versions higher than 10.4. The reason is that MariaDB doesn\u0027t use"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":8,"id":"0c34a7d5_279ddd5f","line":7,"updated":"2026-02-03 15:45:13.000000000","message":"```suggestion\nDefaults config path in MariaDB 10.4+ for official images\n```","commit_id":"b5042de62865559f5f610663096cfbb2382e2bd3"},{"author":{"_account_id":36080,"name":"Erkin Mussurmankulov","display_name":"Eric","email":"mangust404@gmail.com","username":"mongoose404","status":"PS Cloud services employee"},"change_message_id":"234ff1a11bc6f32360d8202bc2ee54076fedbadc","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Erkin Mussurmankulov \u003cmangust404@gmail.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2026-02-03 20:38:53 +0500"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Defaults config path in MaraiDB 10.4+ for official images"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"For official mariadb docker images from dockerhub, MariaDB tests fails"},{"line_number":10,"context_line":"for versions higher than 10.4. The reason is that MariaDB doesn\u0027t use"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":8,"id":"ef387bb1_602dd707","line":7,"in_reply_to":"0c34a7d5_279ddd5f","updated":"2026-02-03 16:13:41.000000000","message":"Acknowledged\nI forgot to change it locally","commit_id":"b5042de62865559f5f610663096cfbb2382e2bd3"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":36080,"name":"Erkin Mussurmankulov","display_name":"Eric","email":"mangust404@gmail.com","username":"mongoose404","status":"PS Cloud services employee"},"change_message_id":"2948f00f16dbaf0c29793821358a23f02adad6e2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"dc27aa35_07671a23","updated":"2026-01-27 18:30:17.000000000","message":"Hello, Wu. Can you please review this patch?\n\nI did manual tests against images with TestInstanceBasicMariaDB:\n\nRun against current master:\n10.4.34-focal ok\n10.11.9-ubi9 fail\n11.4.5-ubi9 fail\n11.8.5-ubi9 fail\n\nRun with this patch applied:\n10.4.34-focal ok\n10.11.9-ubi9 ok\n11.4.5-ubi9 ok\n11.8.5-ubi9 ok\n\nI suppose the changes with defaults file were somewhere in 10.4-10.11 (probably in 10.7, which is no longer supported).\n\nIt seems that images from zuul jobs (`quay.io/openstack.trove/mariadb`) differs\nfrom official images, and I can\u0027t find any info that user should use these images instead of official mariadb images.\n\nProbably we should run tempest tests against official images in zuul, how do you\nthink?\n\nOr, in the other hand, we should make corresponding changes everywhere (docs,\ncfg.py) that user should use quay images instead?","commit_id":"ee05155a4d46487b29d99b2385d0023b7e4f9950"},{"author":{"_account_id":26285,"name":"wu.chunyang","email":"wchy1001@gmail.com","username":"wu.chunyang"},"change_message_id":"311cf04fe83e48c70ec89fb7d452b7876e0df62f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"c8f586bf_2933f6a4","updated":"2026-01-28 08:15:24.000000000","message":"Hi, The more logs are [here](https://zuul.opendev.org/t/openstack/build/59f34b4034b94fbfb287de7d72463675/log/controller/logs/guest-agent-logs/c35079b6-4ab1-4fa8-ac47-012c4d78a5f0/log/trove/trove-guestagent_log.txt)","commit_id":"ee05155a4d46487b29d99b2385d0023b7e4f9950"},{"author":{"_account_id":36080,"name":"Erkin Mussurmankulov","display_name":"Eric","email":"mangust404@gmail.com","username":"mongoose404","status":"PS Cloud services employee"},"change_message_id":"fde45e3dfdf098f1fdcd27019ea03b379bcb34a0","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"394a309a_d452696f","updated":"2026-01-28 06:10:48.000000000","message":"recheck","commit_id":"ee05155a4d46487b29d99b2385d0023b7e4f9950"},{"author":{"_account_id":36080,"name":"Erkin Mussurmankulov","display_name":"Eric","email":"mangust404@gmail.com","username":"mongoose404","status":"PS Cloud services employee"},"change_message_id":"630f780dde725846cb4268837e4c55514e73feb2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"70e1cdc1_00859154","in_reply_to":"c8f586bf_2933f6a4","updated":"2026-01-28 09:57:56.000000000","message":"Hi! Ok, thanks for the hint.\nI will figure out what\u0027s wrong and will provide a fix ASAP. It should work with any docker images.\n\nThis issue is blocking next features, because MaraiDB tests are failing in our OpenStack setup 😭","commit_id":"ee05155a4d46487b29d99b2385d0023b7e4f9950"},{"author":{"_account_id":26285,"name":"wu.chunyang","email":"wchy1001@gmail.com","username":"wu.chunyang"},"change_message_id":"4184a566c8b4d9686ade00c489682317040a2980","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"275dff7e_5ba408fb","in_reply_to":"dc27aa35_07671a23","updated":"2026-01-28 01:35:13.000000000","message":"Hello,\nIt looks like there is a slight difference in the UBI9 distribution. Since this distribution hasn\u0027t been tested in the CI gate, all the failed test cases are related to it.\nMariaDB has been tested in the CI, and its status is green now. Therefore, we may need to figure out what the differences are in the UBI9 distribution.\nThe image at `quay.io/openstack.trove/mariadb` is slightly different from the official image. I changed the default timeout in the script from 30s to 120s, making it more suitable for the CI testing environment. More details are available [here](https://github.com/openstack/trove/blob/master/playbooks/images/mariadb/11.8/Dockerfile)","commit_id":"ee05155a4d46487b29d99b2385d0023b7e4f9950"},{"author":{"_account_id":37598,"name":"Ivan Anfimov","display_name":"Ivan Anfimov","email":"lazekteam@gmail.com","username":"anfimovir"},"change_message_id":"95796c396558fa5d8b0554bc70c7108cb8cee46b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"b5fe6326_3621596f","updated":"2026-02-02 13:29:38.000000000","message":"recheck","commit_id":"b3afd18914fa8b5177dc1187295b1a1dfa866302"},{"author":{"_account_id":36080,"name":"Erkin Mussurmankulov","display_name":"Eric","email":"mangust404@gmail.com","username":"mongoose404","status":"PS Cloud services employee"},"change_message_id":"ec4b4e0a4e757222c1b9d28e5976daba7f726103","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"a5b6488f_42f6e2ba","in_reply_to":"b5fe6326_3621596f","updated":"2026-02-02 13:38:18.000000000","message":"Hi! No need to recheck yet, work in progress with this patch.\nI took a big side-quest with improving instance list method and I\u0027m still there for today 😊","commit_id":"b3afd18914fa8b5177dc1187295b1a1dfa866302"},{"author":{"_account_id":36080,"name":"Erkin Mussurmankulov","display_name":"Eric","email":"mangust404@gmail.com","username":"mongoose404","status":"PS Cloud services employee"},"change_message_id":"e8c77f3512d503230cc5cc7cf1e839da6e42addc","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":15,"id":"30219d93_f2158841","updated":"2026-02-04 20:14:17.000000000","message":"recheck","commit_id":"a98c90064aa34d4590761a6c88757806111e3122"},{"author":{"_account_id":36080,"name":"Erkin Mussurmankulov","display_name":"Eric","email":"mangust404@gmail.com","username":"mongoose404","status":"PS Cloud services employee"},"change_message_id":"03edfd03f1bd70a1ddc6881c454709c20a542745","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":15,"id":"d24de39c_2ffde817","updated":"2026-02-05 06:12:56.000000000","message":"recheck","commit_id":"a98c90064aa34d4590761a6c88757806111e3122"},{"author":{"_account_id":36080,"name":"Erkin Mussurmankulov","display_name":"Eric","email":"mangust404@gmail.com","username":"mongoose404","status":"PS Cloud services employee"},"change_message_id":"5f96d7d6925ea899f1adb55f121fc479b11efce9","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":15,"id":"e49d2a88_3c482d24","updated":"2026-02-06 16:07:07.000000000","message":"recheck","commit_id":"a98c90064aa34d4590761a6c88757806111e3122"},{"author":{"_account_id":36080,"name":"Erkin Mussurmankulov","display_name":"Eric","email":"mangust404@gmail.com","username":"mongoose404","status":"PS Cloud services employee"},"change_message_id":"ea74f42c394da6945456c5f322857e04ba4f7272","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":17,"id":"35774672_adccc773","updated":"2026-02-07 17:54:03.000000000","message":"Hello, guys! Everything with MariaDB seems fine now. It works with both official mariadb images from dockerhub and quay.io/openstack.trove/mariadb images too.\n\nMySQL backup doesn\u0027t work though, but it isn\u0027t related to changes in this patchset.\nProposed fix for mysql backup is here: https://review.opendev.org/c/openstack/trove/+/976002\n\nAlso there is some issue with PostgreSQL backup test scenario, but it\u0027s also not related to this patchset.","commit_id":"3a8666113d320710bba532d30050801fc4e54476"},{"author":{"_account_id":26285,"name":"wu.chunyang","email":"wchy1001@gmail.com","username":"wu.chunyang"},"change_message_id":"f47c3d17d05348d26f21546e8c1cc5d853fa89df","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":17,"id":"4dac6265_a6737948","updated":"2026-02-26 01:38:39.000000000","message":"Hello, it looks like there\u0027s a bug [here](https://zuul.opendev.org/t/openstack/build/0b09a7767dd34d19af180a715fc3be24/log/controller/logs/guest-agent-logs/938d41ae-0cf8-4a6d-bff5-9637beb85309/log/trove/trove-guestagent_log.txt#1206) – the tests are failing with the mariadb:11.4-ubi9 image. For details, you can check the test logs.","commit_id":"3a8666113d320710bba532d30050801fc4e54476"},{"author":{"_account_id":36080,"name":"Erkin Mussurmankulov","display_name":"Eric","email":"mangust404@gmail.com","username":"mongoose404","status":"PS Cloud services employee"},"change_message_id":"8d74f6e52ef18483423f1f5d40ef1b7673ad98b3","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":17,"id":"14579cb8_9de72842","in_reply_to":"031f28a0_b0923951","updated":"2026-02-28 06:20:30.000000000","message":"Yes, I got it, finally it works now.\n\nThe root cause is that `docker-entrypoint.sh` in ubi image calls `mariadb-upgrade` command when restoring from backup without specified `--defaults-file`.\nI\u0027ve added a workaround, binding `/etc/mysql/my.cnf` to `/etc/my.cnf` (`mariadb-upgrade` considers `/etc/my.cnf` by default).\nLooks like the trend of going away from all \"my*\" in config names in MariaDB is still present.\n\nSo probably in the future, instead of adding more and more workarounds, we should create a YAML file e.g. in `trove/templates/mariadb/paths.yaml` (and the same for mysql) which will contain correct paths to configurations and dirs, depending on datastore version. And trove will prepare and generate configs in correct dirs, which are suitable for a certain mariadb version.\nHow do you think?","commit_id":"3a8666113d320710bba532d30050801fc4e54476"},{"author":{"_account_id":26285,"name":"wu.chunyang","email":"wchy1001@gmail.com","username":"wu.chunyang"},"change_message_id":"624a6e5f7bfdc2a64c9144bf1b848c4e393e88bc","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":17,"id":"031f28a0_b0923951","in_reply_to":"10d3b897_9d7ab0ba","updated":"2026-02-27 02:44:41.000000000","message":"Hi, this is not caused by the id_admin issue. From the trove guest agent logs:\n\n\n```\nGot the following error when executing the \u0027mysql\u0027 command line client\", \"ERROR 2002 (HY000): Can\u0027t connect to local server through socket \u0027/run/mariadb/mariadb.sock\u0027 (2)\", \u0027FATAL ERROR: Upgrade failed\u0027, \u00272026-02-26 20:53:47+00:00 [Note] [Entrypoint]: Entrypoint script for MariaDB Server 11.4.10 started.\u0027] reset_password_for_restore\n```\n\nIt looks like the database upgrade failed while setting the password. However, in my development environment, the mariadb:11.4-ubi9 image works fine. This suggests that `mysqld` may not have started when the password was being set due to the lower performance in the CI environment, this may be the same reason we set a default timeout [here](https://github.com/openstack/trove/blob/master/playbooks/images/mariadb/11.4/Dockerfile)\n\nThe new logs: https://zuul.opendev.org/t/openstack/build/20acab4ea9134bcab7e858ccfa1564b7/log/controller/logs/guest-agent-logs/fcdee0ef-72bf-4ae0-9c55-c440fc3b18c6/log/trove/trove-guestagent_log.txt","commit_id":"3a8666113d320710bba532d30050801fc4e54476"},{"author":{"_account_id":36080,"name":"Erkin Mussurmankulov","display_name":"Eric","email":"mangust404@gmail.com","username":"mongoose404","status":"PS Cloud services employee"},"change_message_id":"3e6e3674591625030ef795e04af6c43ede4f52bb","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":17,"id":"10d3b897_9d7ab0ba","in_reply_to":"356b3160_ae580891","updated":"2026-02-26 10:55:42.000000000","message":"is_admin works differently now: https://github.com/openstack/oslo.context/commit/45fea5f220b9d07ee93b409f802ac518ea4b148b","commit_id":"3a8666113d320710bba532d30050801fc4e54476"},{"author":{"_account_id":26285,"name":"wu.chunyang","email":"wchy1001@gmail.com","username":"wu.chunyang"},"change_message_id":"8ddce17c24c506c9eab1ed8fb9516dcb2069328b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":17,"id":"4f9bcbf2_d2779ef4","in_reply_to":"35774672_adccc773","updated":"2026-02-12 13:16:22.000000000","message":"Hello, I add a new test [here](https://review.opendev.org/c/openstack/trove/+/976485), Let\u0027s wait for the result","commit_id":"3a8666113d320710bba532d30050801fc4e54476"},{"author":{"_account_id":36080,"name":"Erkin Mussurmankulov","display_name":"Eric","email":"mangust404@gmail.com","username":"mongoose404","status":"PS Cloud services employee"},"change_message_id":"c89d2bace1ca4aec34afeeb5cbbe434de3ab51b0","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":17,"id":"356b3160_ae580891","in_reply_to":"4dac6265_a6737948","updated":"2026-02-26 10:51:38.000000000","message":"Hello! It\u0027s a bug in context object, looks like it was induced by new version of oslo_context library, is_admin is not a property anymore.","commit_id":"3a8666113d320710bba532d30050801fc4e54476"}]}
