)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":32755,"name":"Christian Rohmann","email":"christian.rohmann@inovex.de","username":"frittentheke"},"change_message_id":"4ae1ad29c98c168f3882372fd4abadf950db0cd4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"baf5fb88_7ec5dfe8","updated":"2022-08-26 15:30:40.000000000","message":"This depends on / makes use of https://review.opendev.org/c/openstack/openstack-ansible-galera_server/+/854787 and enables proxy protocol for galera backend servers resulting in the actual client (IP) being known and shown by MariaDB.\n\n","commit_id":"539d4df55282137e14f36c715593537766993bd6"},{"author":{"_account_id":32755,"name":"Christian Rohmann","email":"christian.rohmann@inovex.de","username":"frittentheke"},"change_message_id":"d38ec8e4dd949d9d6ac85373c6f5be58545527dc","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"70cef86e_28467084","updated":"2022-08-29 13:15:44.000000000","message":"Dimity could you kindly help me with the CI issue here?\nI am not really sure if CI is broken in general or if this is my change ...","commit_id":"3b09cd8a23daa1728472bb0cf1f5800de1c965ea"},{"author":{"_account_id":28619,"name":"Dmitriy Rabotyagov","email":"noonedeadpunk@gmail.com","username":"noonedeadpunk"},"change_message_id":"c636d93570924c1676b8c8a96cd66d92c01bedbc","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":3,"id":"9154dfcd_aae6f526","in_reply_to":"1592eb4e_0d6135c6","updated":"2022-08-29 13:25:28.000000000","message":"Ah, you missed depends-on.","commit_id":"3b09cd8a23daa1728472bb0cf1f5800de1c965ea"},{"author":{"_account_id":28619,"name":"Dmitriy Rabotyagov","email":"noonedeadpunk@gmail.com","username":"noonedeadpunk"},"change_message_id":"70b7e4e1e8e311523b874545309efd2c1ca113cf","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":3,"id":"1592eb4e_0d6135c6","in_reply_to":"70cef86e_28467084","updated":"2022-08-29 13:24:43.000000000","message":"Yeah, but I would need to spawn a sandbox. But that looks like related to the patch. Also, to have that said, I don\u0027t see a config being set in my.cnf: https://zuul.opendev.org/t/openstack/build/394d41b89caf4c33a8cff86ca5525c32/log/logs/etc/openstack/aio1_galera_container-09eef188/mysql/mariadb.cnf.txt#27-28","commit_id":"3b09cd8a23daa1728472bb0cf1f5800de1c965ea"},{"author":{"_account_id":32755,"name":"Christian Rohmann","email":"christian.rohmann@inovex.de","username":"frittentheke"},"change_message_id":"407d6ac37a715e024e4ed01f6b583568b590e965","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"e920ab07_bf3cd17c","in_reply_to":"9154dfcd_aae6f526","updated":"2022-08-29 13:33:32.000000000","message":"Ack","commit_id":"3b09cd8a23daa1728472bb0cf1f5800de1c965ea"},{"author":{"_account_id":28619,"name":"Dmitriy Rabotyagov","email":"noonedeadpunk@gmail.com","username":"noonedeadpunk"},"change_message_id":"c6b5a4346c486e402a19d77b80e3b4bef47845b5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"00e42e35_320cd6d2","updated":"2022-08-29 13:37:44.000000000","message":"recheck dependent patch updated","commit_id":"d3bb90e0506b1812f06766fec874f93f80d4346e"}],"inventory/group_vars/galera_all.yml":[{"author":{"_account_id":28619,"name":"Dmitriy Rabotyagov","email":"noonedeadpunk@gmail.com","username":"noonedeadpunk"},"change_message_id":"a4f8851f14921e8b8a546947c80bd79976684038","unresolved":true,"context_lines":[{"line_number":40,"context_line":""},{"line_number":41,"context_line":"# Accept PROXY protocol information from the load balancers"},{"line_number":42,"context_line":"# See https://mariadb.com/kb/en/proxy-protocol-support for details"},{"line_number":43,"context_line":"galera_server_proxy_protocol_networks: \u003e-"},{"line_number":44,"context_line":"  {{"},{"line_number":45,"context_line":"    groups[\u0027haproxy\u0027]"},{"line_number":46,"context_line":"      | map(\u0027extract\u0027, hostvars, \u0027ansible_host\u0027)"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"14aec32e_892761d6","line":43,"updated":"2022-08-29 09:03:57.000000000","message":"I think you must add localhost here as well. As unix socket is used for connections as well in some cases","commit_id":"539d4df55282137e14f36c715593537766993bd6"},{"author":{"_account_id":32755,"name":"Christian Rohmann","email":"christian.rohmann@inovex.de","username":"frittentheke"},"change_message_id":"2e083889a839af48a7e9731f559c62c6ba20cb5f","unresolved":true,"context_lines":[{"line_number":40,"context_line":""},{"line_number":41,"context_line":"# Accept PROXY protocol information from the load balancers"},{"line_number":42,"context_line":"# See https://mariadb.com/kb/en/proxy-protocol-support for details"},{"line_number":43,"context_line":"galera_server_proxy_protocol_networks: \u003e-"},{"line_number":44,"context_line":"  {{"},{"line_number":45,"context_line":"    groups[\u0027haproxy\u0027]"},{"line_number":46,"context_line":"      | map(\u0027extract\u0027, hostvars, \u0027ansible_host\u0027)"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"d647bfc7_054da45d","line":43,"in_reply_to":"14aec32e_892761d6","updated":"2022-08-29 10:04:06.000000000","message":"I added \"localhost\" in patch set 2. \n\nFollowing the idea that the machines trusts local connections with providing proxy protocol data, I suppose one could add \"::1, 127.0.0.0/8\" as well. \n\nWhat do you think?","commit_id":"539d4df55282137e14f36c715593537766993bd6"},{"author":{"_account_id":32755,"name":"Christian Rohmann","email":"christian.rohmann@inovex.de","username":"frittentheke"},"change_message_id":"b25637681cf639bc005da51ab6e9fb5dafa00e05","unresolved":true,"context_lines":[{"line_number":40,"context_line":""},{"line_number":41,"context_line":"# Accept PROXY protocol information from the load balancers"},{"line_number":42,"context_line":"# See https://mariadb.com/kb/en/proxy-protocol-support for details"},{"line_number":43,"context_line":"galera_server_proxy_protocol_networks: \u003e-"},{"line_number":44,"context_line":"  {{"},{"line_number":45,"context_line":"    groups[\u0027haproxy\u0027]"},{"line_number":46,"context_line":"      | map(\u0027extract\u0027, hostvars, \u0027ansible_host\u0027)"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"6477b5f0_9effaf9f","line":43,"in_reply_to":"73895b1c_8d37a8d1","updated":"2022-08-29 10:55:00.000000000","message":"Oh yes ... reading https://mariadb.com/kb/en/proxy-protocol-support/ again .. it seems like \"localhost\" does cover 127.0.0.1 as well, but ::1 is still required.\n\nIf you don\u0027t mind I\u0027d add this to the list to have \"my machine\" covered fully","commit_id":"539d4df55282137e14f36c715593537766993bd6"},{"author":{"_account_id":28619,"name":"Dmitriy Rabotyagov","email":"noonedeadpunk@gmail.com","username":"noonedeadpunk"},"change_message_id":"f088a509d4c8cedbd844de68be47dc3526fad41e","unresolved":true,"context_lines":[{"line_number":40,"context_line":""},{"line_number":41,"context_line":"# Accept PROXY protocol information from the load balancers"},{"line_number":42,"context_line":"# See https://mariadb.com/kb/en/proxy-protocol-support for details"},{"line_number":43,"context_line":"galera_server_proxy_protocol_networks: \u003e-"},{"line_number":44,"context_line":"  {{"},{"line_number":45,"context_line":"    groups[\u0027haproxy\u0027]"},{"line_number":46,"context_line":"      | map(\u0027extract\u0027, hostvars, \u0027ansible_host\u0027)"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"73895b1c_8d37a8d1","line":43,"in_reply_to":"d647bfc7_054da45d","updated":"2022-08-29 10:10:29.000000000","message":"I think having just 127.0.0.1 or localhost should be enough. Let\u0027s see what CI thinks.","commit_id":"539d4df55282137e14f36c715593537766993bd6"}]}