)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"f851c2b6e845db1ec41372a45e2f39ec253aa2b5","unresolved":true,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Fix: Convert status filter string to list in notification query"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Fixes bug #1953707: query notifications by status return null"},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"When querying notifications with status parameter:"},{"line_number":12,"context_line":"  GET /v1/notifications?status\u003dfinished,failed,error"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"7fb85801_8604b3fd","line":9,"range":{"start_line":9,"start_character":0,"end_line":9,"end_character":18},"updated":"2026-07-06 01:01:10.000000000","message":"Please use `Closes-Bug: #1953707` tag.","commit_id":"54aa0ee59e861007b8e23a9f57eadea2a9fba79d"},{"author":{"_account_id":39222,"name":"Gomathi G","display_name":"gomathi","email":"gomathig@cdac.in","username":"gomathi"},"change_message_id":"f13c1612c4df75ec7b04a8a4b262b4bfc3728788","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Fix: Convert status filter string to list in notification query"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Fixes bug #1953707: query notifications by status return null"},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"When querying notifications with status parameter:"},{"line_number":12,"context_line":"  GET /v1/notifications?status\u003dfinished,failed,error"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"a41b80e0_df38e305","line":9,"range":{"start_line":9,"start_character":0,"end_line":9,"end_character":18},"in_reply_to":"7fb85801_8604b3fd","updated":"2026-07-06 06:20:54.000000000","message":"These are intentional fixes for two related bugs:\n\n1. Bug #1953707: Notification status filter\n   - Status parameter was passed as string, should be list\n   - Fixes queries returning empty results\n   - Added 4 unit tests\n\n2. Bug #2023464: Handle deleted VM during evacuation  \n   - VM deleted during evacuation caused flow to fail\n   - Added try-except to handle NotFound gracefully\n   - Added 3 unit tests\n\nBoth bugs are in the notification/evacuation flow, so fixing them\ntogether in one change is appropriate. The changes are not duplicate\nbut complementary fixes.\n\nI will update the commit message to use \"Closes-Bug\" tags for clarity.","commit_id":"54aa0ee59e861007b8e23a9f57eadea2a9fba79d"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"f851c2b6e845db1ec41372a45e2f39ec253aa2b5","unresolved":true,"context_lines":[{"line_number":17,"context_line":"Solution: Convert comma-separated status string to list before"},{"line_number":18,"context_line":"querying the database."},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"Testing: Added 4 unit tests covering string, list, and missing filters."},{"line_number":21,"context_line":"Change-Id: I4309bba007d85423d770a011c3a000491f73b049"},{"line_number":22,"context_line":"Signed-off-by: Gomathi \u003cgomathig@cdac.in\u003e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"e03a399c_d821daa8","line":20,"range":{"start_line":20,"start_character":0,"end_line":20,"end_character":71},"updated":"2026-07-06 01:01:10.000000000","message":"(nit) This is clear in diff, so I\u0027d suggest removing this.","commit_id":"54aa0ee59e861007b8e23a9f57eadea2a9fba79d"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"23c968702a8c355819590d0619d0928fa8185b29","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"a1463cc6_b5f76612","updated":"2026-07-06 01:02:05.000000000","message":"Also I\u0027m wondering if this is a duplicate of https://review.opendev.org/c/openstack/masakari/+/995459 ?","commit_id":"54aa0ee59e861007b8e23a9f57eadea2a9fba79d"}],"masakari/engine/drivers/taskflow/host_failure.py":[{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"f851c2b6e845db1ec41372a45e2f39ec253aa2b5","unresolved":true,"context_lines":[{"line_number":302,"context_line":"                    # it should be set to \u0027error\u0027 after recovery."},{"line_number":303,"context_line":"                    if vm_state \u003d\u003d \u0027error\u0027:"},{"line_number":304,"context_line":"                        self.novaclient.reset_instance_state("},{"line_number":305,"context_line":"                            context, instance.id)"},{"line_number":306,"context_line":"            try:"},{"line_number":307,"context_line":"                instance \u003d self.novaclient.get_server(context, instance_uuid)"},{"line_number":308,"context_line":"                dest_host \u003d getattr("},{"line_number":309,"context_line":"                    instance, \"OS-EXT-SRV-ATTR:hypervisor_hostname\")"},{"line_number":310,"context_line":"            except exception.NotFound:"},{"line_number":311,"context_line":"                dest_host \u003d None"},{"line_number":312,"context_line":"                LOG.info(\"Instance \u0027%s\u0027 not found after evacuation. \""},{"line_number":313,"context_line":"                         \"The instance may have been deleted.\", instance_uuid)"},{"line_number":314,"context_line":"            _update_vmove("},{"line_number":315,"context_line":"                vmove,"},{"line_number":316,"context_line":"                status\u003dfields.VMoveStatus.SUCCEEDED,"}],"source_content_type":"text/x-python","patch_set":1,"id":"1dcdfb1d_2d368740","line":313,"range":{"start_line":305,"start_character":49,"end_line":313,"end_character":78},"updated":"2026-07-06 01:01:10.000000000","message":"Is this unrelated ?","commit_id":"54aa0ee59e861007b8e23a9f57eadea2a9fba79d"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"f851c2b6e845db1ec41372a45e2f39ec253aa2b5","unresolved":true,"context_lines":[{"line_number":315,"context_line":"                vmove,"},{"line_number":316,"context_line":"                status\u003dfields.VMoveStatus.SUCCEEDED,"},{"line_number":317,"context_line":"                dest_host\u003ddest_host)"},{"line_number":318,"context_line":"#            instance \u003d self.novaclient.get_server(context, instance_uuid)"},{"line_number":319,"context_line":"#            dest_host \u003d getattr("},{"line_number":320,"context_line":"#                instance, \"OS-EXT-SRV-ATTR:hypervisor_hostname\")"},{"line_number":321,"context_line":"#            _update_vmove("},{"line_number":322,"context_line":"#                vmove,"},{"line_number":323,"context_line":"#                status\u003dfields.VMoveStatus.SUCCEEDED,"},{"line_number":324,"context_line":"#                dest_host\u003ddest_host)"},{"line_number":325,"context_line":"        except loopingcall.LoopingCallTimeOut:"},{"line_number":326,"context_line":"            # Instance is not stop in the expected time_limit."},{"line_number":327,"context_line":"            msg \u003d \"Failed reason: timeout.\""}],"source_content_type":"text/x-python","patch_set":1,"id":"fe8ced9b_cda2dcb8","line":324,"range":{"start_line":318,"start_character":0,"end_line":324,"end_character":37},"updated":"2026-07-06 01:01:10.000000000","message":"What are these for ?","commit_id":"54aa0ee59e861007b8e23a9f57eadea2a9fba79d"}]}