)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":15197,"name":"Pierre Riteau","email":"pierre@stackhpc.com","username":"priteau","status":"StackHPC"},"change_message_id":"57fcd6f26e386ec8c15abfaf718a091400c77aa1","unresolved":true,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"So far, the test-proxysql.sh has been failing because ``mariadb``"},{"line_number":10,"context_line":"command is not installed on testing host."},{"line_number":11,"context_line":"The problem is, it has been terminated  silently without reporting its"},{"line_number":12,"context_line":"failure."},{"line_number":13,"context_line":"Added errexit option and mariadb installtion function to fix this."},{"line_number":14,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"9e56b904_f7c87384","line":11,"range":{"start_line":11,"start_character":38,"end_line":11,"end_character":40},"updated":"2026-03-02 13:26:44.000000000","message":"duplicate space","commit_id":"6aa9d1c39cc36aded2d280db28ab9f997794f179"},{"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":"d4e143d8d31bf8e5825ba1a108594dcaff1db288","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"So far, the test-proxysql.sh has been failing because ``mariadb``"},{"line_number":10,"context_line":"command is not installed on testing host."},{"line_number":11,"context_line":"The problem is, it has been terminated  silently without reporting its"},{"line_number":12,"context_line":"failure."},{"line_number":13,"context_line":"Added errexit option and mariadb installtion function to fix this."},{"line_number":14,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"8db98d37_79408f83","line":11,"range":{"start_line":11,"start_character":38,"end_line":11,"end_character":40},"in_reply_to":"9e56b904_f7c87384","updated":"2026-03-02 14:11:10.000000000","message":"Acknowledged","commit_id":"6aa9d1c39cc36aded2d280db28ab9f997794f179"},{"author":{"_account_id":15197,"name":"Pierre Riteau","email":"pierre@stackhpc.com","username":"priteau","status":"StackHPC"},"change_message_id":"57fcd6f26e386ec8c15abfaf718a091400c77aa1","unresolved":true,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Fix ProxySQL test silently failing"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"So far, the test-proxysql.sh has been failing because ``mariadb``"},{"line_number":10,"context_line":"command is not installed on testing host."},{"line_number":11,"context_line":"The problem is, it has been terminated  silently without reporting its"},{"line_number":12,"context_line":"failure."},{"line_number":13,"context_line":"Added errexit option and mariadb installtion function to fix this."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Change-Id: I5a6f9599e83cb2ea32ae7a16764967e6c8b73eda"},{"line_number":16,"context_line":"Signed-off-by: Seunghun Lee \u003cseunghun@stackhpc.com\u003e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"79864c61_40020490","line":13,"range":{"start_line":9,"start_character":0,"end_line":13,"end_character":66},"updated":"2026-03-02 13:26:44.000000000","message":"Style nit: continue on the same line or make a new paragraph with an empty line","commit_id":"6aa9d1c39cc36aded2d280db28ab9f997794f179"},{"author":{"_account_id":15197,"name":"Pierre Riteau","email":"pierre@stackhpc.com","username":"priteau","status":"StackHPC"},"change_message_id":"57fcd6f26e386ec8c15abfaf718a091400c77aa1","unresolved":true,"context_lines":[{"line_number":10,"context_line":"command is not installed on testing host."},{"line_number":11,"context_line":"The problem is, it has been terminated  silently without reporting its"},{"line_number":12,"context_line":"failure."},{"line_number":13,"context_line":"Added errexit option and mariadb installtion function to fix this."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Change-Id: I5a6f9599e83cb2ea32ae7a16764967e6c8b73eda"},{"line_number":16,"context_line":"Signed-off-by: Seunghun Lee \u003cseunghun@stackhpc.com\u003e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"486c1ad3_cd2303f1","line":13,"range":{"start_line":13,"start_character":33,"end_line":13,"end_character":44},"updated":"2026-03-02 13:26:44.000000000","message":"installation","commit_id":"6aa9d1c39cc36aded2d280db28ab9f997794f179"},{"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":"d4e143d8d31bf8e5825ba1a108594dcaff1db288","unresolved":false,"context_lines":[{"line_number":10,"context_line":"command is not installed on testing host."},{"line_number":11,"context_line":"The problem is, it has been terminated  silently without reporting its"},{"line_number":12,"context_line":"failure."},{"line_number":13,"context_line":"Added errexit option and mariadb installtion function to fix this."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Change-Id: I5a6f9599e83cb2ea32ae7a16764967e6c8b73eda"},{"line_number":16,"context_line":"Signed-off-by: Seunghun Lee \u003cseunghun@stackhpc.com\u003e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"ce26be52_6f1e2baf","line":13,"range":{"start_line":13,"start_character":33,"end_line":13,"end_character":44},"in_reply_to":"486c1ad3_cd2303f1","updated":"2026-03-02 14:11:10.000000000","message":"Acknowledged","commit_id":"6aa9d1c39cc36aded2d280db28ab9f997794f179"},{"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":"d4e143d8d31bf8e5825ba1a108594dcaff1db288","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Fix ProxySQL test silently failing"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"So far, the test-proxysql.sh has been failing because ``mariadb``"},{"line_number":10,"context_line":"command is not installed on testing host."},{"line_number":11,"context_line":"The problem is, it has been terminated  silently without reporting its"},{"line_number":12,"context_line":"failure."},{"line_number":13,"context_line":"Added errexit option and mariadb installtion function to fix this."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Change-Id: I5a6f9599e83cb2ea32ae7a16764967e6c8b73eda"},{"line_number":16,"context_line":"Signed-off-by: Seunghun Lee \u003cseunghun@stackhpc.com\u003e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"060a43e0_d987b101","line":13,"range":{"start_line":9,"start_character":0,"end_line":13,"end_character":66},"in_reply_to":"79864c61_40020490","updated":"2026-03-02 14:11:10.000000000","message":"Acknowledged","commit_id":"6aa9d1c39cc36aded2d280db28ab9f997794f179"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"1e2b26539e33ee83da42f081287bf8a8611b2140","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":2,"id":"3800efd7_13a4dbbd","updated":"2026-03-02 14:46:31.000000000","message":"Not that I\u0027m picky - but maybe let\u0027s rework it to Ansible like the majority of other tests while we\u0027re touching this?\n\nAdd CI: prefix in commit subject","commit_id":"b570ca51c1ec8e03236d3abccb0c777c1af91f88"},{"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":"74d65fc72b62cd8e14aa88045dca33cfdd8e5bb8","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"b11166b4_0438e6dc","in_reply_to":"09a09ba1_74a0339d","updated":"2026-03-03 12:46:46.000000000","message":"Done","commit_id":"b570ca51c1ec8e03236d3abccb0c777c1af91f88"},{"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":"ea1d13beb9b038f1e584d7b52416a1875f61af58","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":2,"id":"a0d56861_14dd3baa","in_reply_to":"3800efd7_13a4dbbd","updated":"2026-03-03 10:01:52.000000000","message":"What tests are you referring here? I\u0027m probably missing the place to look for tests in Ansible.","commit_id":"b570ca51c1ec8e03236d3abccb0c777c1af91f88"},{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"f0a8911168168361b9469263e88bbc99b00f40fa","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":2,"id":"09a09ba1_74a0339d","in_reply_to":"a0d56861_14dd3baa","updated":"2026-03-03 10:04:46.000000000","message":"Look under roles/ directory","commit_id":"b570ca51c1ec8e03236d3abccb0c777c1af91f88"},{"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":"36a850db764153acc80477676097c5e40cd1f663","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"ce253688_bfd06300","updated":"2026-03-03 11:47:43.000000000","message":"Zuul is broken it seems","commit_id":"c4030b459b72657967d1ff691245bf6a5551072b"},{"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":"7471dc5381aadf419a048b19fccfb13265bdf4c1","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"182f3f77_fcb97535","updated":"2026-03-03 12:51:57.000000000","message":"recheck","commit_id":"c4030b459b72657967d1ff691245bf6a5551072b"},{"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":"c17317b18fe248c24b257fd4642484a11dd297cc","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"b99e9fc2_f15ce5a2","in_reply_to":"ce253688_bfd06300","updated":"2026-03-03 12:59:29.000000000","message":"No I was just blind","commit_id":"c4030b459b72657967d1ff691245bf6a5551072b"},{"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":"6a9a662e4f428638399c32c18733963493e2c806","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":16,"id":"af1b5667_f82a5e74","updated":"2026-03-05 12:32:15.000000000","message":"Cells scenarios is non-voting, so need to check if those are passing","commit_id":"e6b9ca86bf0b3e846aeb840ec9fe5a1e9cbd860c"},{"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":"c5d83ceca3b8c0ffb14908498d9e00731d82a07a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":16,"id":"e499b28e_a12a4b4e","updated":"2026-03-06 16:34:44.000000000","message":"recheck: CI was broken","commit_id":"e6b9ca86bf0b3e846aeb840ec9fe5a1e9cbd860c"},{"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":"cde9cc00a4ef9dd717791235d8b76f69dd83fefa","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":16,"id":"f0661ebc_8b45f344","updated":"2026-03-11 09:35:35.000000000","message":"recheck: There was CI failure due to heat-dashboard","commit_id":"e6b9ca86bf0b3e846aeb840ec9fe5a1e9cbd860c"},{"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":"43f0694f65247e92cfef4bb11a9b6a399ef02dc3","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":16,"id":"0f53b27a_b0440291","updated":"2026-03-06 13:16:15.000000000","message":"recheck: Trying again","commit_id":"e6b9ca86bf0b3e846aeb840ec9fe5a1e9cbd860c"},{"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":"c7d3f256a92a3ec45da8d78061626ecbbf1cb321","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":16,"id":"40bb8c4a_4d0df54a","updated":"2026-03-06 09:44:30.000000000","message":"recheck: debian cells scenario failed with tempest. This change is irrelevant to openstack functionalities","commit_id":"e6b9ca86bf0b3e846aeb840ec9fe5a1e9cbd860c"},{"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":"32c3867e0a8fe81174a1f34246be135b6ed4acb0","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":16,"id":"b8e6ac45_6bc84e64","updated":"2026-03-09 09:28:53.000000000","message":"recheck: unrelated failures","commit_id":"e6b9ca86bf0b3e846aeb840ec9fe5a1e9cbd860c"},{"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":"2eafb881cbf8fcc00b0fbc587270dfca8124f88e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":16,"id":"c3e59e68_d8320203","in_reply_to":"40bb8c4a_4d0df54a","updated":"2026-03-06 11:21:16.000000000","message":"recheck: tox fix has landed","commit_id":"e6b9ca86bf0b3e846aeb840ec9fe5a1e9cbd860c"},{"author":{"_account_id":37306,"name":"Piotr Milewski","display_name":"Piotr Milewski","email":"vurmil@gmail.com","username":"vurmil"},"change_message_id":"b847b0f4e9aea4ba65a726f4026f81d7b8295c3b","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":17,"id":"137d7e61_276d78b4","updated":"2026-03-13 15:17:05.000000000","message":"# Check Galera cluster health (size and sync status)\n- name: Check Galera Cluster health\n  community.mysql.mysql_query:\n    login_host: \"{{ test_database_vip }}\"\n    login_user: \"{{ test_database_user }}\"\n    login_password: \"{{ test_database_password }}\"\n    login_port: \"{{ test_database_port }}\"\n    query: \n      - SHOW STATUS LIKE \u0027wsrep_cluster_size\u0027\n      - SHOW STATUS LIKE \u0027wsrep_local_state_comment\u0027\n  register: cluster_status\n  vars:\n    ansible_python_interpreter: \"{{ kolla_ansible_venv_path }}/bin/python\"\n\n- name: Assert that all database nodes are synced and present\n  ansible.builtin.assert:\n    that:\n      - cluster_status.query_result[0][0].Value | int \u003e\u003d 3\n      - cluster_status.query_result[1][0].Value \u003d\u003d \u0027Synced\u0027\n    fail_msg: \"Galera cluster is unhealthy or nodes are missing/desynced\"\n\n\nMaybe it\u0027s worth adding it right away?","commit_id":"d6b824ec777493ef06d05596eae20b49929ae1c4"},{"author":{"_account_id":17669,"name":"Doug Szumski","email":"doug@stackhpc.com","username":"DougSzumski"},"change_message_id":"2afe9fde0c45bba4a3282ba739da73fdf47af8d9","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":17,"id":"a73e9b1c_f9f11c45","updated":"2026-03-13 14:54:20.000000000","message":"Thanks for fixing it up - it mostly looks fine to me.","commit_id":"d6b824ec777493ef06d05596eae20b49929ae1c4"},{"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":"ac044bcf786b86275f0a3a951a3fe7c77151aac5","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":17,"id":"1ff8c52c_20e50139","in_reply_to":"137d7e61_276d78b4","updated":"2026-03-13 15:49:25.000000000","message":"I think this should be part of mariadb test not proxysql","commit_id":"d6b824ec777493ef06d05596eae20b49929ae1c4"},{"author":{"_account_id":37306,"name":"Piotr Milewski","display_name":"Piotr Milewski","email":"vurmil@gmail.com","username":"vurmil"},"change_message_id":"41dbe4cb433d75eb0585ea016c5b5683ec91bcd7","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":17,"id":"310273b9_a04ed95f","in_reply_to":"1ff8c52c_20e50139","updated":"2026-03-13 15:57:31.000000000","message":"maybe something like this from the proxysql point of view?\n\n- name: Check if ProxySQL sees all backend nodes as ONLINE\n  community.mysql.mysql_query:\n    login_host: \"{{ test_database_vip }}\"\n    login_port: 6032           # ProxySQL Admin Port\n    login_user: \"admin\"\n    login_password: \"{{ proxysql_admin_password }}\"\n    query: \"SELECT COUNT(*) as online_nodes FROM mysql_servers WHERE status\u003d\u0027ONLINE\u0027\"\n  register: proxysql_nodes\n\n\nsometimes in some strange errors it can be useful information","commit_id":"d6b824ec777493ef06d05596eae20b49929ae1c4"},{"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":"149ab6f96bc67fd6453063a3340a14623d8ce251","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":17,"id":"248d1870_ec89f660","in_reply_to":"310273b9_a04ed95f","updated":"2026-03-13 16:31:35.000000000","message":"This sounds fair. I\u0027ll upload a new patchset with something like this.","commit_id":"d6b824ec777493ef06d05596eae20b49929ae1c4"},{"author":{"_account_id":17669,"name":"Doug Szumski","email":"doug@stackhpc.com","username":"DougSzumski"},"change_message_id":"8feb7cccd832d61d3d1aeff64969d26c8a53a070","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":19,"id":"900a362c_7444cb98","updated":"2026-03-16 13:47:49.000000000","message":"Thanks Seunghun","commit_id":"f5f65f98e021df2aad255d6135c6f7afb6153c3a"},{"author":{"_account_id":17669,"name":"Doug Szumski","email":"doug@stackhpc.com","username":"DougSzumski"},"change_message_id":"b327bd88968e20b7f5ebe7e1e365da456302862b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":20,"id":"9f9e7ba4_b2f1fd29","updated":"2026-03-16 17:50:09.000000000","message":"recheck instance build fail in Tempest","commit_id":"6b88348996ec7ae8745ec0b44371e2eb07fc0a15"}],"roles/kolla-ansible-test-proxysql/defaults/main.yml":[{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"dcb3d05e37be8bfab851c3b5f6089bd00dfab05e","unresolved":true,"context_lines":[{"line_number":2,"context_line":"test_database_port: 3306"},{"line_number":3,"context_line":"test_database_user: \"root_shard_0\""},{"line_number":4,"context_line":"test_database_tls_enabled: false"},{"line_number":5,"context_line":"test_database_vip: \"\""},{"line_number":6,"context_line":"test_database_password: \"\""}],"source_content_type":"text/x-yaml","patch_set":6,"id":"b4dfa9a4_057624da","line":5,"updated":"2026-03-03 18:29:43.000000000","message":"maybe use undef filter?","commit_id":"8f80ace7852d0c0c07641ecd18c2f2632e8de9f5"},{"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":"6a9a662e4f428638399c32c18733963493e2c806","unresolved":false,"context_lines":[{"line_number":2,"context_line":"test_database_port: 3306"},{"line_number":3,"context_line":"test_database_user: \"root_shard_0\""},{"line_number":4,"context_line":"test_database_tls_enabled: false"},{"line_number":5,"context_line":"test_database_vip: \"\""},{"line_number":6,"context_line":"test_database_password: \"\""}],"source_content_type":"text/x-yaml","patch_set":6,"id":"6b88b0a4_a436053b","line":5,"in_reply_to":"b4dfa9a4_057624da","updated":"2026-03-05 12:32:15.000000000","message":"Done","commit_id":"8f80ace7852d0c0c07641ecd18c2f2632e8de9f5"},{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"dcb3d05e37be8bfab851c3b5f6089bd00dfab05e","unresolved":true,"context_lines":[{"line_number":3,"context_line":"test_database_user: \"root_shard_0\""},{"line_number":4,"context_line":"test_database_tls_enabled: false"},{"line_number":5,"context_line":"test_database_vip: \"\""},{"line_number":6,"context_line":"test_database_password: \"\""}],"source_content_type":"text/x-yaml","patch_set":6,"id":"8d2aeacb_caf1dbc3","line":6,"updated":"2026-03-03 18:29:43.000000000","message":"ditto","commit_id":"8f80ace7852d0c0c07641ecd18c2f2632e8de9f5"},{"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":"6a9a662e4f428638399c32c18733963493e2c806","unresolved":false,"context_lines":[{"line_number":3,"context_line":"test_database_user: \"root_shard_0\""},{"line_number":4,"context_line":"test_database_tls_enabled: false"},{"line_number":5,"context_line":"test_database_vip: \"\""},{"line_number":6,"context_line":"test_database_password: \"\""}],"source_content_type":"text/x-yaml","patch_set":6,"id":"9521ba4b_3395f83e","line":6,"in_reply_to":"8d2aeacb_caf1dbc3","updated":"2026-03-05 12:32:15.000000000","message":"Done","commit_id":"8f80ace7852d0c0c07641ecd18c2f2632e8de9f5"}],"roles/kolla-ansible-test-proxysql/tasks/main.yml":[{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"dcb3d05e37be8bfab851c3b5f6089bd00dfab05e","unresolved":true,"context_lines":[{"line_number":2,"context_line":"- name: Assert that database VIP is set"},{"line_number":3,"context_line":"  ansible.builtin.assert:"},{"line_number":4,"context_line":"    that:"},{"line_number":5,"context_line":"      - test_database_vip is defined"},{"line_number":6,"context_line":"      - test_database_vip !\u003d \"\""},{"line_number":7,"context_line":""},{"line_number":8,"context_line":"- name: Assert that database password is set"}],"source_content_type":"text/x-yaml","patch_set":6,"id":"55ccf4d1_4b9c7984","line":5,"updated":"2026-03-03 18:29:43.000000000","message":"it\u0027s always defined because you added it to defaults","commit_id":"8f80ace7852d0c0c07641ecd18c2f2632e8de9f5"},{"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":"229a07e732be3b287587f9d0e93b27f673445458","unresolved":false,"context_lines":[{"line_number":2,"context_line":"- name: Assert that database VIP is set"},{"line_number":3,"context_line":"  ansible.builtin.assert:"},{"line_number":4,"context_line":"    that:"},{"line_number":5,"context_line":"      - test_database_vip is defined"},{"line_number":6,"context_line":"      - test_database_vip !\u003d \"\""},{"line_number":7,"context_line":""},{"line_number":8,"context_line":"- name: Assert that database password is set"}],"source_content_type":"text/x-yaml","patch_set":6,"id":"8a774c01_27cc34de","line":5,"in_reply_to":"55ccf4d1_4b9c7984","updated":"2026-03-04 10:22:43.000000000","message":"Done","commit_id":"8f80ace7852d0c0c07641ecd18c2f2632e8de9f5"},{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"dcb3d05e37be8bfab851c3b5f6089bd00dfab05e","unresolved":true,"context_lines":[{"line_number":8,"context_line":"- name: Assert that database password is set"},{"line_number":9,"context_line":"  ansible.builtin.assert:"},{"line_number":10,"context_line":"    that:"},{"line_number":11,"context_line":"      - test_database_password is defined"},{"line_number":12,"context_line":"      - test_database_password !\u003d \"\""},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"- name: Install MariaDB client (Redhat family)"}],"source_content_type":"text/x-yaml","patch_set":6,"id":"de0b4780_cae29bb4","line":11,"updated":"2026-03-03 18:29:43.000000000","message":"ditto","commit_id":"8f80ace7852d0c0c07641ecd18c2f2632e8de9f5"},{"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":"229a07e732be3b287587f9d0e93b27f673445458","unresolved":false,"context_lines":[{"line_number":8,"context_line":"- name: Assert that database password is set"},{"line_number":9,"context_line":"  ansible.builtin.assert:"},{"line_number":10,"context_line":"    that:"},{"line_number":11,"context_line":"      - test_database_password is defined"},{"line_number":12,"context_line":"      - test_database_password !\u003d \"\""},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"- name: Install MariaDB client (Redhat family)"}],"source_content_type":"text/x-yaml","patch_set":6,"id":"7d07d867_0f85478f","line":11,"in_reply_to":"de0b4780_cae29bb4","updated":"2026-03-04 10:22:43.000000000","message":"Done","commit_id":"8f80ace7852d0c0c07641ecd18c2f2632e8de9f5"},{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"dcb3d05e37be8bfab851c3b5f6089bd00dfab05e","unresolved":true,"context_lines":[{"line_number":13,"context_line":""},{"line_number":14,"context_line":"- name: Install MariaDB client (Redhat family)"},{"line_number":15,"context_line":"  ansible.builtin.dnf:"},{"line_number":16,"context_line":"    name: \"mariadb\""},{"line_number":17,"context_line":"    state: \"present\""},{"line_number":18,"context_line":"  when: ansible_os_family \u003d\u003d \"RedHat\""},{"line_number":19,"context_line":"  become: true"}],"source_content_type":"text/x-yaml","patch_set":6,"id":"dc86ff38_fc682217","line":16,"updated":"2026-03-03 18:29:43.000000000","message":"no need for quotes","commit_id":"8f80ace7852d0c0c07641ecd18c2f2632e8de9f5"},{"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":"229a07e732be3b287587f9d0e93b27f673445458","unresolved":false,"context_lines":[{"line_number":13,"context_line":""},{"line_number":14,"context_line":"- name: Install MariaDB client (Redhat family)"},{"line_number":15,"context_line":"  ansible.builtin.dnf:"},{"line_number":16,"context_line":"    name: \"mariadb\""},{"line_number":17,"context_line":"    state: \"present\""},{"line_number":18,"context_line":"  when: ansible_os_family \u003d\u003d \"RedHat\""},{"line_number":19,"context_line":"  become: true"}],"source_content_type":"text/x-yaml","patch_set":6,"id":"2ee10f5b_d355be15","line":16,"in_reply_to":"dc86ff38_fc682217","updated":"2026-03-04 10:22:43.000000000","message":"Done","commit_id":"8f80ace7852d0c0c07641ecd18c2f2632e8de9f5"},{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"dcb3d05e37be8bfab851c3b5f6089bd00dfab05e","unresolved":true,"context_lines":[{"line_number":14,"context_line":"- name: Install MariaDB client (Redhat family)"},{"line_number":15,"context_line":"  ansible.builtin.dnf:"},{"line_number":16,"context_line":"    name: \"mariadb\""},{"line_number":17,"context_line":"    state: \"present\""},{"line_number":18,"context_line":"  when: ansible_os_family \u003d\u003d \"RedHat\""},{"line_number":19,"context_line":"  become: true"},{"line_number":20,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":6,"id":"17320c26_a2c62469","line":17,"updated":"2026-03-03 18:29:43.000000000","message":"ditto","commit_id":"8f80ace7852d0c0c07641ecd18c2f2632e8de9f5"},{"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":"229a07e732be3b287587f9d0e93b27f673445458","unresolved":false,"context_lines":[{"line_number":14,"context_line":"- name: Install MariaDB client (Redhat family)"},{"line_number":15,"context_line":"  ansible.builtin.dnf:"},{"line_number":16,"context_line":"    name: \"mariadb\""},{"line_number":17,"context_line":"    state: \"present\""},{"line_number":18,"context_line":"  when: ansible_os_family \u003d\u003d \"RedHat\""},{"line_number":19,"context_line":"  become: true"},{"line_number":20,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":6,"id":"9910aa17_b9578357","line":17,"in_reply_to":"17320c26_a2c62469","updated":"2026-03-04 10:22:43.000000000","message":"Done","commit_id":"8f80ace7852d0c0c07641ecd18c2f2632e8de9f5"},{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"dcb3d05e37be8bfab851c3b5f6089bd00dfab05e","unresolved":true,"context_lines":[{"line_number":15,"context_line":"  ansible.builtin.dnf:"},{"line_number":16,"context_line":"    name: \"mariadb\""},{"line_number":17,"context_line":"    state: \"present\""},{"line_number":18,"context_line":"  when: ansible_os_family \u003d\u003d \"RedHat\""},{"line_number":19,"context_line":"  become: true"},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"- name: Install MariaDB client (Debian family)"}],"source_content_type":"text/x-yaml","patch_set":6,"id":"f1947761_0f16418f","line":18,"updated":"2026-03-03 18:29:43.000000000","message":"use ansible_facts.os_family","commit_id":"8f80ace7852d0c0c07641ecd18c2f2632e8de9f5"},{"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":"229a07e732be3b287587f9d0e93b27f673445458","unresolved":false,"context_lines":[{"line_number":15,"context_line":"  ansible.builtin.dnf:"},{"line_number":16,"context_line":"    name: \"mariadb\""},{"line_number":17,"context_line":"    state: \"present\""},{"line_number":18,"context_line":"  when: ansible_os_family \u003d\u003d \"RedHat\""},{"line_number":19,"context_line":"  become: true"},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"- name: Install MariaDB client (Debian family)"}],"source_content_type":"text/x-yaml","patch_set":6,"id":"aadd9e60_99a70798","line":18,"in_reply_to":"f1947761_0f16418f","updated":"2026-03-04 10:22:43.000000000","message":"Done","commit_id":"8f80ace7852d0c0c07641ecd18c2f2632e8de9f5"},{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"dcb3d05e37be8bfab851c3b5f6089bd00dfab05e","unresolved":true,"context_lines":[{"line_number":20,"context_line":""},{"line_number":21,"context_line":"- name: Install MariaDB client (Debian family)"},{"line_number":22,"context_line":"  ansible.builtin.apt:"},{"line_number":23,"context_line":"    name: \"mariadb-client\""},{"line_number":24,"context_line":"    state: \"present\""},{"line_number":25,"context_line":"  when: ansible_os_family \u003d\u003d \"Debian\""},{"line_number":26,"context_line":"  become: true"}],"source_content_type":"text/x-yaml","patch_set":6,"id":"fe725683_b7d58f2a","line":23,"updated":"2026-03-03 18:29:43.000000000","message":"ditto","commit_id":"8f80ace7852d0c0c07641ecd18c2f2632e8de9f5"},{"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":"229a07e732be3b287587f9d0e93b27f673445458","unresolved":false,"context_lines":[{"line_number":20,"context_line":""},{"line_number":21,"context_line":"- name: Install MariaDB client (Debian family)"},{"line_number":22,"context_line":"  ansible.builtin.apt:"},{"line_number":23,"context_line":"    name: \"mariadb-client\""},{"line_number":24,"context_line":"    state: \"present\""},{"line_number":25,"context_line":"  when: ansible_os_family \u003d\u003d \"Debian\""},{"line_number":26,"context_line":"  become: true"}],"source_content_type":"text/x-yaml","patch_set":6,"id":"fcbce9f0_8897a66f","line":23,"in_reply_to":"fe725683_b7d58f2a","updated":"2026-03-04 10:22:43.000000000","message":"Done","commit_id":"8f80ace7852d0c0c07641ecd18c2f2632e8de9f5"},{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"dcb3d05e37be8bfab851c3b5f6089bd00dfab05e","unresolved":true,"context_lines":[{"line_number":21,"context_line":"- name: Install MariaDB client (Debian family)"},{"line_number":22,"context_line":"  ansible.builtin.apt:"},{"line_number":23,"context_line":"    name: \"mariadb-client\""},{"line_number":24,"context_line":"    state: \"present\""},{"line_number":25,"context_line":"  when: ansible_os_family \u003d\u003d \"Debian\""},{"line_number":26,"context_line":"  become: true"},{"line_number":27,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":6,"id":"e0111788_4fc50da2","line":24,"updated":"2026-03-03 18:29:43.000000000","message":"ditto","commit_id":"8f80ace7852d0c0c07641ecd18c2f2632e8de9f5"},{"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":"229a07e732be3b287587f9d0e93b27f673445458","unresolved":false,"context_lines":[{"line_number":21,"context_line":"- name: Install MariaDB client (Debian family)"},{"line_number":22,"context_line":"  ansible.builtin.apt:"},{"line_number":23,"context_line":"    name: \"mariadb-client\""},{"line_number":24,"context_line":"    state: \"present\""},{"line_number":25,"context_line":"  when: ansible_os_family \u003d\u003d \"Debian\""},{"line_number":26,"context_line":"  become: true"},{"line_number":27,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":6,"id":"2278c5ce_3260c586","line":24,"in_reply_to":"e0111788_4fc50da2","updated":"2026-03-04 10:22:43.000000000","message":"Done","commit_id":"8f80ace7852d0c0c07641ecd18c2f2632e8de9f5"},{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"dcb3d05e37be8bfab851c3b5f6089bd00dfab05e","unresolved":true,"context_lines":[{"line_number":22,"context_line":"  ansible.builtin.apt:"},{"line_number":23,"context_line":"    name: \"mariadb-client\""},{"line_number":24,"context_line":"    state: \"present\""},{"line_number":25,"context_line":"  when: ansible_os_family \u003d\u003d \"Debian\""},{"line_number":26,"context_line":"  become: true"},{"line_number":27,"context_line":""},{"line_number":28,"context_line":"- name: Test ProxySQL connection"}],"source_content_type":"text/x-yaml","patch_set":6,"id":"37022a04_1b756084","line":25,"updated":"2026-03-03 18:29:43.000000000","message":"ditto","commit_id":"8f80ace7852d0c0c07641ecd18c2f2632e8de9f5"},{"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":"229a07e732be3b287587f9d0e93b27f673445458","unresolved":false,"context_lines":[{"line_number":22,"context_line":"  ansible.builtin.apt:"},{"line_number":23,"context_line":"    name: \"mariadb-client\""},{"line_number":24,"context_line":"    state: \"present\""},{"line_number":25,"context_line":"  when: ansible_os_family \u003d\u003d \"Debian\""},{"line_number":26,"context_line":"  become: true"},{"line_number":27,"context_line":""},{"line_number":28,"context_line":"- name: Test ProxySQL connection"}],"source_content_type":"text/x-yaml","patch_set":6,"id":"60274d5c_018f735a","line":25,"in_reply_to":"37022a04_1b756084","updated":"2026-03-04 10:22:43.000000000","message":"Done","commit_id":"8f80ace7852d0c0c07641ecd18c2f2632e8de9f5"},{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"dcb3d05e37be8bfab851c3b5f6089bd00dfab05e","unresolved":true,"context_lines":[{"line_number":26,"context_line":"  become: true"},{"line_number":27,"context_line":""},{"line_number":28,"context_line":"- name: Test ProxySQL connection"},{"line_number":29,"context_line":"  ansible.builtin.command:"},{"line_number":30,"context_line":"    cmd: mariadb -h $VIP -P$DATABASE_PORT -u$DATABASE_USER -p$DATABASE_PASSWORD -e \u0027SHOW TABLES\u0027"},{"line_number":31,"context_line":"  environment:"},{"line_number":32,"context_line":"    VIP: \"{{ test_database_vip }}\""}],"source_content_type":"text/x-yaml","patch_set":6,"id":"0bceec4d_00482c02","line":29,"updated":"2026-03-03 18:29:43.000000000","message":"community.mysql.mysql_query ?","commit_id":"8f80ace7852d0c0c07641ecd18c2f2632e8de9f5"},{"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":"229a07e732be3b287587f9d0e93b27f673445458","unresolved":false,"context_lines":[{"line_number":26,"context_line":"  become: true"},{"line_number":27,"context_line":""},{"line_number":28,"context_line":"- name: Test ProxySQL connection"},{"line_number":29,"context_line":"  ansible.builtin.command:"},{"line_number":30,"context_line":"    cmd: mariadb -h $VIP -P$DATABASE_PORT -u$DATABASE_USER -p$DATABASE_PASSWORD -e \u0027SHOW TABLES\u0027"},{"line_number":31,"context_line":"  environment:"},{"line_number":32,"context_line":"    VIP: \"{{ test_database_vip }}\""}],"source_content_type":"text/x-yaml","patch_set":6,"id":"a9da8644_447c4cf4","line":29,"in_reply_to":"0bceec4d_00482c02","updated":"2026-03-04 10:22:43.000000000","message":"Done","commit_id":"8f80ace7852d0c0c07641ecd18c2f2632e8de9f5"},{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"dcb3d05e37be8bfab851c3b5f6089bd00dfab05e","unresolved":true,"context_lines":[{"line_number":39,"context_line":"  when: test_database_tls_enabled"},{"line_number":40,"context_line":"  block:"},{"line_number":41,"context_line":"    - name: Query SSL information"},{"line_number":42,"context_line":"      ansible.builtin.command:"},{"line_number":43,"context_line":"        cmd: mariadb -h $VIP -P$DATABASE_PORT -u$DATABASE_USER -p$DATABASE_PASSWORD -e \u0027$QUERY\u0027 --silent"},{"line_number":44,"context_line":"      environment:"},{"line_number":45,"context_line":"        VIP: \"{{ test_database_vip }}\""}],"source_content_type":"text/x-yaml","patch_set":6,"id":"7b88dca5_f93e8a80","line":42,"updated":"2026-03-03 18:29:43.000000000","message":"ditto","commit_id":"8f80ace7852d0c0c07641ecd18c2f2632e8de9f5"},{"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":"229a07e732be3b287587f9d0e93b27f673445458","unresolved":false,"context_lines":[{"line_number":39,"context_line":"  when: test_database_tls_enabled"},{"line_number":40,"context_line":"  block:"},{"line_number":41,"context_line":"    - name: Query SSL information"},{"line_number":42,"context_line":"      ansible.builtin.command:"},{"line_number":43,"context_line":"        cmd: mariadb -h $VIP -P$DATABASE_PORT -u$DATABASE_USER -p$DATABASE_PASSWORD -e \u0027$QUERY\u0027 --silent"},{"line_number":44,"context_line":"      environment:"},{"line_number":45,"context_line":"        VIP: \"{{ test_database_vip }}\""}],"source_content_type":"text/x-yaml","patch_set":6,"id":"fab0e2ae_6e595936","line":42,"in_reply_to":"7b88dca5_f93e8a80","updated":"2026-03-04 10:22:43.000000000","message":"Done","commit_id":"8f80ace7852d0c0c07641ecd18c2f2632e8de9f5"},{"author":{"_account_id":17669,"name":"Doug Szumski","email":"doug@stackhpc.com","username":"DougSzumski"},"change_message_id":"2afe9fde0c45bba4a3282ba739da73fdf47af8d9","unresolved":true,"context_lines":[{"line_number":2,"context_line":"- name: Assert that database VIP is set"},{"line_number":3,"context_line":"  ansible.builtin.assert:"},{"line_number":4,"context_line":"    that:"},{"line_number":5,"context_line":"      - test_database_vip is defined"},{"line_number":6,"context_line":"      - test_database_vip !\u003d \"\""},{"line_number":7,"context_line":"    fail_msg: The variable test_database_vip is required"},{"line_number":8,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":17,"id":"b9756f64_e29ed17b","line":5,"updated":"2026-03-13 14:54:20.000000000","message":"nit. This is always defined via a task var in the task that calls this playbook.","commit_id":"d6b824ec777493ef06d05596eae20b49929ae1c4"},{"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":"ac044bcf786b86275f0a3a951a3fe7c77151aac5","unresolved":true,"context_lines":[{"line_number":2,"context_line":"- name: Assert that database VIP is set"},{"line_number":3,"context_line":"  ansible.builtin.assert:"},{"line_number":4,"context_line":"    that:"},{"line_number":5,"context_line":"      - test_database_vip is defined"},{"line_number":6,"context_line":"      - test_database_vip !\u003d \"\""},{"line_number":7,"context_line":"    fail_msg: The variable test_database_vip is required"},{"line_number":8,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":17,"id":"e4486ead_2679a90a","line":5,"in_reply_to":"b9756f64_e29ed17b","updated":"2026-03-13 15:49:25.000000000","message":"I was also considering the use case outside of our CI but if that\u0027s out of scope, we can remove this assertion.","commit_id":"d6b824ec777493ef06d05596eae20b49929ae1c4"},{"author":{"_account_id":17669,"name":"Doug Szumski","email":"doug@stackhpc.com","username":"DougSzumski"},"change_message_id":"8feb7cccd832d61d3d1aeff64969d26c8a53a070","unresolved":false,"context_lines":[{"line_number":2,"context_line":"- name: Assert that database VIP is set"},{"line_number":3,"context_line":"  ansible.builtin.assert:"},{"line_number":4,"context_line":"    that:"},{"line_number":5,"context_line":"      - test_database_vip is defined"},{"line_number":6,"context_line":"      - test_database_vip !\u003d \"\""},{"line_number":7,"context_line":"    fail_msg: The variable test_database_vip is required"},{"line_number":8,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":17,"id":"93f14066_737df16f","line":5,"in_reply_to":"e4486ead_2679a90a","updated":"2026-03-16 13:47:49.000000000","message":"Thanks, that makes sense, let\u0027s keep it then.","commit_id":"d6b824ec777493ef06d05596eae20b49929ae1c4"},{"author":{"_account_id":17669,"name":"Doug Szumski","email":"doug@stackhpc.com","username":"DougSzumski"},"change_message_id":"2afe9fde0c45bba4a3282ba739da73fdf47af8d9","unresolved":true,"context_lines":[{"line_number":19,"context_line":"      changed_when: false"},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"    - name: Set fact about database password"},{"line_number":22,"context_line":"      ansible.builtin.set_fact:"},{"line_number":23,"context_line":"        test_database_password: \"{{ db_password_grep.stdout }}\""},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"- name: Install required packages"}],"source_content_type":"text/x-yaml","patch_set":17,"id":"4dbb7f4c_ddfed3ca","line":22,"updated":"2026-03-13 14:54:20.000000000","message":"nit: I think you could replace this block with something like:\n\nset_fact:\n  test_database_password: \"{{ (lookup(\u0027file\u0027, \u0027/etc/kolla/passwords.yml\u0027)).database_password }}\"\n\nor something like this?","commit_id":"d6b824ec777493ef06d05596eae20b49929ae1c4"},{"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":"ac044bcf786b86275f0a3a951a3fe7c77151aac5","unresolved":true,"context_lines":[{"line_number":19,"context_line":"      changed_when: false"},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"    - name: Set fact about database password"},{"line_number":22,"context_line":"      ansible.builtin.set_fact:"},{"line_number":23,"context_line":"        test_database_password: \"{{ db_password_grep.stdout }}\""},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"- name: Install required packages"}],"source_content_type":"text/x-yaml","patch_set":17,"id":"e2be1eb0_8cdba121","line":22,"in_reply_to":"4dbb7f4c_ddfed3ca","updated":"2026-03-13 15:49:25.000000000","message":"Tested this but it does not work unfortunately. Looks like passwords need to be dynamically read. I think it\u0027s because passwords.yml is not populated before running run.yml? The previous test-proxysql.sh is also fetch password like this. I doubt this method would\u0027ve been used if passing password from Ansible variable was possible.","commit_id":"d6b824ec777493ef06d05596eae20b49929ae1c4"},{"author":{"_account_id":17669,"name":"Doug Szumski","email":"doug@stackhpc.com","username":"DougSzumski"},"change_message_id":"8feb7cccd832d61d3d1aeff64969d26c8a53a070","unresolved":false,"context_lines":[{"line_number":19,"context_line":"      changed_when: false"},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"    - name: Set fact about database password"},{"line_number":22,"context_line":"      ansible.builtin.set_fact:"},{"line_number":23,"context_line":"        test_database_password: \"{{ db_password_grep.stdout }}\""},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"- name: Install required packages"}],"source_content_type":"text/x-yaml","patch_set":17,"id":"94cc1410_af857d63","line":22,"in_reply_to":"e2be1eb0_8cdba121","updated":"2026-03-16 13:47:49.000000000","message":"Done","commit_id":"d6b824ec777493ef06d05596eae20b49929ae1c4"},{"author":{"_account_id":37306,"name":"Piotr Milewski","display_name":"Piotr Milewski","email":"vurmil@gmail.com","username":"vurmil"},"change_message_id":"b847b0f4e9aea4ba65a726f4026f81d7b8295c3b","unresolved":true,"context_lines":[{"line_number":24,"context_line":""},{"line_number":25,"context_line":"- name: Install required packages"},{"line_number":26,"context_line":"  ansible.builtin.pip:"},{"line_number":27,"context_line":"    name: PyMySQL"},{"line_number":28,"context_line":"    virtualenv: \"{{ kolla_ansible_venv_path }}\""},{"line_number":29,"context_line":"    virtualenv_command: \"python3 -m venv\""},{"line_number":30,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":17,"id":"38e75147_627b16fe","line":27,"range":{"start_line":27,"start_character":4,"end_line":27,"end_character":17},"updated":"2026-03-13 15:17:05.000000000","message":"pip install cryptography ? Isn\u0027t it by any chance required for TLS ?","commit_id":"d6b824ec777493ef06d05596eae20b49929ae1c4"},{"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":"ac044bcf786b86275f0a3a951a3fe7c77151aac5","unresolved":false,"context_lines":[{"line_number":24,"context_line":""},{"line_number":25,"context_line":"- name: Install required packages"},{"line_number":26,"context_line":"  ansible.builtin.pip:"},{"line_number":27,"context_line":"    name: PyMySQL"},{"line_number":28,"context_line":"    virtualenv: \"{{ kolla_ansible_venv_path }}\""},{"line_number":29,"context_line":"    virtualenv_command: \"python3 -m venv\""},{"line_number":30,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":17,"id":"ab60a2d6_9d2900fd","line":27,"range":{"start_line":27,"start_character":4,"end_line":27,"end_character":17},"in_reply_to":"38e75147_627b16fe","updated":"2026-03-13 15:49:25.000000000","message":"Tested with/without database TLS enabled, cryptography is not needed.","commit_id":"d6b824ec777493ef06d05596eae20b49929ae1c4"},{"author":{"_account_id":17669,"name":"Doug Szumski","email":"doug@stackhpc.com","username":"DougSzumski"},"change_message_id":"2afe9fde0c45bba4a3282ba739da73fdf47af8d9","unresolved":true,"context_lines":[{"line_number":34,"context_line":"    login_user: \"{{ test_database_user }}\""},{"line_number":35,"context_line":"    login_password: \"{{ test_database_password }}\""},{"line_number":36,"context_line":"    login_port: \"{{ test_database_port }}\""},{"line_number":37,"context_line":"    query: SHOW TABLES"},{"line_number":38,"context_line":"  register: table_query"},{"line_number":39,"context_line":"  vars:"},{"line_number":40,"context_line":"    ansible_python_interpreter: \"{{ kolla_ansible_venv_path }}/bin/python\""}],"source_content_type":"text/x-yaml","patch_set":17,"id":"d83d17b9_c0223d3d","line":37,"updated":"2026-03-13 14:54:20.000000000","message":"I assume there is some default timeout on this?","commit_id":"d6b824ec777493ef06d05596eae20b49929ae1c4"},{"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":"ac044bcf786b86275f0a3a951a3fe7c77151aac5","unresolved":false,"context_lines":[{"line_number":34,"context_line":"    login_user: \"{{ test_database_user }}\""},{"line_number":35,"context_line":"    login_password: \"{{ test_database_password }}\""},{"line_number":36,"context_line":"    login_port: \"{{ test_database_port }}\""},{"line_number":37,"context_line":"    query: SHOW TABLES"},{"line_number":38,"context_line":"  register: table_query"},{"line_number":39,"context_line":"  vars:"},{"line_number":40,"context_line":"    ansible_python_interpreter: \"{{ kolla_ansible_venv_path }}/bin/python\""}],"source_content_type":"text/x-yaml","patch_set":17,"id":"75657303_c0f0b70c","line":37,"in_reply_to":"d83d17b9_c0223d3d","updated":"2026-03-13 15:49:25.000000000","message":"Yes, default is 30 seconds","commit_id":"d6b824ec777493ef06d05596eae20b49929ae1c4"},{"author":{"_account_id":37306,"name":"Piotr Milewski","display_name":"Piotr Milewski","email":"vurmil@gmail.com","username":"vurmil"},"change_message_id":"b847b0f4e9aea4ba65a726f4026f81d7b8295c3b","unresolved":true,"context_lines":[{"line_number":41,"context_line":""},{"line_number":42,"context_line":"- name: Assert that test query was successful"},{"line_number":43,"context_line":"  ansible.builtin.assert:"},{"line_number":44,"context_line":"    that: not table_query.failed"},{"line_number":45,"context_line":""},{"line_number":46,"context_line":"- name: Test ProxySQL SSL connection"},{"line_number":47,"context_line":"  when: test_database_tls_enabled"}],"source_content_type":"text/x-yaml","patch_set":17,"id":"049fbcd2_59e0e404","line":44,"range":{"start_line":44,"start_character":3,"end_line":44,"end_character":32},"updated":"2026-03-13 15:17:05.000000000","message":"fail_msg: \"Database connection or query failed\" - ?","commit_id":"d6b824ec777493ef06d05596eae20b49929ae1c4"},{"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":"ac044bcf786b86275f0a3a951a3fe7c77151aac5","unresolved":false,"context_lines":[{"line_number":41,"context_line":""},{"line_number":42,"context_line":"- name: Assert that test query was successful"},{"line_number":43,"context_line":"  ansible.builtin.assert:"},{"line_number":44,"context_line":"    that: not table_query.failed"},{"line_number":45,"context_line":""},{"line_number":46,"context_line":"- name: Test ProxySQL SSL connection"},{"line_number":47,"context_line":"  when: test_database_tls_enabled"}],"source_content_type":"text/x-yaml","patch_set":17,"id":"cedb2532_dc1303ae","line":44,"range":{"start_line":44,"start_character":3,"end_line":44,"end_character":32},"in_reply_to":"049fbcd2_59e0e404","updated":"2026-03-13 15:49:25.000000000","message":"Acknowledged","commit_id":"d6b824ec777493ef06d05596eae20b49929ae1c4"},{"author":{"_account_id":37306,"name":"Piotr Milewski","display_name":"Piotr Milewski","email":"vurmil@gmail.com","username":"vurmil"},"change_message_id":"b847b0f4e9aea4ba65a726f4026f81d7b8295c3b","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":17,"id":"b236c313_4c88e837","line":66,"updated":"2026-03-13 15:17:05.000000000","message":"fail_msg: \"SSL is enabled but the current session is not encrypted\" - ?","commit_id":"d6b824ec777493ef06d05596eae20b49929ae1c4"},{"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":"ac044bcf786b86275f0a3a951a3fe7c77151aac5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":17,"id":"81e3a812_0ebee6ba","line":66,"in_reply_to":"b236c313_4c88e837","updated":"2026-03-13 15:49:25.000000000","message":"Acknowledged","commit_id":"d6b824ec777493ef06d05596eae20b49929ae1c4"},{"author":{"_account_id":17669,"name":"Doug Szumski","email":"doug@stackhpc.com","username":"DougSzumski"},"change_message_id":"8feb7cccd832d61d3d1aeff64969d26c8a53a070","unresolved":true,"context_lines":[{"line_number":75,"context_line":"  ansible.builtin.assert:"},{"line_number":76,"context_line":"    that:"},{"line_number":77,"context_line":"      - not mariadb_count.failed"},{"line_number":78,"context_line":"      - mariadb_count.rowconut[0] !\u003d 0 # Number of returned row of the first query is not zero"},{"line_number":79,"context_line":"      - mariadb_count.query_result[0][0][\u0027online_nodes\u0027] is defined # The first row of the first query is defined"},{"line_number":80,"context_line":"      - (mariadb_count.query_result[0][0][\u0027online_nodes\u0027] | int) \u003d\u003d (groups[\u0027mariadb\u0027] | length) # number of online nodes matches number of mariadb hosts"},{"line_number":81,"context_line":"    fail_msg: Number of online MariaDB servers does not match the number of MariaDB hosts"}],"source_content_type":"text/x-yaml","patch_set":19,"id":"2985bbfe_62d02ea9","line":78,"updated":"2026-03-16 13:47:49.000000000","message":"s/rowconut/rowcount ?","commit_id":"f5f65f98e021df2aad255d6135c6f7afb6153c3a"},{"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":"1aa2bd5a15df177ae0347b0e9cb9e6e3bd3f0dec","unresolved":false,"context_lines":[{"line_number":75,"context_line":"  ansible.builtin.assert:"},{"line_number":76,"context_line":"    that:"},{"line_number":77,"context_line":"      - not mariadb_count.failed"},{"line_number":78,"context_line":"      - mariadb_count.rowconut[0] !\u003d 0 # Number of returned row of the first query is not zero"},{"line_number":79,"context_line":"      - mariadb_count.query_result[0][0][\u0027online_nodes\u0027] is defined # The first row of the first query is defined"},{"line_number":80,"context_line":"      - (mariadb_count.query_result[0][0][\u0027online_nodes\u0027] | int) \u003d\u003d (groups[\u0027mariadb\u0027] | length) # number of online nodes matches number of mariadb hosts"},{"line_number":81,"context_line":"    fail_msg: Number of online MariaDB servers does not match the number of MariaDB hosts"}],"source_content_type":"text/x-yaml","patch_set":19,"id":"dbd0b29b_bd6076c6","line":78,"in_reply_to":"2985bbfe_62d02ea9","updated":"2026-03-16 13:50:07.000000000","message":"Oops. Nice catch. Thank you","commit_id":"f5f65f98e021df2aad255d6135c6f7afb6153c3a"}],"tests/test-proxysql.sh":[{"author":{"_account_id":15197,"name":"Pierre Riteau","email":"pierre@stackhpc.com","username":"priteau","status":"StackHPC"},"change_message_id":"57fcd6f26e386ec8c15abfaf718a091400c77aa1","unresolved":true,"context_lines":[{"line_number":4,"context_line":"set -o pipefail"},{"line_number":5,"context_line":""},{"line_number":6,"context_line":"function install_mariadb {"},{"line_number":7,"context_line":"    if [[ $(which dnf) -eq 0 ]]; then"},{"line_number":8,"context_line":"        sudo dnf install -y mariadb"},{"line_number":9,"context_line":"    else"},{"line_number":10,"context_line":"        sudo apt install -y mariadb-client"}],"source_content_type":"text/x-sh","patch_set":1,"id":"bbc99ee7_e32a4c02","line":7,"range":{"start_line":7,"start_character":0,"end_line":7,"end_character":37},"updated":"2026-03-02 13:26:44.000000000","message":"In AUFN we have switched to using:\n\nif command -v dnf \u003e/dev/null 2\u003e\u00261; then","commit_id":"6aa9d1c39cc36aded2d280db28ab9f997794f179"},{"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":"d4e143d8d31bf8e5825ba1a108594dcaff1db288","unresolved":false,"context_lines":[{"line_number":4,"context_line":"set -o pipefail"},{"line_number":5,"context_line":""},{"line_number":6,"context_line":"function install_mariadb {"},{"line_number":7,"context_line":"    if [[ $(which dnf) -eq 0 ]]; then"},{"line_number":8,"context_line":"        sudo dnf install -y mariadb"},{"line_number":9,"context_line":"    else"},{"line_number":10,"context_line":"        sudo apt install -y mariadb-client"}],"source_content_type":"text/x-sh","patch_set":1,"id":"6ac4f85a_0018e54e","line":7,"range":{"start_line":7,"start_character":0,"end_line":7,"end_character":37},"in_reply_to":"bbc99ee7_e32a4c02","updated":"2026-03-02 14:11:10.000000000","message":"Thanks for the suggestion","commit_id":"6aa9d1c39cc36aded2d280db28ab9f997794f179"}]}