)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":14200,"name":"Maksim Malchuk","email":"maksim.malchuk@gmail.com","username":"mmalchuk"},"change_message_id":"46e79d3743e7951074c620a1e2996665d52a2148","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":2,"id":"9dabb648_4507fd15","updated":"2026-03-04 20:00:04.000000000","message":"why not backport this at least to 2025.2 ?","commit_id":"4880de6bae8aa1a586433e2f26bab3f2e4b560ff"},{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"fdc126487a992517c61bdd3fe0621f1dba64d39f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"a6342f9d_675444f6","in_reply_to":"2aef4521_dd9160e5","updated":"2026-03-05 05:48:57.000000000","message":"It’s a feature and we normally don’t backport features.","commit_id":"4880de6bae8aa1a586433e2f26bab3f2e4b560ff"},{"author":{"_account_id":290,"name":"Antony Messerli","email":"antony@mes.ser.li","username":"antonym"},"change_message_id":"f9af2e96b4397653968d9982934f21b6ff5da75d","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":2,"id":"2aef4521_dd9160e5","in_reply_to":"9dabb648_4507fd15","updated":"2026-03-04 20:47:42.000000000","message":"I think it could be backported, we were actually testing it on 2024.1 without a great way to override it.","commit_id":"4880de6bae8aa1a586433e2f26bab3f2e4b560ff"}],"ansible/roles/ovn-db/defaults/main.yml":[{"author":{"_account_id":37203,"name":"Bertrand Lanson","display_name":"Bertrand Lanson","email":"bertrand.lanson@infomaniak.com","username":"lanson","status":"Infomaniak Network SA"},"change_message_id":"1dd6cc186bb6e8950b9f70e83f6080c9691a4302","unresolved":true,"context_lines":[{"line_number":145,"context_line":"ovn_leader_election_pause: 5"},{"line_number":146,"context_line":""},{"line_number":147,"context_line":"# Extra command line arguments for ovn-northd"},{"line_number":148,"context_line":"ovn_northd_cmdline_extras: \"\""}],"source_content_type":"text/x-yaml","patch_set":2,"id":"af613af9_d00f9982","line":148,"range":{"start_line":148,"start_character":0,"end_line":148,"end_character":2},"updated":"2026-03-18 19:11:14.000000000","message":"could we make that an argument list that we join rather than a string ?\n\nThis is the \"standard\" way to do it nowadays I think for user facing extra arguments.\n\nI\u0027m fine either way I just think it looks better","commit_id":"4880de6bae8aa1a586433e2f26bab3f2e4b560ff"},{"author":{"_account_id":37203,"name":"Bertrand Lanson","display_name":"Bertrand Lanson","email":"bertrand.lanson@infomaniak.com","username":"lanson","status":"Infomaniak Network SA"},"change_message_id":"998478d57843bc601f02bfad30dfcfdc31f8b186","unresolved":false,"context_lines":[{"line_number":145,"context_line":"ovn_leader_election_pause: 5"},{"line_number":146,"context_line":""},{"line_number":147,"context_line":"# Extra command line arguments for ovn-northd"},{"line_number":148,"context_line":"ovn_northd_cmdline_extras: \"\""}],"source_content_type":"text/x-yaml","patch_set":2,"id":"254a0776_22f687f3","line":148,"range":{"start_line":148,"start_character":0,"end_line":148,"end_character":2},"in_reply_to":"8e76bb3f_d6037ed5","updated":"2026-03-23 18:19:21.000000000","message":"I think we can leave it as is for now, it seems to be ok for other cores. \n\nit\u0027s not a super complicated thing to change in the future","commit_id":"4880de6bae8aa1a586433e2f26bab3f2e4b560ff"},{"author":{"_account_id":290,"name":"Antony Messerli","email":"antony@mes.ser.li","username":"antonym"},"change_message_id":"e194f23e6f26b49a9c997e0167ba02fab902db4d","unresolved":true,"context_lines":[{"line_number":145,"context_line":"ovn_leader_election_pause: 5"},{"line_number":146,"context_line":""},{"line_number":147,"context_line":"# Extra command line arguments for ovn-northd"},{"line_number":148,"context_line":"ovn_northd_cmdline_extras: \"\""}],"source_content_type":"text/x-yaml","patch_set":2,"id":"8e76bb3f_d6037ed5","line":148,"range":{"start_line":148,"start_character":0,"end_line":148,"end_character":2},"in_reply_to":"af613af9_d00f9982","updated":"2026-03-20 20:53:11.000000000","message":"Thanks for the suggestion! \n\nThe string type here was intentional to stay consistent with the established cmdline_extras pattern used throughout kolla-ansible. (e.g., prometheus_cmdline_extras, prometheus_*_exporter_cmdline_extras, designate_backend_bind9_cmdline_extras, prometheus_cadvisor_cmdline_extras.)\n\nThe only list-based exception is ironic_kernel_cmdline_extras, which is a special case for kernel boot parameters.\n\nI\u0027d prefer to keep this as a string for consistency, but happy to change it if the desire is to move toward lists as the new convention going forward. Maybe that\u0027s a future iteration?","commit_id":"4880de6bae8aa1a586433e2f26bab3f2e4b560ff"}]}