)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":36114,"name":"Adam Stackhouse","display_name":"adstak","email":"adstak@gmail.com","username":"adstak","status":"Godaddy"},"change_message_id":"f66f433f0c48842d3adade0f183643951bf0d062","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"cd0fbc4d_e56de926","updated":"2023-06-21 09:55:24.000000000","message":"I do apologise as I am new to this as you can tell. I have created a bug report here --\u003e https://bugs.launchpad.net/kolla-ansible/+bug/2024554 and linked this patch.\n\nNot sure how to link your self to the bug report","commit_id":"3f12b85b18931366c226f7dd4f4a42be828e2a8e"},{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"67d9e84ccf796980e4783548be973526ada99d19","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"9cb9d364_c21002db","updated":"2023-06-21 09:18:36.000000000","message":"You might want to raise a bug and link it in the commit message, if you\u0027d want that backported","commit_id":"3f12b85b18931366c226f7dd4f4a42be828e2a8e"},{"author":{"_account_id":14200,"name":"Maksim Malchuk","email":"maksim.malchuk@gmail.com","username":"mmalchuk"},"change_message_id":"fe72a1b96b2fb5393ae01a505de3563e018dc16d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"7cb807c2_b134e9de","updated":"2023-06-21 09:22:52.000000000","message":"how it can be used? why there need to change standard port?","commit_id":"3f12b85b18931366c226f7dd4f4a42be828e2a8e"},{"author":{"_account_id":36114,"name":"Adam Stackhouse","display_name":"adstak","email":"adstak@gmail.com","username":"adstak","status":"Godaddy"},"change_message_id":"f66f433f0c48842d3adade0f183643951bf0d062","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"b323893a_285a39aa","in_reply_to":"213e5e90_5293b147","updated":"2023-06-21 09:55:24.000000000","message":"I do apologise as I am new to this as you can tell","commit_id":"3f12b85b18931366c226f7dd4f4a42be828e2a8e"},{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"1a6da845f482ec8785cba18823ec82af8e414f51","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"b4b9ce3f_d3a3d939","in_reply_to":"39dfb97d_a7980dc9","updated":"2023-06-21 10:04:12.000000000","message":"And put an empty line between commit subject and closes-bug","commit_id":"3f12b85b18931366c226f7dd4f4a42be828e2a8e"},{"author":{"_account_id":36114,"name":"Adam Stackhouse","display_name":"adstak","email":"adstak@gmail.com","username":"adstak","status":"Godaddy"},"change_message_id":"6f03768f7bfb7dfc9e9ca15089aa8bd3d725599b","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"5dc13070_570f54d3","in_reply_to":"48d560c2_597af628","updated":"2023-06-21 10:01:57.000000000","message":"Thanks. I have included to tag for the bug report","commit_id":"3f12b85b18931366c226f7dd4f4a42be828e2a8e"},{"author":{"_account_id":36114,"name":"Adam Stackhouse","display_name":"adstak","email":"adstak@gmail.com","username":"adstak","status":"Godaddy"},"change_message_id":"4786c8c51432de46c9c52a14a2a51940915dae44","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"ff07e665_ce3a2df0","in_reply_to":"4ed2f0e3_72220e23","updated":"2023-06-21 10:05:59.000000000","message":"Yeah looked at an example someone else had done when I submitted a bug the other day","commit_id":"3f12b85b18931366c226f7dd4f4a42be828e2a8e"},{"author":{"_account_id":14200,"name":"Maksim Malchuk","email":"maksim.malchuk@gmail.com","username":"mmalchuk"},"change_message_id":"834ec66777b3da1d3771ddf3a470d7a0ef272f91","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"213e5e90_5293b147","in_reply_to":"56c24f5c_4412b31d","updated":"2023-06-21 09:43:59.000000000","message":"nice. then address Michal comment and add the link to the bugreport if this is an issue not feature.","commit_id":"3f12b85b18931366c226f7dd4f4a42be828e2a8e"},{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"580e4e75c279e2fbdae02cfdb8fd0e718b3b2576","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"39dfb97d_a7980dc9","in_reply_to":"5dc13070_570f54d3","updated":"2023-06-21 10:03:21.000000000","message":"Unfortunately it can\u0027t be the first line - because then it gets to be the commit subject - swap the lines please (and remove the bullet - *)","commit_id":"3f12b85b18931366c226f7dd4f4a42be828e2a8e"},{"author":{"_account_id":36114,"name":"Adam Stackhouse","display_name":"adstak","email":"adstak@gmail.com","username":"adstak","status":"Godaddy"},"change_message_id":"9e8316c1f438c76e840a4f453008fb943b99238a","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"56c24f5c_4412b31d","in_reply_to":"7cb807c2_b134e9de","updated":"2023-06-21 09:38:19.000000000","message":"So in my setup it was needed to change as i wanted proxysql to listen on 3306 as the frontend so moved mariadb to 3307. I wanted this as proxysql was more existing to cache queries to increase performance.\n\nThis change should also not change a standard setup as shell out command will just use 3306 which is the default similar to the \"Wait for MariaDB service port liveness\" play above this one in the same file.\n\nJust gives people options in my opinion without breaking defaults","commit_id":"3f12b85b18931366c226f7dd4f4a42be828e2a8e"},{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"0f313fa27f3008ad9efdd430a5fa4042ab53f50e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"f0f32df3_fff564ad","in_reply_to":"b323893a_285a39aa","updated":"2023-06-21 10:04:47.000000000","message":"Done","commit_id":"3f12b85b18931366c226f7dd4f4a42be828e2a8e"},{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"a4064454525696b0ac126aa49c0d6aba14e369a6","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"4ed2f0e3_72220e23","in_reply_to":"b4b9ce3f_d3a3d939","updated":"2023-06-21 10:04:31.000000000","message":"Looks good now :)","commit_id":"3f12b85b18931366c226f7dd4f4a42be828e2a8e"},{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"0645a164fc109672c0815fbb20ad10d82528c9c4","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"48d560c2_597af628","in_reply_to":"cd0fbc4d_e56de926","updated":"2023-06-21 09:58:11.000000000","message":"https://wiki.openstack.org/wiki/GitCommitMessages#Including_external_references","commit_id":"3f12b85b18931366c226f7dd4f4a42be828e2a8e"},{"author":{"_account_id":36114,"name":"Adam Stackhouse","display_name":"adstak","email":"adstak@gmail.com","username":"adstak","status":"Godaddy"},"change_message_id":"7c311df0317e09db621a2763d5b3b3340c043b26","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"b527bcb3_02604a59","in_reply_to":"ff07e665_ce3a2df0","updated":"2023-06-21 10:12:18.000000000","message":"Done","commit_id":"3f12b85b18931366c226f7dd4f4a42be828e2a8e"},{"author":{"_account_id":27339,"name":"Michal Arbet","email":"michal.arbet@ultimum.io","username":"michalarbet"},"change_message_id":"3ce53b8283800042309f37758fb095ef5cd5b120","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"e75c9cd6_8278ee96","updated":"2023-06-21 10:04:33.000000000","message":"This is good idea as we have database_port which should be frontend of mysql, and mariadb port which should be backend.\n\nWhat i don\u0027t understand is, why you need to change default port ? \n\nProxysql is running VIP:3306 and mariadb IP:3306\n\ncheck my ss output\n\nroot@controller0:~# ss -tulpn | grep 3306\ntcp   LISTEN 0      900               192.168.205.10:3306       0.0.0.0:*    users:((\"mariadbd\",pid\u003d4043,fd\u003d69))                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                   \ntcp   LISTEN 0      1024             192.168.205.254:3306       0.0.0.0:*    users:((\"proxysql\",pid\u003d2388,fd\u003d39))                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                   \ntcp   LISTEN 0      1024             192.168.205.254:3306       0.0.0.0:*    users:((\"proxysql\",pid\u003d2388,fd\u003d38))                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                   \ntcp   LISTEN 0      1024             192.168.205.254:3306       0.0.0.0:*    users:((\"proxysql\",pid\u003d2388,fd\u003d37))                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                   \ntcp   LISTEN 0      1024             192.168.205.254:3306       0.0.0.0:*    users:((\"proxysql\",pid\u003d2388,fd\u003d36))                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                   \ntcp   LISTEN 0      1024             192.168.205.254:3306       0.0.0.0:*    users:((\"proxysql\",pid\u003d2388,fd\u003d34))","commit_id":"6d01007f1c51179cdcdc47e8ef50749e37a1f7e5"},{"author":{"_account_id":36114,"name":"Adam Stackhouse","display_name":"adstak","email":"adstak@gmail.com","username":"adstak","status":"Godaddy"},"change_message_id":"41cf68dda1c6c18c4e093bd9aa90acc057532e87","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":2,"id":"fd79f506_baca8477","in_reply_to":"e75c9cd6_8278ee96","updated":"2023-06-21 10:10:55.000000000","message":"So in my setup we have a network based load balancer so the VIP of proxysql is actually setup to be an external IP that doesn\u0027t exist on the host. However I also have proxysql setup to listen on IP:3306 so that locally containers can talk to their own proxysql locally rather than go through the network LB. Reason for this is we are currently testing performance.\n\nIn short we have the following:\n\nproxysql --\u003e VIP:3306 and IP:3306\nMariadb --\u003e IP:3307\n\nAll containers that need the db connect to either of the 3306 endpoints to get the proxysql and mariadb of course has to listen on something else as 3306 is taken. \n\nHope that makes sense","commit_id":"6d01007f1c51179cdcdc47e8ef50749e37a1f7e5"},{"author":{"_account_id":36114,"name":"Adam Stackhouse","display_name":"adstak","email":"adstak@gmail.com","username":"adstak","status":"Godaddy"},"change_message_id":"c958804eaf0b1b6469a6048526e4daa2e4180081","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"813f5412_57129685","in_reply_to":"fd79f506_baca8477","updated":"2023-06-21 15:31:11.000000000","message":"Done","commit_id":"6d01007f1c51179cdcdc47e8ef50749e37a1f7e5"},{"author":{"_account_id":36114,"name":"Adam Stackhouse","display_name":"adstak","email":"adstak@gmail.com","username":"adstak","status":"Godaddy"},"change_message_id":"48b97a5e9c35bbb6ed4ce131a8481fe18ecaa0f5","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":5,"id":"ae97b4ab_38b786df","updated":"2023-06-21 10:26:15.000000000","message":"If I wanted to get this changed into zed and other stable versions, Is there a process to get these backported or do I just branch from those branches and submit the same change?","commit_id":"2d1d869e60636af679525bb21609f48f931742d7"},{"author":{"_account_id":36114,"name":"Adam Stackhouse","display_name":"adstak","email":"adstak@gmail.com","username":"adstak","status":"Godaddy"},"change_message_id":"f92263ba275825f14f84b7e3f03241f64809f663","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"876a6520_b3f421e8","in_reply_to":"209ddbd8_468670d4","updated":"2023-06-23 07:45:09.000000000","message":"Done","commit_id":"2d1d869e60636af679525bb21609f48f931742d7"},{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"24b69b599dfef34deb26471f769f73c3b1deb0eb","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"36f26763_f922339b","in_reply_to":"ae97b4ab_38b786df","updated":"2023-06-21 10:27:35.000000000","message":"Once this one is merged - you can backport (cherry pick) to stable branches","commit_id":"2d1d869e60636af679525bb21609f48f931742d7"},{"author":{"_account_id":14200,"name":"Maksim Malchuk","email":"maksim.malchuk@gmail.com","username":"mmalchuk"},"change_message_id":"b692d5f48f7499f4f1f7277e026c8e98075004b1","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":5,"id":"ddcb8a3d_c455a297","in_reply_to":"ae97b4ab_38b786df","updated":"2023-06-21 10:29:45.000000000","message":"first this change should be merged in the master branch, then you should cherry-pick with -x option this change to the stable branches. I\u0027ve already created the series of stable versions in the bugtracker, and here we can help you with backports later.","commit_id":"2d1d869e60636af679525bb21609f48f931742d7"},{"author":{"_account_id":36114,"name":"Adam Stackhouse","display_name":"adstak","email":"adstak@gmail.com","username":"adstak","status":"Godaddy"},"change_message_id":"9ae8ab10b572dfc3662cee8a8bf3a8ecef1e1e7d","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":5,"id":"e02acdf8_de066dc2","in_reply_to":"ddcb8a3d_c455a297","updated":"2023-06-21 10:31:36.000000000","message":"Yeah sounds good. Will work on release notes similar to my other patch just reading through that doc","commit_id":"2d1d869e60636af679525bb21609f48f931742d7"},{"author":{"_account_id":14200,"name":"Maksim Malchuk","email":"maksim.malchuk@gmail.com","username":"mmalchuk"},"change_message_id":"739fe205efd4bb656d6fa3d913cb550ea4172347","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":5,"id":"52a2c907_5c238199","in_reply_to":"e02acdf8_de066dc2","updated":"2023-06-21 10:34:25.000000000","message":"btw, also, you can join to the irc channel #openstack-kolla on OFTC network and ask help online.","commit_id":"2d1d869e60636af679525bb21609f48f931742d7"},{"author":{"_account_id":27339,"name":"Michal Arbet","email":"michal.arbet@ultimum.io","username":"michalarbet"},"change_message_id":"4fd3479f536aed2f33d02167410614bdfedbb671","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":5,"id":"209ddbd8_468670d4","in_reply_to":"e02acdf8_de066dc2","updated":"2023-06-21 10:34:30.000000000","message":"btw, when this change will be merged in a master, you can just click cherry pick and select stable branch you want in right top corner of this page.","commit_id":"2d1d869e60636af679525bb21609f48f931742d7"},{"author":{"_account_id":36114,"name":"Adam Stackhouse","display_name":"adstak","email":"adstak@gmail.com","username":"adstak","status":"Godaddy"},"change_message_id":"3cecbf6c342140a4df941d29e7e6a6f81f80aa32","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":8,"id":"83414ea4_902e515f","updated":"2023-06-21 14:18:26.000000000","message":"Releases notes have been generated and added","commit_id":"72d0feaa66a1fe461d5a33d71ffb39d10c016a99"},{"author":{"_account_id":32553,"name":"Sven Kieske","email":"sven_oss@posteo.de","username":"skieske"},"change_message_id":"e374be39272fb6ba7df6fda6c642b7f1af83b8b5","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":8,"id":"9fae839b_1897e3ff","updated":"2023-06-21 14:20:47.000000000","message":"shouldn\u0027t a default value be declared for the port?","commit_id":"72d0feaa66a1fe461d5a33d71ffb39d10c016a99"},{"author":{"_account_id":14200,"name":"Maksim Malchuk","email":"maksim.malchuk@gmail.com","username":"mmalchuk"},"change_message_id":"9f9209812be73687e03dcd415bb638efcaefe822","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"458541ea_a0cc5591","updated":"2023-06-21 14:23:18.000000000","message":"you can check the CI jobs before all of them finished here: https://zuul.openstack.org/status#886581","commit_id":"72d0feaa66a1fe461d5a33d71ffb39d10c016a99"},{"author":{"_account_id":36114,"name":"Adam Stackhouse","display_name":"adstak","email":"adstak@gmail.com","username":"adstak","status":"Godaddy"},"change_message_id":"c958804eaf0b1b6469a6048526e4daa2e4180081","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"1dffffb2_12fc2b63","in_reply_to":"0fd50aa9_7a1b5c72","updated":"2023-06-21 15:31:11.000000000","message":"Done","commit_id":"72d0feaa66a1fe461d5a33d71ffb39d10c016a99"},{"author":{"_account_id":14200,"name":"Maksim Malchuk","email":"maksim.malchuk@gmail.com","username":"mmalchuk"},"change_message_id":"cce8adcacce85b1ac359f974b0fb08c6cba61dda","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":8,"id":"d1fb67c2_9326803f","in_reply_to":"83414ea4_902e515f","updated":"2023-06-21 14:21:38.000000000","message":"it should fail with long lines","commit_id":"72d0feaa66a1fe461d5a33d71ffb39d10c016a99"},{"author":{"_account_id":14200,"name":"Maksim Malchuk","email":"maksim.malchuk@gmail.com","username":"mmalchuk"},"change_message_id":"2e8f14397b0a285f319c36d1716c723c7dbd6dc3","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"11a2618b_3f30a890","in_reply_to":"9fae839b_1897e3ff","updated":"2023-06-21 14:25:31.000000000","message":"\u003e shouldn\u0027t a default value be declared for the port?\nit declared in the ansible/group_vars/all.yml\nmariadb_port: \"{{ database_port }}\"","commit_id":"72d0feaa66a1fe461d5a33d71ffb39d10c016a99"},{"author":{"_account_id":36114,"name":"Adam Stackhouse","display_name":"adstak","email":"adstak@gmail.com","username":"adstak","status":"Godaddy"},"change_message_id":"db90d9894c6b212ed3f9e7951b5fef94c0b78f81","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":8,"id":"0fd50aa9_7a1b5c72","in_reply_to":"9fae839b_1897e3ff","updated":"2023-06-21 14:32:09.000000000","message":"default value for mariadb_port should already be set to point to database_port unless overridden in globals.yaml.\n\nhttps://opendev.org/openstack/kolla-ansible/src/commit/45ec9c020b405e329e1af50404d5e888c7ac5c09/ansible/group_vars/all.yml#L388","commit_id":"72d0feaa66a1fe461d5a33d71ffb39d10c016a99"},{"author":{"_account_id":36114,"name":"Adam Stackhouse","display_name":"adstak","email":"adstak@gmail.com","username":"adstak","status":"Godaddy"},"change_message_id":"c958804eaf0b1b6469a6048526e4daa2e4180081","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"50f67058_12d91bbc","in_reply_to":"d1fb67c2_9326803f","updated":"2023-06-21 15:31:11.000000000","message":"Done","commit_id":"72d0feaa66a1fe461d5a33d71ffb39d10c016a99"},{"author":{"_account_id":27339,"name":"Michal Arbet","email":"michal.arbet@ultimum.io","username":"michalarbet"},"change_message_id":"bf54599a23b4329a11c4c6b52cae161f479536e9","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"06ba4d05_3b1cec6f","updated":"2023-06-26 09:41:27.000000000","message":"Please, check telegraf template, there is also database_port mentioned but telegraf is gathering metrics for mariadb node, so i think replacement should be also there..\n\nansible/roles/telegraf/templates/telegraf.conf.j2-{% if inventory_hostname in groups[\u0027mariadb\u0027] and (enable_mariadb or enable_external_mariadb_load_balancer) | bool %}\nansible/roles/telegraf/templates/telegraf.conf.j2-[[inputs.mysql]]\nansible/roles/telegraf/templates/telegraf.conf.j2:  servers \u003d [\"{{ database_user }}:{{ database_password }}@{{ mariadb_proto }}({{ api_interface_address | put_address_in_context(\u0027url\u0027) }}:{{ database_port }})/\"]\nansible/roles/telegraf/templates/telegraf.conf.j2-  perf_events_statements_digest_text_limit  \u003d 120\n\nWhat about prechecks ? \n\nansible/roles/mariadb/tasks/precheck.yml-  register: container_facts\nansible/roles/mariadb/tasks/precheck.yml-\nansible/roles/mariadb/tasks/precheck.yml-- name: Checking free port for MariaDB\nansible/roles/mariadb/tasks/precheck.yml-  wait_for:\nansible/roles/mariadb/tasks/precheck.yml-    host: \"{{ api_interface_address }}\"\nansible/roles/mariadb/tasks/precheck.yml:    port: \"{{ database_port }}\"\nansible/roles/mariadb/tasks/precheck.yml-    connect_timeout: 1\nansible/roles/mariadb/tasks/precheck.yml-    timeout: 1\nansible/roles/mariadb/tasks/precheck.yml-    state: stopped\nansible/roles/mariadb/tasks/precheck.yml-  when:\nansible/roles/mariadb/tasks/precheck.yml-    - container_facts[\u0027mariadb\u0027] is not defined","commit_id":"76d67dbd710d8cfd7e9fed13fd5d4e5b257871e0"},{"author":{"_account_id":32553,"name":"Sven Kieske","email":"sven_oss@posteo.de","username":"skieske"},"change_message_id":"ed2758738326c75a36579cd806660a4322311bdb","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":9,"id":"3dc0eeaf_11c66cc8","in_reply_to":"04786119_952dfbe3","updated":"2023-06-26 14:53:12.000000000","message":"so now we check the mariadb port with patchset 10, but we don\u0027t check the VIP database_port anymore. was it a mistake that this was checked and isn\u0027t needed anymore?","commit_id":"76d67dbd710d8cfd7e9fed13fd5d4e5b257871e0"},{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"c0a284adabe5c4c2cb5ff572a9576e5b0c6d4b12","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"ffeb7918_59c83365","in_reply_to":"06ba4d05_3b1cec6f","updated":"2023-06-26 09:46:00.000000000","message":"So what\u0027s the difference between database_port and mariadb_port?","commit_id":"76d67dbd710d8cfd7e9fed13fd5d4e5b257871e0"},{"author":{"_account_id":27339,"name":"Michal Arbet","email":"michal.arbet@ultimum.io","username":"michalarbet"},"change_message_id":"39cbfdc009c4c69608cf5396f01fe68ecf37c5ce","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"9cdbb55e_605e094c","in_reply_to":"3dc0eeaf_11c66cc8","updated":"2023-06-26 14:58:04.000000000","message":"really ? so check ansible/roles/loadbalancer/tasks/precheck.yml again :) and you will get it ;-)","commit_id":"76d67dbd710d8cfd7e9fed13fd5d4e5b257871e0"},{"author":{"_account_id":36114,"name":"Adam Stackhouse","display_name":"adstak","email":"adstak@gmail.com","username":"adstak","status":"Godaddy"},"change_message_id":"71d1a1f056d9bec0f648c2ac003f391c21e082c5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"04786119_952dfbe3","in_reply_to":"49e59176_2e3d1707","updated":"2023-06-26 13:31:19.000000000","message":"Uploaded precheck change","commit_id":"76d67dbd710d8cfd7e9fed13fd5d4e5b257871e0"},{"author":{"_account_id":27339,"name":"Michal Arbet","email":"michal.arbet@ultimum.io","username":"michalarbet"},"change_message_id":"bb6d7bd409b39ae151142433368a6e58fd4d07cf","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":9,"id":"fc090631_2ca1b047","in_reply_to":"8d5b3235_bf57bb6b","updated":"2023-06-27 15:36:24.000000000","message":"no, it was reply to Sven, there has to be database_port as it is checking VIP:database_port which is frontend. revert it back.","commit_id":"76d67dbd710d8cfd7e9fed13fd5d4e5b257871e0"},{"author":{"_account_id":36114,"name":"Adam Stackhouse","display_name":"adstak","email":"adstak@gmail.com","username":"adstak","status":"Godaddy"},"change_message_id":"054194c02122a1a55a5b44f9097214929710d8d3","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":9,"id":"8d5b3235_bf57bb6b","in_reply_to":"9cdbb55e_605e094c","updated":"2023-06-27 12:27:12.000000000","message":"Oh yeah my bad. Updated","commit_id":"76d67dbd710d8cfd7e9fed13fd5d4e5b257871e0"},{"author":{"_account_id":36114,"name":"Adam Stackhouse","display_name":"adstak","email":"adstak@gmail.com","username":"adstak","status":"Godaddy"},"change_message_id":"8769d5819ed7942efdf70006768509fee2a5e3a4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"b5c9e5f7_7851de1d","in_reply_to":"a0e6e5de_23ef2eeb","updated":"2023-06-28 09:42:47.000000000","message":"Done","commit_id":"76d67dbd710d8cfd7e9fed13fd5d4e5b257871e0"},{"author":{"_account_id":36114,"name":"Adam Stackhouse","display_name":"adstak","email":"adstak@gmail.com","username":"adstak","status":"Godaddy"},"change_message_id":"992588be5b2fad447dfb1a440572f52bddd53396","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":9,"id":"49e59176_2e3d1707","in_reply_to":"b2fc0a33_81938f4f","updated":"2023-06-26 13:24:28.000000000","message":"Will take a look at the pre-checks shortly, must have missed this comment","commit_id":"76d67dbd710d8cfd7e9fed13fd5d4e5b257871e0"},{"author":{"_account_id":36114,"name":"Adam Stackhouse","display_name":"adstak","email":"adstak@gmail.com","username":"adstak","status":"Godaddy"},"change_message_id":"85a86185e848c1dd75b95e7a5fc42172710e8b39","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":9,"id":"a0e6e5de_23ef2eeb","in_reply_to":"fc090631_2ca1b047","updated":"2023-06-28 07:32:04.000000000","message":"my apologies reverted","commit_id":"76d67dbd710d8cfd7e9fed13fd5d4e5b257871e0"},{"author":{"_account_id":27339,"name":"Michal Arbet","email":"michal.arbet@ultimum.io","username":"michalarbet"},"change_message_id":"9535c6e838b82c68c7cd0c1e27427920bc234a56","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"b2fc0a33_81938f4f","in_reply_to":"ffeb7918_59c83365","updated":"2023-06-26 10:38:47.000000000","message":"VIP:database_port , IP:mariadb_port , so you can modify your globals or hostvars and deploy mariadb nodes on different port ..check comments above...","commit_id":"76d67dbd710d8cfd7e9fed13fd5d4e5b257871e0"},{"author":{"_account_id":36114,"name":"Adam Stackhouse","display_name":"adstak","email":"adstak@gmail.com","username":"adstak","status":"Godaddy"},"change_message_id":"9303eada0d678de28023f35057d4aebca04563da","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":13,"id":"597ba483_ccef5bfa","updated":"2023-08-30 15:01:31.000000000","message":"recheck","commit_id":"842e758221a63e4bc5adf31bec061ea7aa22b3cb"},{"author":{"_account_id":36114,"name":"Adam Stackhouse","display_name":"adstak","email":"adstak@gmail.com","username":"adstak","status":"Godaddy"},"change_message_id":"2b06a5e2e17ebd25fc8fcf99b1e61a9edeb6bc6f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":13,"id":"78427aa6_66ba6392","updated":"2023-08-23 09:55:01.000000000","message":"recheck","commit_id":"842e758221a63e4bc5adf31bec061ea7aa22b3cb"}],"ansible/roles/mariadb/tasks/restart_services.yml":[{"author":{"_account_id":14200,"name":"Maksim Malchuk","email":"maksim.malchuk@gmail.com","username":"mmalchuk"},"change_message_id":"1cf78b17bfc713a7dbb05ad49a40e6adecb06656","unresolved":true,"context_lines":[{"line_number":31,"context_line":"  become: true"},{"line_number":32,"context_line":"  command: \u003e-"},{"line_number":33,"context_line":"    {{ kolla_container_engine }} exec {{ mariadb_service.container_name }}"},{"line_number":34,"context_line":"    mysql -uroot -p{{ database_password }} -P{{ mariadb_port }}"},{"line_number":35,"context_line":"    --silent --skip-column-names"},{"line_number":36,"context_line":"    -e \u0027SHOW STATUS LIKE \"wsrep_local_state_comment\"\u0027"},{"line_number":37,"context_line":"  changed_when: false"}],"source_content_type":"text/x-yaml","patch_set":12,"id":"d2050ae0_46e0cbb7","line":34,"range":{"start_line":34,"start_character":43,"end_line":34,"end_character":45},"updated":"2023-08-14 09:22:54.000000000","message":"space missed. only there must be no space between --password\u003d or -p and the password following it.","commit_id":"26056f68d1decbf195e350374af4043a9a7ae55c"},{"author":{"_account_id":14200,"name":"Maksim Malchuk","email":"maksim.malchuk@gmail.com","username":"mmalchuk"},"change_message_id":"47ab0b702a6c8d3779899ae419440626aa88542b","unresolved":false,"context_lines":[{"line_number":31,"context_line":"  become: true"},{"line_number":32,"context_line":"  command: \u003e-"},{"line_number":33,"context_line":"    {{ kolla_container_engine }} exec {{ mariadb_service.container_name }}"},{"line_number":34,"context_line":"    mysql -uroot -p{{ database_password }} -P{{ mariadb_port }}"},{"line_number":35,"context_line":"    --silent --skip-column-names"},{"line_number":36,"context_line":"    -e \u0027SHOW STATUS LIKE \"wsrep_local_state_comment\"\u0027"},{"line_number":37,"context_line":"  changed_when: false"}],"source_content_type":"text/x-yaml","patch_set":12,"id":"8b06cb82_4dde219c","line":34,"range":{"start_line":34,"start_character":43,"end_line":34,"end_character":45},"in_reply_to":"0abb96ee_4de2959b","updated":"2023-08-22 11:09:36.000000000","message":"thanks","commit_id":"26056f68d1decbf195e350374af4043a9a7ae55c"},{"author":{"_account_id":36114,"name":"Adam Stackhouse","display_name":"adstak","email":"adstak@gmail.com","username":"adstak","status":"Godaddy"},"change_message_id":"6220e90d9f71d279311ba52220870b6cd9c507d8","unresolved":false,"context_lines":[{"line_number":31,"context_line":"  become: true"},{"line_number":32,"context_line":"  command: \u003e-"},{"line_number":33,"context_line":"    {{ kolla_container_engine }} exec {{ mariadb_service.container_name }}"},{"line_number":34,"context_line":"    mysql -uroot -p{{ database_password }} -P{{ mariadb_port }}"},{"line_number":35,"context_line":"    --silent --skip-column-names"},{"line_number":36,"context_line":"    -e \u0027SHOW STATUS LIKE \"wsrep_local_state_comment\"\u0027"},{"line_number":37,"context_line":"  changed_when: false"}],"source_content_type":"text/x-yaml","patch_set":12,"id":"0abb96ee_4de2959b","line":34,"range":{"start_line":34,"start_character":43,"end_line":34,"end_character":45},"in_reply_to":"2890ac1a_47d11a1b","updated":"2023-08-22 11:02:32.000000000","message":"Ah I understand now. Sure updated it with a space after the port argument, hopefully thats all good now 😊","commit_id":"26056f68d1decbf195e350374af4043a9a7ae55c"},{"author":{"_account_id":14200,"name":"Maksim Malchuk","email":"maksim.malchuk@gmail.com","username":"mmalchuk"},"change_message_id":"1bf4d661ea5067766664576b2f6320d7a4ce0012","unresolved":true,"context_lines":[{"line_number":31,"context_line":"  become: true"},{"line_number":32,"context_line":"  command: \u003e-"},{"line_number":33,"context_line":"    {{ kolla_container_engine }} exec {{ mariadb_service.container_name }}"},{"line_number":34,"context_line":"    mysql -uroot -p{{ database_password }} -P{{ mariadb_port }}"},{"line_number":35,"context_line":"    --silent --skip-column-names"},{"line_number":36,"context_line":"    -e \u0027SHOW STATUS LIKE \"wsrep_local_state_comment\"\u0027"},{"line_number":37,"context_line":"  changed_when: false"}],"source_content_type":"text/x-yaml","patch_set":12,"id":"fef954cb_b3bd4988","line":34,"range":{"start_line":34,"start_character":43,"end_line":34,"end_character":45},"in_reply_to":"5c653d64_c6cb0806","updated":"2023-08-21 09:57:06.000000000","message":"yep","commit_id":"26056f68d1decbf195e350374af4043a9a7ae55c"},{"author":{"_account_id":27339,"name":"Michal Arbet","email":"michal.arbet@ultimum.io","username":"michalarbet"},"change_message_id":"60b3a7c4446c5218d56630aa5b6afc0310990e21","unresolved":true,"context_lines":[{"line_number":31,"context_line":"  become: true"},{"line_number":32,"context_line":"  command: \u003e-"},{"line_number":33,"context_line":"    {{ kolla_container_engine }} exec {{ mariadb_service.container_name }}"},{"line_number":34,"context_line":"    mysql -uroot -p{{ database_password }} -P{{ mariadb_port }}"},{"line_number":35,"context_line":"    --silent --skip-column-names"},{"line_number":36,"context_line":"    -e \u0027SHOW STATUS LIKE \"wsrep_local_state_comment\"\u0027"},{"line_number":37,"context_line":"  changed_when: false"}],"source_content_type":"text/x-yaml","patch_set":12,"id":"5c653d64_c6cb0806","line":34,"range":{"start_line":34,"start_character":43,"end_line":34,"end_character":45},"in_reply_to":"6132d1ba_536658da","updated":"2023-08-21 07:57:41.000000000","message":"even if it is working, port parameter don\u0027t need to have space before the port number while password needs to have ....\n\nSo, space before port number increase readability....","commit_id":"26056f68d1decbf195e350374af4043a9a7ae55c"},{"author":{"_account_id":36114,"name":"Adam Stackhouse","display_name":"adstak","email":"adstak@gmail.com","username":"adstak","status":"Godaddy"},"change_message_id":"ed4f50d0e4d89cecb0ba1940e9dd6fd29deac779","unresolved":true,"context_lines":[{"line_number":31,"context_line":"  become: true"},{"line_number":32,"context_line":"  command: \u003e-"},{"line_number":33,"context_line":"    {{ kolla_container_engine }} exec {{ mariadb_service.container_name }}"},{"line_number":34,"context_line":"    mysql -uroot -p{{ database_password }} -P{{ mariadb_port }}"},{"line_number":35,"context_line":"    --silent --skip-column-names"},{"line_number":36,"context_line":"    -e \u0027SHOW STATUS LIKE \"wsrep_local_state_comment\"\u0027"},{"line_number":37,"context_line":"  changed_when: false"}],"source_content_type":"text/x-yaml","patch_set":12,"id":"6132d1ba_536658da","line":34,"range":{"start_line":34,"start_character":43,"end_line":34,"end_character":45},"in_reply_to":"d2050ae0_46e0cbb7","updated":"2023-08-21 07:53:57.000000000","message":"I don\u0027t understand maybe I\u0027m missing something here. There is no space after the -p for password or -P for mariadb_port. Running this on the cli look to work as expected for me connecting to a DB. \n\nagain probs missing something so feel free to further explain","commit_id":"26056f68d1decbf195e350374af4043a9a7ae55c"},{"author":{"_account_id":27339,"name":"Michal Arbet","email":"michal.arbet@ultimum.io","username":"michalarbet"},"change_message_id":"0c55d1d3b11add70ae446499998c42deb2d701ee","unresolved":true,"context_lines":[{"line_number":31,"context_line":"  become: true"},{"line_number":32,"context_line":"  command: \u003e-"},{"line_number":33,"context_line":"    {{ kolla_container_engine }} exec {{ mariadb_service.container_name }}"},{"line_number":34,"context_line":"    mysql -uroot -p{{ database_password }} -P{{ mariadb_port }}"},{"line_number":35,"context_line":"    --silent --skip-column-names"},{"line_number":36,"context_line":"    -e \u0027SHOW STATUS LIKE \"wsrep_local_state_comment\"\u0027"},{"line_number":37,"context_line":"  changed_when: false"}],"source_content_type":"text/x-yaml","patch_set":12,"id":"2890ac1a_47d11a1b","line":34,"range":{"start_line":34,"start_character":43,"end_line":34,"end_character":45},"in_reply_to":"fef954cb_b3bd4988","updated":"2023-08-21 10:00:14.000000000","message":"sorry, I meant -pPASS works but -p PASS don\u0027t work ... -P port -Pport work both ... but we want -pPASS -P port","commit_id":"26056f68d1decbf195e350374af4043a9a7ae55c"}]}
