)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"d362e786408aeb8cb5c3d18e4b5ad598790b20c8","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"2010abc7_462d294e","updated":"2022-04-29 14:24:47.000000000","message":"Suggestion for a clarification inline.  Thanks for the notes in the bug report, and for including the related-bugs here, it makes it a lot easier to review!","commit_id":"d57f16969078eefe974fb9a078caef8c32ed891b"},{"author":{"_account_id":20813,"name":"Sofia Enriquez","email":"lsofia.enriquez@gmail.com","username":"enriquetaso"},"change_message_id":"1be40aea4fa206109628d6e6095c6f2129fe9f6e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"809e5ba6_66e7d4a2","updated":"2022-04-29 18:08:33.000000000","message":"Thanks for addressing the comments. ","commit_id":"7ee223837ec99bf85d8a4b220457e6c1302acc3a"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"a20e5b43fd2c92644436db9171be221ef7e5c4bd","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"e62a47fc_6a0e08f0","updated":"2022-04-29 15:11:04.000000000","message":"Thanks, Vladislav!","commit_id":"7ee223837ec99bf85d8a4b220457e6c1302acc3a"},{"author":{"_account_id":10459,"name":"Luigi Toscano","email":"ltoscano@redhat.com","username":"ltoscano"},"change_message_id":"f1c78a1f36dad32b3bc3db543eb10f98698d049b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"83ab0fdb_2cf414eb","updated":"2022-04-29 15:37:29.000000000","message":"actual behavior++","commit_id":"7ee223837ec99bf85d8a4b220457e6c1302acc3a"}],"doc/source/contributor/migration.rst":[{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"d362e786408aeb8cb5c3d18e4b5ad598790b20c8","unresolved":true,"context_lines":[{"line_number":35,"context_line":""},{"line_number":36,"context_line":"For example, for older releases (before Ussuri), the migration of"},{"line_number":37,"context_line":"\u0027in-use\u0027 volume from RBD to LVM will succeed, but from LVM to RBD,"},{"line_number":38,"context_line":"it will fail. Currently migration of \u0027in-use\u0027 volumes for non-active"},{"line_number":39,"context_line":"VMs will fail for any backend, i.e. VMs must be running to succeed"},{"line_number":40,"context_line":"volume relocation."},{"line_number":41,"context_line":""},{"line_number":42,"context_line":"There are two major scenarios, which volume migration supports"},{"line_number":43,"context_line":"in Cinder:"}],"source_content_type":"text/x-rst","patch_set":1,"id":"6848645c_de249721","line":40,"range":{"start_line":38,"start_character":14,"end_line":40,"end_character":18},"updated":"2022-04-29 14:24:47.000000000","message":"Maybe rephrase as:\n\nCurrently, migration of \u0027in-use\u0027 volumes will not succeed for any backend when they are attached to an instance in any of these states: SHUTOFF, SUSPENDED, or SOFT-DELETED.","commit_id":"d57f16969078eefe974fb9a078caef8c32ed891b"},{"author":{"_account_id":13671,"name":"Vladislav Belogrudov","email":"v.belogrudov@yadro.com","username":"vb"},"change_message_id":"e1a4e175b5a32967104bbf0ae0b03c79b768ba01","unresolved":false,"context_lines":[{"line_number":35,"context_line":""},{"line_number":36,"context_line":"For example, for older releases (before Ussuri), the migration of"},{"line_number":37,"context_line":"\u0027in-use\u0027 volume from RBD to LVM will succeed, but from LVM to RBD,"},{"line_number":38,"context_line":"it will fail. Currently migration of \u0027in-use\u0027 volumes for non-active"},{"line_number":39,"context_line":"VMs will fail for any backend, i.e. VMs must be running to succeed"},{"line_number":40,"context_line":"volume relocation."},{"line_number":41,"context_line":""},{"line_number":42,"context_line":"There are two major scenarios, which volume migration supports"},{"line_number":43,"context_line":"in Cinder:"}],"source_content_type":"text/x-rst","patch_set":1,"id":"5dc04c89_4d699bef","line":40,"range":{"start_line":38,"start_character":14,"end_line":40,"end_character":18},"in_reply_to":"6848645c_de249721","updated":"2022-04-29 14:48:06.000000000","message":"thanks Brian, you suggestion is much more specific, I will include it in the next patch set.","commit_id":"d57f16969078eefe974fb9a078caef8c32ed891b"}]}