)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"1a76240980b3a959cf9a6ab079431aa16d28a057","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"833c4d0f_74333137","updated":"2026-08-30 18:59:14.000000000","message":"I don\u0027t think changing the final status of the vmove is the right approach because evacuation actually failed.\n\nAlso its not clear what the actual problem reported by the bug. If the logging verbosity is the problem then we should just take care of that part.","commit_id":"3f18a6c24bbedea80fe480c2fbd1531f49f0cd32"},{"author":{"_account_id":39222,"name":"Gomathi G","display_name":"gomathi","email":"gomathig@cdac.in","username":"gomathi"},"change_message_id":"fe4dad05a0de1ad539f9a7f41d374c64f5520c6a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"466fd4be_b51e7248","in_reply_to":"833c4d0f_74333137","updated":"2026-08-31 10:02:31.000000000","message":"Thanks Takashi! I agree that altering the vmove status to SUCCESS misrepresents the recovery outcome. In Patch Set 2 / Patch Set 3, I\u0027ve updated the implementation to catch the NotFound exception cleanly to log an informative message and avoid flooding logs with unhandled tracebacks, while keeping the status logic untouched. Rebased to fix the merge conflict.","commit_id":"3f18a6c24bbedea80fe480c2fbd1531f49f0cd32"}],"masakari/engine/drivers/taskflow/host_failure.py":[{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"f1aa25edf73bbda15cf03ce0864c9afad87c2958","unresolved":false,"context_lines":[{"line_number":314,"context_line":"                    instance, \"OS-EXT-SRV-ATTR:hypervisor_hostname\")"},{"line_number":315,"context_line":"            except exception.NotFound:"},{"line_number":316,"context_line":"                dest_host \u003d None"},{"line_number":317,"context_line":"                LOG.info(\"Instance %s no longer exists or was deleted during evacuation. \""},{"line_number":318,"context_line":"                         \"Skipping further evacuation steps for this instance.\", instance_uuid)"},{"line_number":319,"context_line":"            _update_vmove("},{"line_number":320,"context_line":"                vmove,"}],"source_content_type":"text/x-python","patch_set":3,"id":"5924dfe1_12de0ac1","line":317,"updated":"2026-08-31 10:38:55.000000000","message":"linters: E501 line too long (90 \u003e 79 characters)","commit_id":"13dbba8ed5a8e182f1b12c5bff822ad8c003c004"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"f1aa25edf73bbda15cf03ce0864c9afad87c2958","unresolved":false,"context_lines":[{"line_number":314,"context_line":"                    instance, \"OS-EXT-SRV-ATTR:hypervisor_hostname\")"},{"line_number":315,"context_line":"            except exception.NotFound:"},{"line_number":316,"context_line":"                dest_host \u003d None"},{"line_number":317,"context_line":"                LOG.info(\"Instance %s no longer exists or was deleted during evacuation. \""},{"line_number":318,"context_line":"                         \"Skipping further evacuation steps for this instance.\", instance_uuid)"},{"line_number":319,"context_line":"            _update_vmove("},{"line_number":320,"context_line":"                vmove,"}],"source_content_type":"text/x-python","patch_set":3,"id":"df22cd9d_d48728ea","line":317,"updated":"2026-08-31 10:38:55.000000000","message":"pep8: E501 line too long (90 \u003e 79 characters)","commit_id":"13dbba8ed5a8e182f1b12c5bff822ad8c003c004"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"f1aa25edf73bbda15cf03ce0864c9afad87c2958","unresolved":false,"context_lines":[{"line_number":315,"context_line":"            except exception.NotFound:"},{"line_number":316,"context_line":"                dest_host \u003d None"},{"line_number":317,"context_line":"                LOG.info(\"Instance %s no longer exists or was deleted during evacuation. \""},{"line_number":318,"context_line":"                         \"Skipping further evacuation steps for this instance.\", instance_uuid)"},{"line_number":319,"context_line":"            _update_vmove("},{"line_number":320,"context_line":"                vmove,"},{"line_number":321,"context_line":"                status\u003dfields.VMoveStatus.SUCCEEDED,"}],"source_content_type":"text/x-python","patch_set":3,"id":"d725eb73_44323c93","line":318,"updated":"2026-08-31 10:38:55.000000000","message":"linters: E501 line too long (95 \u003e 79 characters)","commit_id":"13dbba8ed5a8e182f1b12c5bff822ad8c003c004"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"f1aa25edf73bbda15cf03ce0864c9afad87c2958","unresolved":false,"context_lines":[{"line_number":315,"context_line":"            except exception.NotFound:"},{"line_number":316,"context_line":"                dest_host \u003d None"},{"line_number":317,"context_line":"                LOG.info(\"Instance %s no longer exists or was deleted during evacuation. \""},{"line_number":318,"context_line":"                         \"Skipping further evacuation steps for this instance.\", instance_uuid)"},{"line_number":319,"context_line":"            _update_vmove("},{"line_number":320,"context_line":"                vmove,"},{"line_number":321,"context_line":"                status\u003dfields.VMoveStatus.SUCCEEDED,"}],"source_content_type":"text/x-python","patch_set":3,"id":"ce63aa23_852f2659","line":318,"updated":"2026-08-31 10:38:55.000000000","message":"pep8: E501 line too long (95 \u003e 79 characters)","commit_id":"13dbba8ed5a8e182f1b12c5bff822ad8c003c004"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"ccc65fa32bc4277da05e53d8b863f5152aa7174c","unresolved":false,"context_lines":[{"line_number":315,"context_line":"            except exception.NotFound:"},{"line_number":316,"context_line":"                dest_host \u003d None"},{"line_number":317,"context_line":"                LOG.info("},{"line_number":318,"context_line":"\t\t    \"Instance %s no longer exists or was deleted during evacuation. \""},{"line_number":319,"context_line":"                    \"Skipping further evacuation steps for this instance.\", "},{"line_number":320,"context_line":"\t\t    instance_uuid,"},{"line_number":321,"context_line":"\t\t)"}],"source_content_type":"text/x-python","patch_set":5,"id":"ea5313d0_5a728c90","line":318,"updated":"2026-09-02 08:55:35.000000000","message":"linters: E101 indentation contains mixed spaces and tabs","commit_id":"a23d3bea820ea832142d2b00d8a000ea45d6d413"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"ccc65fa32bc4277da05e53d8b863f5152aa7174c","unresolved":false,"context_lines":[{"line_number":315,"context_line":"            except exception.NotFound:"},{"line_number":316,"context_line":"                dest_host \u003d None"},{"line_number":317,"context_line":"                LOG.info("},{"line_number":318,"context_line":"\t\t    \"Instance %s no longer exists or was deleted during evacuation. \""},{"line_number":319,"context_line":"                    \"Skipping further evacuation steps for this instance.\", "},{"line_number":320,"context_line":"\t\t    instance_uuid,"},{"line_number":321,"context_line":"\t\t)"}],"source_content_type":"text/x-python","patch_set":5,"id":"f2ef3b15_2fc7662e","line":318,"updated":"2026-09-02 08:55:35.000000000","message":"linters: W191 indentation contains tabs","commit_id":"a23d3bea820ea832142d2b00d8a000ea45d6d413"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"ccc65fa32bc4277da05e53d8b863f5152aa7174c","unresolved":false,"context_lines":[{"line_number":315,"context_line":"            except exception.NotFound:"},{"line_number":316,"context_line":"                dest_host \u003d None"},{"line_number":317,"context_line":"                LOG.info("},{"line_number":318,"context_line":"\t\t    \"Instance %s no longer exists or was deleted during evacuation. \""},{"line_number":319,"context_line":"                    \"Skipping further evacuation steps for this instance.\", "},{"line_number":320,"context_line":"\t\t    instance_uuid,"},{"line_number":321,"context_line":"\t\t)"}],"source_content_type":"text/x-python","patch_set":5,"id":"a0848108_43464c0e","line":318,"updated":"2026-09-02 08:55:35.000000000","message":"pep8: E101 indentation contains mixed spaces and tabs","commit_id":"a23d3bea820ea832142d2b00d8a000ea45d6d413"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"ccc65fa32bc4277da05e53d8b863f5152aa7174c","unresolved":false,"context_lines":[{"line_number":315,"context_line":"            except exception.NotFound:"},{"line_number":316,"context_line":"                dest_host \u003d None"},{"line_number":317,"context_line":"                LOG.info("},{"line_number":318,"context_line":"\t\t    \"Instance %s no longer exists or was deleted during evacuation. \""},{"line_number":319,"context_line":"                    \"Skipping further evacuation steps for this instance.\", "},{"line_number":320,"context_line":"\t\t    instance_uuid,"},{"line_number":321,"context_line":"\t\t)"}],"source_content_type":"text/x-python","patch_set":5,"id":"95eeb0aa_35b374cf","line":318,"updated":"2026-09-02 08:55:35.000000000","message":"pep8: W191 indentation contains tabs","commit_id":"a23d3bea820ea832142d2b00d8a000ea45d6d413"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"ccc65fa32bc4277da05e53d8b863f5152aa7174c","unresolved":false,"context_lines":[{"line_number":316,"context_line":"                dest_host \u003d None"},{"line_number":317,"context_line":"                LOG.info("},{"line_number":318,"context_line":"\t\t    \"Instance %s no longer exists or was deleted during evacuation. \""},{"line_number":319,"context_line":"                    \"Skipping further evacuation steps for this instance.\", "},{"line_number":320,"context_line":"\t\t    instance_uuid,"},{"line_number":321,"context_line":"\t\t)"},{"line_number":322,"context_line":"            _update_vmove("}],"source_content_type":"text/x-python","patch_set":5,"id":"93f2c3b3_33541935","line":319,"updated":"2026-09-02 08:55:35.000000000","message":"linters: W291 trailing whitespace","commit_id":"a23d3bea820ea832142d2b00d8a000ea45d6d413"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"ccc65fa32bc4277da05e53d8b863f5152aa7174c","unresolved":false,"context_lines":[{"line_number":316,"context_line":"                dest_host \u003d None"},{"line_number":317,"context_line":"                LOG.info("},{"line_number":318,"context_line":"\t\t    \"Instance %s no longer exists or was deleted during evacuation. \""},{"line_number":319,"context_line":"                    \"Skipping further evacuation steps for this instance.\", "},{"line_number":320,"context_line":"\t\t    instance_uuid,"},{"line_number":321,"context_line":"\t\t)"},{"line_number":322,"context_line":"            _update_vmove("}],"source_content_type":"text/x-python","patch_set":5,"id":"d7dc3c23_a8274862","line":319,"updated":"2026-09-02 08:55:35.000000000","message":"pep8: W291 trailing whitespace","commit_id":"a23d3bea820ea832142d2b00d8a000ea45d6d413"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"ccc65fa32bc4277da05e53d8b863f5152aa7174c","unresolved":false,"context_lines":[{"line_number":317,"context_line":"                LOG.info("},{"line_number":318,"context_line":"\t\t    \"Instance %s no longer exists or was deleted during evacuation. \""},{"line_number":319,"context_line":"                    \"Skipping further evacuation steps for this instance.\", "},{"line_number":320,"context_line":"\t\t    instance_uuid,"},{"line_number":321,"context_line":"\t\t)"},{"line_number":322,"context_line":"            _update_vmove("},{"line_number":323,"context_line":"                vmove,"}],"source_content_type":"text/x-python","patch_set":5,"id":"55b23733_95301487","line":320,"updated":"2026-09-02 08:55:35.000000000","message":"linters: E101 indentation contains mixed spaces and tabs","commit_id":"a23d3bea820ea832142d2b00d8a000ea45d6d413"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"ccc65fa32bc4277da05e53d8b863f5152aa7174c","unresolved":false,"context_lines":[{"line_number":317,"context_line":"                LOG.info("},{"line_number":318,"context_line":"\t\t    \"Instance %s no longer exists or was deleted during evacuation. \""},{"line_number":319,"context_line":"                    \"Skipping further evacuation steps for this instance.\", "},{"line_number":320,"context_line":"\t\t    instance_uuid,"},{"line_number":321,"context_line":"\t\t)"},{"line_number":322,"context_line":"            _update_vmove("},{"line_number":323,"context_line":"                vmove,"}],"source_content_type":"text/x-python","patch_set":5,"id":"81d961fe_0832e1f2","line":320,"updated":"2026-09-02 08:55:35.000000000","message":"linters: W191 indentation contains tabs","commit_id":"a23d3bea820ea832142d2b00d8a000ea45d6d413"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"ccc65fa32bc4277da05e53d8b863f5152aa7174c","unresolved":false,"context_lines":[{"line_number":317,"context_line":"                LOG.info("},{"line_number":318,"context_line":"\t\t    \"Instance %s no longer exists or was deleted during evacuation. \""},{"line_number":319,"context_line":"                    \"Skipping further evacuation steps for this instance.\", "},{"line_number":320,"context_line":"\t\t    instance_uuid,"},{"line_number":321,"context_line":"\t\t)"},{"line_number":322,"context_line":"            _update_vmove("},{"line_number":323,"context_line":"                vmove,"}],"source_content_type":"text/x-python","patch_set":5,"id":"edd1fa61_4ff7f3cf","line":320,"updated":"2026-09-02 08:55:35.000000000","message":"pep8: E101 indentation contains mixed spaces and tabs","commit_id":"a23d3bea820ea832142d2b00d8a000ea45d6d413"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"ccc65fa32bc4277da05e53d8b863f5152aa7174c","unresolved":false,"context_lines":[{"line_number":317,"context_line":"                LOG.info("},{"line_number":318,"context_line":"\t\t    \"Instance %s no longer exists or was deleted during evacuation. \""},{"line_number":319,"context_line":"                    \"Skipping further evacuation steps for this instance.\", "},{"line_number":320,"context_line":"\t\t    instance_uuid,"},{"line_number":321,"context_line":"\t\t)"},{"line_number":322,"context_line":"            _update_vmove("},{"line_number":323,"context_line":"                vmove,"}],"source_content_type":"text/x-python","patch_set":5,"id":"fd36f90c_224e0ecf","line":320,"updated":"2026-09-02 08:55:35.000000000","message":"pep8: W191 indentation contains tabs","commit_id":"a23d3bea820ea832142d2b00d8a000ea45d6d413"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"ccc65fa32bc4277da05e53d8b863f5152aa7174c","unresolved":false,"context_lines":[{"line_number":318,"context_line":"\t\t    \"Instance %s no longer exists or was deleted during evacuation. \""},{"line_number":319,"context_line":"                    \"Skipping further evacuation steps for this instance.\", "},{"line_number":320,"context_line":"\t\t    instance_uuid,"},{"line_number":321,"context_line":"\t\t)"},{"line_number":322,"context_line":"            _update_vmove("},{"line_number":323,"context_line":"                vmove,"},{"line_number":324,"context_line":"                status\u003dfields.VMoveStatus.SUCCEEDED,"}],"source_content_type":"text/x-python","patch_set":5,"id":"9780b1d4_42ad8408","line":321,"updated":"2026-09-02 08:55:35.000000000","message":"linters: E101 indentation contains mixed spaces and tabs","commit_id":"a23d3bea820ea832142d2b00d8a000ea45d6d413"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"ccc65fa32bc4277da05e53d8b863f5152aa7174c","unresolved":false,"context_lines":[{"line_number":318,"context_line":"\t\t    \"Instance %s no longer exists or was deleted during evacuation. \""},{"line_number":319,"context_line":"                    \"Skipping further evacuation steps for this instance.\", "},{"line_number":320,"context_line":"\t\t    instance_uuid,"},{"line_number":321,"context_line":"\t\t)"},{"line_number":322,"context_line":"            _update_vmove("},{"line_number":323,"context_line":"                vmove,"},{"line_number":324,"context_line":"                status\u003dfields.VMoveStatus.SUCCEEDED,"}],"source_content_type":"text/x-python","patch_set":5,"id":"46440b9f_c5f5ecda","line":321,"updated":"2026-09-02 08:55:35.000000000","message":"linters: W191 indentation contains tabs","commit_id":"a23d3bea820ea832142d2b00d8a000ea45d6d413"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"ccc65fa32bc4277da05e53d8b863f5152aa7174c","unresolved":false,"context_lines":[{"line_number":318,"context_line":"\t\t    \"Instance %s no longer exists or was deleted during evacuation. \""},{"line_number":319,"context_line":"                    \"Skipping further evacuation steps for this instance.\", "},{"line_number":320,"context_line":"\t\t    instance_uuid,"},{"line_number":321,"context_line":"\t\t)"},{"line_number":322,"context_line":"            _update_vmove("},{"line_number":323,"context_line":"                vmove,"},{"line_number":324,"context_line":"                status\u003dfields.VMoveStatus.SUCCEEDED,"}],"source_content_type":"text/x-python","patch_set":5,"id":"54f23f66_5f60b703","line":321,"updated":"2026-09-02 08:55:35.000000000","message":"pep8: E101 indentation contains mixed spaces and tabs","commit_id":"a23d3bea820ea832142d2b00d8a000ea45d6d413"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"ccc65fa32bc4277da05e53d8b863f5152aa7174c","unresolved":false,"context_lines":[{"line_number":318,"context_line":"\t\t    \"Instance %s no longer exists or was deleted during evacuation. \""},{"line_number":319,"context_line":"                    \"Skipping further evacuation steps for this instance.\", "},{"line_number":320,"context_line":"\t\t    instance_uuid,"},{"line_number":321,"context_line":"\t\t)"},{"line_number":322,"context_line":"            _update_vmove("},{"line_number":323,"context_line":"                vmove,"},{"line_number":324,"context_line":"                status\u003dfields.VMoveStatus.SUCCEEDED,"}],"source_content_type":"text/x-python","patch_set":5,"id":"f0f71d40_e75b6272","line":321,"updated":"2026-09-02 08:55:35.000000000","message":"pep8: W191 indentation contains tabs","commit_id":"a23d3bea820ea832142d2b00d8a000ea45d6d413"}]}