)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":34034,"name":"Franciszek Przewoźny","display_name":"Franciszek Przewozny","email":"przewozny.franciszek@gmail.com","username":"fprzewozn","status":"OpenStack Engineer / SRE @ Opera Software"},"change_message_id":"810fd4e80ec9b1ef152b0cbdef02b104cdac5f01","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"5b3e6773_05295998","updated":"2025-04-10 09:44:41.000000000","message":"@frickler@offenerstapel.de Can we merge it?","commit_id":"581e579e47680b9f04a29ecaec21efb6b3a1e0cf"},{"author":{"_account_id":34034,"name":"Franciszek Przewoźny","display_name":"Franciszek Przewozny","email":"przewozny.franciszek@gmail.com","username":"fprzewozn","status":"OpenStack Engineer / SRE @ Opera Software"},"change_message_id":"8e6efde92e5d95a77f95a51b9e8479f8e51dca7b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"5e1af32b_c4c0ea1f","updated":"2025-04-22 10:17:47.000000000","message":"@maksim.malchuk@gmail.com I have improved the check even further including your proposition. @frickler@offenerstapel.de Please recheck.","commit_id":"a8addf90d6cbeacf34435e0972d4b5d1ed3418c6"},{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"1325dd84ef069aa2044e457ffea504a3e25e4bf8","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"fdee5b39_b5d77b2a","updated":"2025-04-28 08:29:21.000000000","message":"Merging given previous +2","commit_id":"a8addf90d6cbeacf34435e0972d4b5d1ed3418c6"},{"author":{"_account_id":14200,"name":"Maksim Malchuk","email":"maksim.malchuk@gmail.com","username":"mmalchuk"},"change_message_id":"6b2bb0bf74357a870b7486a3635b85de1462dd08","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"8252ef36_a0e5739b","updated":"2025-04-22 11:45:40.000000000","message":"much better","commit_id":"a8addf90d6cbeacf34435e0972d4b5d1ed3418c6"}],"ansible/roles/prometheus/tasks/bootstrap.yml":[{"author":{"_account_id":14200,"name":"Maksim Malchuk","email":"maksim.malchuk@gmail.com","username":"mmalchuk"},"change_message_id":"5b22f5eb69db5078a020101dc1cf99b4be68a702","unresolved":true,"context_lines":[{"line_number":23,"context_line":"    label: \"{{ shard_host }}\""},{"line_number":24,"context_line":"  run_once: True"},{"line_number":25,"context_line":"  when:"},{"line_number":26,"context_line":"    - service | service_enabled_and_mapped_to_host"},{"line_number":27,"context_line":"    - not use_preconfigured_databases | bool"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"f8f411e3_3cbf2ea8","line":26,"range":{"start_line":26,"start_character":6,"end_line":26,"end_character":50},"updated":"2025-04-07 20:23:26.000000000","message":"It may be reasonable to add a further check, but really, should we drop the \u0027enable_\u0027 switch?","commit_id":"581e579e47680b9f04a29ecaec21efb6b3a1e0cf"},{"author":{"_account_id":14200,"name":"Maksim Malchuk","email":"maksim.malchuk@gmail.com","username":"mmalchuk"},"change_message_id":"8c778dd307c5d7ad545b01b9b986640386c9bdb9","unresolved":false,"context_lines":[{"line_number":23,"context_line":"    label: \"{{ shard_host }}\""},{"line_number":24,"context_line":"  run_once: True"},{"line_number":25,"context_line":"  when:"},{"line_number":26,"context_line":"    - service | service_enabled_and_mapped_to_host"},{"line_number":27,"context_line":"    - not use_preconfigured_databases | bool"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"72157f4e_d441dfcc","line":26,"range":{"start_line":26,"start_character":6,"end_line":26,"end_character":50},"in_reply_to":"0f679171_044ab3db","updated":"2025-04-08 19:29:25.000000000","message":"really. thanks.","commit_id":"581e579e47680b9f04a29ecaec21efb6b3a1e0cf"},{"author":{"_account_id":34034,"name":"Franciszek Przewoźny","display_name":"Franciszek Przewozny","email":"przewozny.franciszek@gmail.com","username":"fprzewozn","status":"OpenStack Engineer / SRE @ Opera Software"},"change_message_id":"68021584682d7cb66cfb87e6d096550b4bb6e7b3","unresolved":true,"context_lines":[{"line_number":23,"context_line":"    label: \"{{ shard_host }}\""},{"line_number":24,"context_line":"  run_once: True"},{"line_number":25,"context_line":"  when:"},{"line_number":26,"context_line":"    - service | service_enabled_and_mapped_to_host"},{"line_number":27,"context_line":"    - not use_preconfigured_databases | bool"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"0f679171_044ab3db","line":26,"range":{"start_line":26,"start_character":6,"end_line":26,"end_character":50},"in_reply_to":"f8f411e3_3cbf2ea8","updated":"2025-04-08 07:00:44.000000000","message":"`service_enabled_and_mapped_to_host` already checks `enable_` switch for this exporter [1], but we can stay with both declared here.\n\n[1] https://github.com/openstack/kolla-ansible/blob/master/ansible/roles/prometheus/defaults/main.yml#L42","commit_id":"581e579e47680b9f04a29ecaec21efb6b3a1e0cf"}]}