)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},"change_message_id":"d6561685bccc13748ed012dd1e1e2fd05b2175fd","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"b084cb92_a6fc05c2","updated":"2026-01-22 19:29:37.000000000","message":"If we want to backport this, we need to include a release note in this patch","commit_id":"b1f548f322f429a1846f49644a967c5f3b82017a"},{"author":{"_account_id":16312,"name":"Alfredo Moralejo","email":"amoralej@redhat.com","username":"amoralej"},"change_message_id":"819e619820b4fa6b4a10366debfb3b4cbbe35b06","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"de541ba7_8538d8ca","updated":"2026-01-22 11:54:20.000000000","message":"check-rdo","commit_id":"b1f548f322f429a1846f49644a967c5f3b82017a"},{"author":{"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},"change_message_id":"a79f21552ce784049f4379aad302494e261dcb76","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"fcc94737_2ef73ff6","updated":"2026-01-27 10:48:05.000000000","message":"LGTM, thanks for adding the reno","commit_id":"857e14a9d8180dc4cfe24a1c3eb77dd579dc3547"},{"author":{"_account_id":16312,"name":"Alfredo Moralejo","email":"amoralej@redhat.com","username":"amoralej"},"change_message_id":"dcad0534c9ba4b2216095909ee6e698deafe2bb1","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"445e87b3_66dab870","updated":"2026-01-27 13:03:11.000000000","message":"recheck","commit_id":"857e14a9d8180dc4cfe24a1c3eb77dd579dc3547"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"158a5caca047550e5e2cf549088665ddcdaf0831","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"fe5fd2fe_ff60de21","updated":"2026-01-26 20:02:24.000000000","message":"recheck","commit_id":"857e14a9d8180dc4cfe24a1c3eb77dd579dc3547"}],"watcher/decision_engine/model/notification/nova.py":[{"author":{"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},"change_message_id":"d6561685bccc13748ed012dd1e1e2fd05b2175fd","unresolved":true,"context_lines":[{"line_number":340,"context_line":"        \u0027instance.rescue.end\u0027: instance_updated,"},{"line_number":341,"context_line":"        \u0027instance.update\u0027: instance_updated,"},{"line_number":342,"context_line":"        \u0027instance.live_migration_force_complete.end\u0027: instance_updated,"},{"line_number":343,"context_line":"        \u0027instance.live_migration_post.end\u0027: instance_updated,"},{"line_number":344,"context_line":"        \u0027instance.delete.end\u0027: instance_deleted,"},{"line_number":345,"context_line":"        \u0027instance.soft_delete.end\u0027: instance_deleted,"},{"line_number":346,"context_line":"        \u0027service.create\u0027: service_updated,"}],"source_content_type":"text/x-python","patch_set":1,"id":"b151cb95_00466489","side":"PARENT","line":343,"range":{"start_line":343,"start_character":18,"end_line":343,"end_character":37},"updated":"2026-01-22 19:29:37.000000000","message":"While looking at nova\u0027s code, I can see the source notification is after destination notification, but it doesn\u0027t have an instance object updated, so we get the previous host info. Not sure if this issue shouldn\u0027t be fixed in nova side too.\nAt this moment seems more correct to process the post_dest since it will point to the right host. IIUC, in nova code, if the post_dest fails, we don\u0027t receive source notification too.","commit_id":"4f17759e794d2f07e7ae209f1f07f4e79733db1d"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"158a5caca047550e5e2cf549088665ddcdaf0831","unresolved":true,"context_lines":[{"line_number":340,"context_line":"        \u0027instance.rescue.end\u0027: instance_updated,"},{"line_number":341,"context_line":"        \u0027instance.update\u0027: instance_updated,"},{"line_number":342,"context_line":"        \u0027instance.live_migration_force_complete.end\u0027: instance_updated,"},{"line_number":343,"context_line":"        \u0027instance.live_migration_post.end\u0027: instance_updated,"},{"line_number":344,"context_line":"        \u0027instance.delete.end\u0027: instance_deleted,"},{"line_number":345,"context_line":"        \u0027instance.soft_delete.end\u0027: instance_deleted,"},{"line_number":346,"context_line":"        \u0027service.create\u0027: service_updated,"}],"source_content_type":"text/x-python","patch_set":1,"id":"2bd1129c_3c5308d2","side":"PARENT","line":343,"range":{"start_line":343,"start_character":18,"end_line":343,"end_character":37},"in_reply_to":"b151cb95_00466489","updated":"2026-01-26 20:02:24.000000000","message":"the way we set this has evovled somehwat in nova over the years so while the overall post live migration shoudl ahve the correct hos i think this is an ok fix as well.\n\nin the future both should have the same instnace.host set ideally","commit_id":"4f17759e794d2f07e7ae209f1f07f4e79733db1d"}],"watcher/tests/unit/decision_engine/model/notification/test_nova_notifications.py":[{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"158a5caca047550e5e2cf549088665ddcdaf0831","unresolved":false,"context_lines":[{"line_number":71,"context_line":"        \u0027instance.update\u0027: \u0027instance-update.json\u0027,"},{"line_number":72,"context_line":"        \u0027instance.live_migration_force_complete.end\u0027:"},{"line_number":73,"context_line":"        \u0027instance-live_migration_force_complete-end.json\u0027,"},{"line_number":74,"context_line":"        \u0027instance.live_migration_post_dest.end\u0027:"},{"line_number":75,"context_line":"        \u0027instance-live_migration_post_dest-end.json\u0027,"},{"line_number":76,"context_line":"        \u0027instance.delete.end\u0027: \u0027instance-delete-end.json\u0027,"},{"line_number":77,"context_line":"        \u0027instance.soft_delete.end\u0027: \u0027instance-soft_delete-end.json\u0027,"}],"source_content_type":"text/x-python","patch_set":3,"id":"ee9d3965_03fe6150","line":74,"in_reply_to":"b2f1f25a_de02f8e3","updated":"2026-01-26 20:02:24.000000000","message":"not relevent","commit_id":"857e14a9d8180dc4cfe24a1c3eb77dd579dc3547"}]}
