)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":32666,"name":"Damian Dąbrowski","email":"damian@dabrowski.cloud","username":"ddabrowski"},"change_message_id":"bea6f6420b539f525b305709ae3261de700c530a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"cfc8bdef_26366b13","updated":"2023-02-01 22:01:39.000000000","message":"recheck - default value for `haproxy_ssl_all_vips` was added in https://review.opendev.org/c/openstack/openstack-ansible/+/871189","commit_id":"45fb346f1b111f5ab5e57740bd5423fc9ccaa955"},{"author":{"_account_id":32666,"name":"Damian Dąbrowski","email":"damian@dabrowski.cloud","username":"ddabrowski"},"change_message_id":"e23c40b5bfea9c588e0dc9f5236ddf5fa40876f6","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"7972d6ad_3fc58c3d","updated":"2023-02-01 18:46:38.000000000","message":"recheck - dependent changes are now fixed","commit_id":"45fb346f1b111f5ab5e57740bd5423fc9ccaa955"},{"author":{"_account_id":32666,"name":"Damian Dąbrowski","email":"damian@dabrowski.cloud","username":"ddabrowski"},"change_message_id":"ee35e684d5de36c53dcb34dfc3ef37f3bc844700","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"12ff7313_73187642","updated":"2023-02-02 08:31:23.000000000","message":"recheck - task `Perform a cell_v2 discover` failed. This is not related to this change and probably intermittent","commit_id":"45fb346f1b111f5ab5e57740bd5423fc9ccaa955"}],"defaults/main.yml":[{"author":{"_account_id":25023,"name":"Jonathan Rosser","email":"jonathan.rosser@rd.bbc.co.uk","username":"jrosser"},"change_message_id":"6353cc9d8325059170629f2eb7cf6e4a9181e07e","unresolved":true,"context_lines":[{"line_number":678,"context_line":"keystone_ssh_keypairs_principals:"},{"line_number":679,"context_line":"  - user: \"{{ keystone_system_user_name }}\""},{"line_number":680,"context_line":"    principals: \"{{ keystone_ssh_key_principals | default([\u0027keystone\u0027]) }}\""},{"line_number":681,"context_line":""},{"line_number":682,"context_line":"_haproxy_services:"},{"line_number":683,"context_line":"  - haproxy_service_name: keystone_service"},{"line_number":684,"context_line":"    haproxy_backend_nodes: \"{{ groups[\u0027keystone_all\u0027] | default([])  }}\""}],"source_content_type":"text/x-yaml","patch_set":1,"id":"b633cb93_95d02f8e","line":681,"updated":"2023-02-03 19:06:36.000000000","message":"this is a role for deploying keystone, there should not be any haproxy variables in here","commit_id":"45fb346f1b111f5ab5e57740bd5423fc9ccaa955"},{"author":{"_account_id":32666,"name":"Damian Dąbrowski","email":"damian@dabrowski.cloud","username":"ddabrowski"},"change_message_id":"5547c8857f54ff190947f209b22baa0093d0f2c3","unresolved":true,"context_lines":[{"line_number":678,"context_line":"keystone_ssh_keypairs_principals:"},{"line_number":679,"context_line":"  - user: \"{{ keystone_system_user_name }}\""},{"line_number":680,"context_line":"    principals: \"{{ keystone_ssh_key_principals | default([\u0027keystone\u0027]) }}\""},{"line_number":681,"context_line":""},{"line_number":682,"context_line":"_haproxy_services:"},{"line_number":683,"context_line":"  - haproxy_service_name: keystone_service"},{"line_number":684,"context_line":"    haproxy_backend_nodes: \"{{ groups[\u0027keystone_all\u0027] | default([])  }}\""}],"source_content_type":"text/x-yaml","patch_set":1,"id":"d1d7ac8d_c1b0be62","line":681,"in_reply_to":"b633cb93_95d02f8e","updated":"2023-02-06 23:26:04.000000000","message":"I thought it would be more convenient to put haproxy_services definition in role defaults.\n\nBut maybe you are right, I moved this part to openstack-ansible repo(inventory/group_vars/\u003cgroup_name\u003e.yml)","commit_id":"45fb346f1b111f5ab5e57740bd5423fc9ccaa955"}]}