)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"2e059f84970fe3273e1092e139c420162df2c7d6","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"a778efed_e365db36","updated":"2025-01-13 14:55:51.000000000","message":"clean backport, vaild bug and merged on preceedign branc(master) so this all looks good to me.\n\nnote on master thsi is one follow up for nova to enable this configuration option in nova-next.\n\nwhile we could backport that i dont feel that its needed.\nAs such i would consider this patch to be the final patch required to close the bug when considering backports.","commit_id":"828b2559a1b3c0b59c543e851c6ea3efb1baae20"},{"author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"change_message_id":"addffd2a656b9d8cf3d4b34e258e52ea19e48970","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"280559ac_55e5115b","in_reply_to":"a778efed_e365db36","updated":"2025-01-21 13:54:51.000000000","message":"Yepp, clean cherry pick indeed \u0026 merged on master branch. A bit \u0027feature-like\u0027 bug fix, but introduced with new config value defaulting the old behaviour looks OK to me. Let\u0027s merge this patch stack.","commit_id":"828b2559a1b3c0b59c543e851c6ea3efb1baae20"}],"placement/conf/placement.py":[{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"2e059f84970fe3273e1092e139c420162df2c7d6","unresolved":true,"context_lines":[{"line_number":91,"context_line":"\"\"\"),"},{"line_number":92,"context_line":"    cfg.StrOpt("},{"line_number":93,"context_line":"        \u0027allocation_candidates_generation_strategy\u0027,"},{"line_number":94,"context_line":"        default\u003d\"depth-first\","},{"line_number":95,"context_line":"        choices\u003d(\"depth-first\", \"breadth-first\"),"},{"line_number":96,"context_line":"        help\u003d\"\"\""},{"line_number":97,"context_line":"Defines the order placement visits viable root providers during allocation"}],"source_content_type":"text/x-python","patch_set":1,"id":"35740a07_4b014481","line":94,"updated":"2025-01-13 14:55:51.000000000","message":"note: again we default to the exisitng/old behavior of doing a depth-first seach so while the breadth-first search is aviable to mitigate the performace bug it is not enabled by default so there is no upgrade impact.\n\nin 2025.2 we will likely want to change the default to breath-first but we can discuss that separately.","commit_id":"828b2559a1b3c0b59c543e851c6ea3efb1baae20"},{"author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"change_message_id":"addffd2a656b9d8cf3d4b34e258e52ea19e48970","unresolved":false,"context_lines":[{"line_number":91,"context_line":"\"\"\"),"},{"line_number":92,"context_line":"    cfg.StrOpt("},{"line_number":93,"context_line":"        \u0027allocation_candidates_generation_strategy\u0027,"},{"line_number":94,"context_line":"        default\u003d\"depth-first\","},{"line_number":95,"context_line":"        choices\u003d(\"depth-first\", \"breadth-first\"),"},{"line_number":96,"context_line":"        help\u003d\"\"\""},{"line_number":97,"context_line":"Defines the order placement visits viable root providers during allocation"}],"source_content_type":"text/x-python","patch_set":1,"id":"d4d14425_c7b15956","line":94,"in_reply_to":"35740a07_4b014481","updated":"2025-01-21 13:54:51.000000000","message":"ACK, thanks for the details Sean, sounds good to me!","commit_id":"828b2559a1b3c0b59c543e851c6ea3efb1baae20"}]}