)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":15197,"name":"Pierre Riteau","email":"pierre@stackhpc.com","username":"priteau","status":"StackHPC"},"change_message_id":"780a27a354cae91e95ce53fa2036da7df939f50f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"4ead828c_09894c61","updated":"2026-01-21 11:27:17.000000000","message":"Please add a release note.","commit_id":"bb21dd4d712bf2a765dd604eab17343d3ee1bbcf"}],"ansible/inventory/group_vars/all/kolla":[{"author":{"_account_id":28048,"name":"Will Szumski","email":"will@stackhpc.com","username":"jovial"},"change_message_id":"6eea06662f8f8cbc5f42dd67e894504f730f3dfc","unresolved":true,"context_lines":[{"line_number":546,"context_line":"kolla_enable_hacluster: \"no\""},{"line_number":547,"context_line":"kolla_enable_heat: \"{{ kolla_enable_openstack_core | bool }}\""},{"line_number":548,"context_line":"kolla_enable_horizon: \"{{ kolla_enable_openstack_core | bool }}\""},{"line_number":549,"context_line":"kolla_enable_influxdb: \"no\""},{"line_number":550,"context_line":"kolla_enable_ironic: \"no\""},{"line_number":551,"context_line":"kolla_enable_ironic_dnsmasq: \"{{ kolla_enable_ironic | bool and kolla_inspector_enable_discovery | bool }}\""},{"line_number":552,"context_line":"kolla_enable_ironic_pxe_filter: \"{{ kolla_enable_ironic | bool and kolla_inspector_enable_discovery | bool }}\""}],"source_content_type":"application/octet-stream","patch_set":4,"id":"815a049b_187372b8","line":549,"updated":"2026-01-27 12:19:27.000000000","message":"Not sure on this one as influxdb is the default storage backend for cloudkitty, so seems like a reasonable default to have in kayobe. Ideally we wouldn\u0027t duplicate these flags in kayobe and would just use the kolla-ansible defaults which take into account the storage backend:\n\n    enable_influxdb: \"{{ enable_cloudkitty | bool and cloudkitty_storage_backend \u003d\u003d \u0027influxdb\u0027 }}\"\n\nBut how exactly we could achieve that needs working out! There was a goal to remove this duplication between Kayobe and Kolla, but we don\u0027t have a concrete plan as of yet.\n\nCould we add some documentation to inform users to disable it if switching cloudkitty_backend instead?","commit_id":"dacffe9e3ff7af7fffcaac72dd96965df6c7daa3"},{"author":{"_account_id":28048,"name":"Will Szumski","email":"will@stackhpc.com","username":"jovial"},"change_message_id":"df11539556e26f297ba84fe7e06f89ec3696e232","unresolved":true,"context_lines":[{"line_number":546,"context_line":"kolla_enable_hacluster: \"no\""},{"line_number":547,"context_line":"kolla_enable_heat: \"{{ kolla_enable_openstack_core | bool }}\""},{"line_number":548,"context_line":"kolla_enable_horizon: \"{{ kolla_enable_openstack_core | bool }}\""},{"line_number":549,"context_line":"kolla_enable_influxdb: \"no\""},{"line_number":550,"context_line":"kolla_enable_ironic: \"no\""},{"line_number":551,"context_line":"kolla_enable_ironic_dnsmasq: \"{{ kolla_enable_ironic | bool and kolla_inspector_enable_discovery | bool }}\""},{"line_number":552,"context_line":"kolla_enable_ironic_pxe_filter: \"{{ kolla_enable_ironic | bool and kolla_inspector_enable_discovery | bool }}\""}],"source_content_type":"application/octet-stream","patch_set":4,"id":"1b980fa1_61a68e1c","line":549,"in_reply_to":"6759bfe3_1ada66f3","updated":"2026-01-27 12:42:45.000000000","message":"I wasn\u0027t aware, shouldn\u0027t we just drop the feature flag when https://review.opendev.org/c/openstack/kolla-ansible/+/973050 merges then?","commit_id":"dacffe9e3ff7af7fffcaac72dd96965df6c7daa3"},{"author":{"_account_id":15197,"name":"Pierre Riteau","email":"pierre@stackhpc.com","username":"priteau","status":"StackHPC"},"change_message_id":"99b732c9552114d3def6b647a77797f49e720649","unresolved":true,"context_lines":[{"line_number":546,"context_line":"kolla_enable_hacluster: \"no\""},{"line_number":547,"context_line":"kolla_enable_heat: \"{{ kolla_enable_openstack_core | bool }}\""},{"line_number":548,"context_line":"kolla_enable_horizon: \"{{ kolla_enable_openstack_core | bool }}\""},{"line_number":549,"context_line":"kolla_enable_influxdb: \"no\""},{"line_number":550,"context_line":"kolla_enable_ironic: \"no\""},{"line_number":551,"context_line":"kolla_enable_ironic_dnsmasq: \"{{ kolla_enable_ironic | bool and kolla_inspector_enable_discovery | bool }}\""},{"line_number":552,"context_line":"kolla_enable_ironic_pxe_filter: \"{{ kolla_enable_ironic | bool and kolla_inspector_enable_discovery | bool }}\""}],"source_content_type":"application/octet-stream","patch_set":4,"id":"6759bfe3_1ada66f3","line":549,"in_reply_to":"815a049b_187372b8","updated":"2026-01-27 12:24:38.000000000","message":"InfluxDB is getting removed from kolla-ansible, so we can\u0027t continue to rely on it.","commit_id":"dacffe9e3ff7af7fffcaac72dd96965df6c7daa3"},{"author":{"_account_id":14200,"name":"Maksim Malchuk","email":"maksim.malchuk@gmail.com","username":"mmalchuk"},"change_message_id":"a3204c9e4fe755dab1aba5df62f6bfddeec64bd1","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":4,"id":"2c3a55d7_b8fbbee8","line":704,"updated":"2026-01-21 14:25:08.000000000","message":"this line not related to the change","commit_id":"dacffe9e3ff7af7fffcaac72dd96965df6c7daa3"},{"author":{"_account_id":35263,"name":"Matt Crees","email":"mattc@stackhpc.com","username":"mattcrees"},"change_message_id":"5cce40356ba8dae7b4051368ae43023726ab3af3","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":4,"id":"1029d198_88169621","line":704,"in_reply_to":"2c3a55d7_b8fbbee8","updated":"2026-01-21 14:33:36.000000000","message":"Looks like the newline was missed from the last patch that touched this file. It\u0027s correct to add it in, in my opinion it\u0027s not worth putting into its own patch","commit_id":"dacffe9e3ff7af7fffcaac72dd96965df6c7daa3"}]}