)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"d824a7637670ab1cd3037394ea61bdc7d3f3d48d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"a458e16c_58b8c8d9","updated":"2026-08-04 11:16:18.000000000","message":"Great minds. I\u0027d also seen this issue, but my solution was [to use ruff (the linter, not the formatter) and its `LOG` rules](https://review.opendev.org/c/openstack/nova/+/974441/8/pyproject.toml#166). That patch needs rebasing, but I wonder if that approach would work for you? Alternatively, there is the existing [flake8-logging plugin](https://github.com/adamchainz/flake8-logging) that we could use instead, rather than writing our own boutique check?","commit_id":"f408ea42d933e15a4043d8b91b7e2861c5d5fce3"},{"author":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"change_message_id":"356e609f0e6d6842b0eb61c0fdac48545f5f380e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"a19ce5a3_86428aed","in_reply_to":"a458e16c_58b8c8d9","updated":"2026-08-11 19:33:09.000000000","message":"Ah, thanks for link -- it looks like that would work well. I have added a comment on the review.","commit_id":"f408ea42d933e15a4043d8b91b7e2861c5d5fce3"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"0f194f06f42a67dbdb62ff23366d1bd1bb09f877","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":5,"id":"0da26ef9_841c4216","updated":"2026-10-12 12:47:11.000000000","message":"im kind of nutral on this\n\ni expect in most cases it wont have any perframce beinifit or cost\n\nand we are giving up the ever evaualtion which catches bugs wheree vairbals ar eout of scope or not initalised and then fail only when debug logging is enabeld\n\nso while i woudl prefer not to regress that im mostly fine iwth this if others agree","commit_id":"e581cc857588aed09923a1f1268b696ee7b517e8"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"fc48713e6aca2150119cead37128689aa7467206","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"c46bb2d3_adec7641","updated":"2026-10-20 10:54:56.000000000","message":"recheck","commit_id":"a099e5e99faf71552016036e3a3d54b20bbf7f88"},{"author":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"change_message_id":"f587374f40e29b4791a1308545b41c8d2f5b8e5e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"7a227957_ea98719f","updated":"2026-10-21 14:08:54.000000000","message":"recheck RuntimeError: nova.tests.functional.libvirt.test_live_migration.LiveMigrationQueuedAbortTestVmStatus.test_queued_live_migration_abort_vm_status: the executors still have tasks that did not finish in 10s","commit_id":"a099e5e99faf71552016036e3a3d54b20bbf7f88"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"86628b7ce7482c5405dc81193464a2d4d5e3c68d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"633a1e2c_4af33be5","updated":"2026-10-20 14:01:41.000000000","message":"recheck nova-multi-cell failure","commit_id":"a099e5e99faf71552016036e3a3d54b20bbf7f88"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"f32cde54677329a84b4d00105fa7a7ce04f722b4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"8573fe9e_8a21e14f","updated":"2026-10-20 13:28:33.000000000","message":"still looks good to me","commit_id":"a099e5e99faf71552016036e3a3d54b20bbf7f88"}],"nova/conductor/manager.py":[{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"0f194f06f42a67dbdb62ff23366d1bd1bb09f877","unresolved":false,"context_lines":[{"line_number":550,"context_line":"                migration.status \u003d \u0027error\u0027"},{"line_number":551,"context_line":"                migration.save()"},{"line_number":552,"context_line":"        except Exception as ex:"},{"line_number":553,"context_line":"            LOG.exception("},{"line_number":554,"context_line":"                \u0027Migration of instance %(instance_id)s to host\u0027"},{"line_number":555,"context_line":"                \u0027 %(dest)s unexpectedly failed.\u0027,"},{"line_number":556,"context_line":"                {\u0027instance_id\u0027: instance.uuid, \u0027dest\u0027: destination})"}],"source_content_type":"text/x-python","patch_set":5,"id":"21635a41_36b6954a","line":553,"range":{"start_line":553,"start_character":16,"end_line":553,"end_character":25},"updated":"2026-10-12 12:47:11.000000000","message":"yes useing log excption is more correct\n\nits effectivly identical in practice but its the correct \"spelling\" to convay intent","commit_id":"e581cc857588aed09923a1f1268b696ee7b517e8"}]}