)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"3c87ca0553ac440ba478c3d694feeffc7b6cb481","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":15,"id":"8853fc46_ae47dcb2","updated":"2021-12-07 16:21:11.000000000","message":"Comments inline.","commit_id":"b55a708cace24a9da96da0bd38048035205b1ca0"},{"author":{"_account_id":26458,"name":"Brin Zhang","email":"zhangbailin@inspur.com","username":"zhangbailin"},"change_message_id":"d839b4ebd790c978a2d566d9d0d5d03d99703a48","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":18,"id":"70953dae_0a3c396e","updated":"2022-03-03 02:17:59.000000000","message":"+1, It\u0027s better for user to maintain the volumes, keep volume resources have the same az, it\u0027s friendly. \n-1, But you should move this spec to zed dir. ","commit_id":"024ad7180d4c341a2cb5af7158b417295f877c89"},{"author":{"_account_id":13425,"name":"Simon Dodsley","email":"simon@purestorage.com","username":"sdodsley"},"change_message_id":"145201e2bafcccfbaf007a6c9afabb894dbc430d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":18,"id":"567e4034_96d66683","updated":"2022-03-03 17:35:36.000000000","message":"This needs to be moved to the Zed directory and Brians\u0027 questions still need to be answered.","commit_id":"024ad7180d4c341a2cb5af7158b417295f877c89"},{"author":{"_account_id":13425,"name":"Simon Dodsley","email":"simon@purestorage.com","username":"sdodsley"},"change_message_id":"c7a467228f2f70c063f3ef831069c816dbbf74a7","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":20,"id":"b1d648ab_0b884770","updated":"2022-03-31 14:33:36.000000000","message":"There are still responses required to previous comments before we can look at this further","commit_id":"3a0b2948dcfd36c943462d594f65bfe0e35eb558"},{"author":{"_account_id":30092,"name":"Xuan Yandong","email":"xuanyd@outlook.com","username":"xuanyandong"},"change_message_id":"a1ad9e867cfbad881389d8732dabfadc6d520623","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":20,"id":"9e2f2d23_efc948b7","in_reply_to":"b1d648ab_0b884770","updated":"2022-08-12 01:35:43.000000000","message":"ok，please review again， thanks very much。","commit_id":"3a0b2948dcfd36c943462d594f65bfe0e35eb558"},{"author":{"_account_id":13425,"name":"Simon Dodsley","email":"simon@purestorage.com","username":"sdodsley"},"change_message_id":"9ad07cb44854641ec896c7b0288986402dbb0b25","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":21,"id":"e52c4c1f_0c3d4acd","updated":"2022-09-15 12:40:57.000000000","message":"Quite a few grammatical and spelling nots","commit_id":"8efa0495598058bc3aaed2b8dfb2d27b0fd4c7ab"},{"author":{"_account_id":13425,"name":"Simon Dodsley","email":"simon@purestorage.com","username":"sdodsley"},"change_message_id":"76f3c94afd9489b581dd41c90ffa582fd4860b0d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":21,"id":"5de41f29_d0b887ab","updated":"2026-02-19 14:41:19.000000000","message":"This appears to be stale so probably should be abandoned","commit_id":"8efa0495598058bc3aaed2b8dfb2d27b0fd4c7ab"}],"specs/xena/update_original_volume_az.rst":[{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"06625b555c2fa300c350ec18132fb7838f0752a0","unresolved":true,"context_lines":[{"line_number":17,"context_line":"Problem description"},{"line_number":18,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"Currently, if user change the backend\u0027s ``availability_zone``, the old volumes\u0027"},{"line_number":21,"context_line":"``availability_zone`` won\u0027t be updated after cinder-volume service restart, but"},{"line_number":22,"context_line":"the cinder-volume service\u0027s ``availability_zone`` has been updated."},{"line_number":23,"context_line":""}],"source_content_type":"text/x-rst","patch_set":6,"id":"ddbd977c_551d3d94","line":20,"range":{"start_line":20,"start_character":14,"end_line":20,"end_character":18},"updated":"2021-04-23 15:23:12.000000000","message":"please make clear that you mean user as operator here, not end user.","commit_id":"24ad98fa0fa7f27d80b8831567e83f8744802864"},{"author":{"_account_id":30092,"name":"Xuan Yandong","email":"xuanyd@outlook.com","username":"xuanyandong"},"change_message_id":"9231d7d67bb315b542f2293fa985199eafaed474","unresolved":true,"context_lines":[{"line_number":17,"context_line":"Problem description"},{"line_number":18,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"Currently, if user change the backend\u0027s ``availability_zone``, the old volumes\u0027"},{"line_number":21,"context_line":"``availability_zone`` won\u0027t be updated after cinder-volume service restart, but"},{"line_number":22,"context_line":"the cinder-volume service\u0027s ``availability_zone`` has been updated."},{"line_number":23,"context_line":""}],"source_content_type":"text/x-rst","patch_set":6,"id":"01fd0b98_60496e39","line":20,"range":{"start_line":20,"start_character":14,"end_line":20,"end_character":18},"in_reply_to":"ddbd977c_551d3d94","updated":"2021-04-25 03:18:52.000000000","message":"Ok, got it, thanks for review.","commit_id":"24ad98fa0fa7f27d80b8831567e83f8744802864"},{"author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"change_message_id":"3a9a553becbd67d9e31806c0006277476463403a","unresolved":true,"context_lines":[{"line_number":42,"context_line":""},{"line_number":43,"context_line":"This proposal would add operations in cinder-namege tool, Update when end user"},{"line_number":44,"context_line":"ruun the ``cinder-manage volume update_az --backend_name \u003cstorage_backend_name\u003e``."},{"line_number":45,"context_line":""},{"line_number":46,"context_line":"Alternatives"},{"line_number":47,"context_line":"------------"},{"line_number":48,"context_line":""}],"source_content_type":"text/x-rst","patch_set":9,"id":"5a8adfd2_158c3428","line":45,"updated":"2021-06-02 15:20:57.000000000","message":"Would this command be allowed to update volumes that are in-use?  Should there be a check that all volumes are in \"available\"?\n\nSimilarly, we need to check for volumes that are in a consistency group -- other volumes in that group would need an update, as would the group itself.","commit_id":"66d5f7d4d25f90da0c18a82dd8cb612b479c5d00"}],"specs/yoga/update_original_volume_az.rst":[{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"3c87ca0553ac440ba478c3d694feeffc7b6cb481","unresolved":true,"context_lines":[{"line_number":23,"context_line":"updated."},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"The ``availability_zone`` of the old resources is no longer correct, and the end"},{"line_number":26,"context_line":"user can\u0027t be update it. And this will affect the subsequent operation of"},{"line_number":27,"context_line":"these resources."},{"line_number":28,"context_line":""},{"line_number":29,"context_line":"E.g., a snapshot was created from the old volume with it\u0027s backend"}],"source_content_type":"text/x-rst","patch_set":15,"id":"d0c40a38_77ad0df6","line":26,"range":{"start_line":26,"start_character":11,"end_line":26,"end_character":13},"updated":"2021-12-07 16:21:11.000000000","message":"nit: don\u0027t need this word","commit_id":"b55a708cace24a9da96da0bd38048035205b1ca0"},{"author":{"_account_id":30092,"name":"Xuan Yandong","email":"xuanyd@outlook.com","username":"xuanyandong"},"change_message_id":"7afe72e1e7b15cf22c99f54758d1eccda3cd0581","unresolved":false,"context_lines":[{"line_number":23,"context_line":"updated."},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"The ``availability_zone`` of the old resources is no longer correct, and the end"},{"line_number":26,"context_line":"user can\u0027t be update it. And this will affect the subsequent operation of"},{"line_number":27,"context_line":"these resources."},{"line_number":28,"context_line":""},{"line_number":29,"context_line":"E.g., a snapshot was created from the old volume with it\u0027s backend"}],"source_content_type":"text/x-rst","patch_set":15,"id":"b8d894b7_56a8d9a1","line":26,"range":{"start_line":26,"start_character":11,"end_line":26,"end_character":13},"in_reply_to":"d0c40a38_77ad0df6","updated":"2021-12-08 02:03:36.000000000","message":"Done","commit_id":"b55a708cace24a9da96da0bd38048035205b1ca0"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"3c87ca0553ac440ba478c3d694feeffc7b6cb481","unresolved":true,"context_lines":[{"line_number":34,"context_line":"Use Cases"},{"line_number":35,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":36,"context_line":""},{"line_number":37,"context_line":"As an operator, I would like to change the backend ``availability_zone``, At the"},{"line_number":38,"context_line":"same time, the ``availability_zone`` of the volume, backup and snapshot can be"},{"line_number":39,"context_line":"updated by the cinder-manage tool"},{"line_number":40,"context_line":""}],"source_content_type":"text/x-rst","patch_set":15,"id":"524b00eb_7ce21c47","line":37,"range":{"start_line":37,"start_character":72,"end_line":37,"end_character":73},"updated":"2021-12-07 16:21:11.000000000","message":"nit: should be a period","commit_id":"b55a708cace24a9da96da0bd38048035205b1ca0"},{"author":{"_account_id":30092,"name":"Xuan Yandong","email":"xuanyd@outlook.com","username":"xuanyandong"},"change_message_id":"7afe72e1e7b15cf22c99f54758d1eccda3cd0581","unresolved":false,"context_lines":[{"line_number":34,"context_line":"Use Cases"},{"line_number":35,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":36,"context_line":""},{"line_number":37,"context_line":"As an operator, I would like to change the backend ``availability_zone``, At the"},{"line_number":38,"context_line":"same time, the ``availability_zone`` of the volume, backup and snapshot can be"},{"line_number":39,"context_line":"updated by the cinder-manage tool"},{"line_number":40,"context_line":""}],"source_content_type":"text/x-rst","patch_set":15,"id":"66b8b2de_3b03273f","line":37,"range":{"start_line":37,"start_character":72,"end_line":37,"end_character":73},"in_reply_to":"524b00eb_7ce21c47","updated":"2021-12-08 02:03:36.000000000","message":"Done","commit_id":"b55a708cace24a9da96da0bd38048035205b1ca0"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"3c87ca0553ac440ba478c3d694feeffc7b6cb481","unresolved":true,"context_lines":[{"line_number":41,"context_line":"Proposed change"},{"line_number":42,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":43,"context_line":""},{"line_number":44,"context_line":"This proposal would add operations in cinder-namege tool, Update when end user"},{"line_number":45,"context_line":"run the ``cinder-manage volume update_az --backend_name \u003cstorage_backend_name\u003e``"},{"line_number":46,"context_line":"and  ``cinder-manage backup update_az --backend_name \u003cstorage_backend_name\u003e``"},{"line_number":47,"context_line":"and ``cinder-manage snapshot update_az --backend_name \u003cstorage_backend_name\u003e``"}],"source_content_type":"text/x-rst","patch_set":15,"id":"3b79fe73_9f99d5c1","line":44,"range":{"start_line":44,"start_character":45,"end_line":44,"end_character":51},"updated":"2021-12-07 16:21:11.000000000","message":"manage","commit_id":"b55a708cace24a9da96da0bd38048035205b1ca0"},{"author":{"_account_id":30092,"name":"Xuan Yandong","email":"xuanyd@outlook.com","username":"xuanyandong"},"change_message_id":"7afe72e1e7b15cf22c99f54758d1eccda3cd0581","unresolved":false,"context_lines":[{"line_number":41,"context_line":"Proposed change"},{"line_number":42,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":43,"context_line":""},{"line_number":44,"context_line":"This proposal would add operations in cinder-namege tool, Update when end user"},{"line_number":45,"context_line":"run the ``cinder-manage volume update_az --backend_name \u003cstorage_backend_name\u003e``"},{"line_number":46,"context_line":"and  ``cinder-manage backup update_az --backend_name \u003cstorage_backend_name\u003e``"},{"line_number":47,"context_line":"and ``cinder-manage snapshot update_az --backend_name \u003cstorage_backend_name\u003e``"}],"source_content_type":"text/x-rst","patch_set":15,"id":"c77bcd6a_5f94863f","line":44,"range":{"start_line":44,"start_character":45,"end_line":44,"end_character":51},"in_reply_to":"3b79fe73_9f99d5c1","updated":"2021-12-08 02:03:36.000000000","message":"Done","commit_id":"b55a708cace24a9da96da0bd38048035205b1ca0"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"3c87ca0553ac440ba478c3d694feeffc7b6cb481","unresolved":true,"context_lines":[{"line_number":45,"context_line":"run the ``cinder-manage volume update_az --backend_name \u003cstorage_backend_name\u003e``"},{"line_number":46,"context_line":"and  ``cinder-manage backup update_az --backend_name \u003cstorage_backend_name\u003e``"},{"line_number":47,"context_line":"and ``cinder-manage snapshot update_az --backend_name \u003cstorage_backend_name\u003e``"},{"line_number":48,"context_line":""},{"line_number":49,"context_line":"Alternatives"},{"line_number":50,"context_line":"------------"},{"line_number":51,"context_line":""}],"source_content_type":"text/x-rst","patch_set":15,"id":"7eb08291_6aadb93e","line":48,"updated":"2021-12-07 16:21:11.000000000","message":"A few questions:\n\n1. Does the operator have to do anything before running the \u0027cinder-manage volume update_az\u0027 command (for example, make sure all the volumes that are affected are in \u0027available\u0027 status)?  Or is that not a concern (if not, please say why this isn\u0027t a problem for in-use volumes).\n\n2. Similar question about backups, snapshots (if it\u0027s not relevant, say so).\n\n3. Please say something about what the command will do ... I take it that it will look up the AZ of the specified backend, then find all volumes/backups/snapshots on that backend, and update them?\n\n4. Is there a reason to split the commands like this?  What I mean is, is there a use case for an operator calling \u0027cinder-manage volume update_az\u0027 but not also calling \u0027cinder-manage backup update_az\u0027?\n\n5. What about generic groups and consistency groups?  Will the tool update these, or do they need to be done by hand (like volume-types?)?","commit_id":"b55a708cace24a9da96da0bd38048035205b1ca0"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"3c87ca0553ac440ba478c3d694feeffc7b6cb481","unresolved":true,"context_lines":[{"line_number":87,"context_line":"None"},{"line_number":88,"context_line":""},{"line_number":89,"context_line":"Other deployer impact"},{"line_number":90,"context_line":"---------------------"},{"line_number":91,"context_line":""},{"line_number":92,"context_line":"None"},{"line_number":93,"context_line":""}],"source_content_type":"text/x-rst","patch_set":15,"id":"18191248_89610669","line":90,"updated":"2021-12-07 16:21:11.000000000","message":"It\u0027s possible for a volume-type to specify an availability zone.  You need to point out that this tool won\u0027t address that: any such volume types will have to be updated individually by the operator (as they do now).","commit_id":"b55a708cace24a9da96da0bd38048035205b1ca0"}],"specs/zed/update_original_volume_az.rst":[{"author":{"_account_id":13425,"name":"Simon Dodsley","email":"simon@purestorage.com","username":"sdodsley"},"change_message_id":"442059cd48d2c233c7ff307d88e2ae3e99bc5ad2","unresolved":true,"context_lines":[{"line_number":17,"context_line":"Problem description"},{"line_number":18,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"Currently, if operator change the backend\u0027s ``availability_zone``, the old volume,"},{"line_number":21,"context_line":"backup and snapshot\u0027s ``availability_zone`` won\u0027t be updated after cinder-volume"},{"line_number":22,"context_line":"service restart, but the cinder-volume service\u0027s ``availability_zone`` has been"},{"line_number":23,"context_line":"updated."}],"source_content_type":"text/x-rst","patch_set":21,"id":"d6358984_ac48a66d","line":20,"range":{"start_line":20,"start_character":23,"end_line":20,"end_character":30},"updated":"2022-09-15 12:48:14.000000000","message":"nit: changes","commit_id":"8efa0495598058bc3aaed2b8dfb2d27b0fd4c7ab"},{"author":{"_account_id":13425,"name":"Simon Dodsley","email":"simon@purestorage.com","username":"sdodsley"},"change_message_id":"442059cd48d2c233c7ff307d88e2ae3e99bc5ad2","unresolved":true,"context_lines":[{"line_number":17,"context_line":"Problem description"},{"line_number":18,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"Currently, if operator change the backend\u0027s ``availability_zone``, the old volume,"},{"line_number":21,"context_line":"backup and snapshot\u0027s ``availability_zone`` won\u0027t be updated after cinder-volume"},{"line_number":22,"context_line":"service restart, but the cinder-volume service\u0027s ``availability_zone`` has been"},{"line_number":23,"context_line":"updated."}],"source_content_type":"text/x-rst","patch_set":21,"id":"f77a5eba_051cf013","line":20,"updated":"2022-09-15 12:48:14.000000000","message":"not: the","commit_id":"8efa0495598058bc3aaed2b8dfb2d27b0fd4c7ab"},{"author":{"_account_id":13425,"name":"Simon Dodsley","email":"simon@purestorage.com","username":"sdodsley"},"change_message_id":"442059cd48d2c233c7ff307d88e2ae3e99bc5ad2","unresolved":true,"context_lines":[{"line_number":19,"context_line":""},{"line_number":20,"context_line":"Currently, if operator change the backend\u0027s ``availability_zone``, the old volume,"},{"line_number":21,"context_line":"backup and snapshot\u0027s ``availability_zone`` won\u0027t be updated after cinder-volume"},{"line_number":22,"context_line":"service restart, but the cinder-volume service\u0027s ``availability_zone`` has been"},{"line_number":23,"context_line":"updated."},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"The ``availability_zone`` of the old resources is no longer correct, and the end"}],"source_content_type":"text/x-rst","patch_set":21,"id":"7c9de4e3_f73342ac","line":22,"range":{"start_line":22,"start_character":8,"end_line":22,"end_character":15},"updated":"2022-09-15 12:48:14.000000000","message":"nit: restarts","commit_id":"8efa0495598058bc3aaed2b8dfb2d27b0fd4c7ab"},{"author":{"_account_id":13425,"name":"Simon Dodsley","email":"simon@purestorage.com","username":"sdodsley"},"change_message_id":"442059cd48d2c233c7ff307d88e2ae3e99bc5ad2","unresolved":true,"context_lines":[{"line_number":22,"context_line":"service restart, but the cinder-volume service\u0027s ``availability_zone`` has been"},{"line_number":23,"context_line":"updated."},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"The ``availability_zone`` of the old resources is no longer correct, and the end"},{"line_number":26,"context_line":"user can\u0027t update it. And this will affect the subsequent operation of"},{"line_number":27,"context_line":"these resources."},{"line_number":28,"context_line":""}],"source_content_type":"text/x-rst","patch_set":21,"id":"8caa0443_10985ea9","line":25,"range":{"start_line":25,"start_character":47,"end_line":25,"end_character":50},"updated":"2022-09-15 12:48:14.000000000","message":"nit: arenit: are","commit_id":"8efa0495598058bc3aaed2b8dfb2d27b0fd4c7ab"},{"author":{"_account_id":13425,"name":"Simon Dodsley","email":"simon@purestorage.com","username":"sdodsley"},"change_message_id":"442059cd48d2c233c7ff307d88e2ae3e99bc5ad2","unresolved":true,"context_lines":[{"line_number":23,"context_line":"updated."},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"The ``availability_zone`` of the old resources is no longer correct, and the end"},{"line_number":26,"context_line":"user can\u0027t update it. And this will affect the subsequent operation of"},{"line_number":27,"context_line":"these resources."},{"line_number":28,"context_line":""},{"line_number":29,"context_line":"E.g., a snapshot was created from the old volume with it\u0027s backend"}],"source_content_type":"text/x-rst","patch_set":21,"id":"7ec7793c_272a5991","line":26,"range":{"start_line":26,"start_character":22,"end_line":26,"end_character":27},"updated":"2022-09-15 12:48:14.000000000","message":"nit: This","commit_id":"8efa0495598058bc3aaed2b8dfb2d27b0fd4c7ab"},{"author":{"_account_id":13425,"name":"Simon Dodsley","email":"simon@purestorage.com","username":"sdodsley"},"change_message_id":"442059cd48d2c233c7ff307d88e2ae3e99bc5ad2","unresolved":true,"context_lines":[{"line_number":23,"context_line":"updated."},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"The ``availability_zone`` of the old resources is no longer correct, and the end"},{"line_number":26,"context_line":"user can\u0027t update it. And this will affect the subsequent operation of"},{"line_number":27,"context_line":"these resources."},{"line_number":28,"context_line":""},{"line_number":29,"context_line":"E.g., a snapshot was created from the old volume with it\u0027s backend"}],"source_content_type":"text/x-rst","patch_set":21,"id":"0cbd8980_a2680e95","line":26,"range":{"start_line":26,"start_character":18,"end_line":26,"end_character":20},"updated":"2022-09-15 12:48:14.000000000","message":"nit: them","commit_id":"8efa0495598058bc3aaed2b8dfb2d27b0fd4c7ab"},{"author":{"_account_id":13425,"name":"Simon Dodsley","email":"simon@purestorage.com","username":"sdodsley"},"change_message_id":"442059cd48d2c233c7ff307d88e2ae3e99bc5ad2","unresolved":true,"context_lines":[{"line_number":26,"context_line":"user can\u0027t update it. And this will affect the subsequent operation of"},{"line_number":27,"context_line":"these resources."},{"line_number":28,"context_line":""},{"line_number":29,"context_line":"E.g., a snapshot was created from the old volume with it\u0027s backend"},{"line_number":30,"context_line":"``availability_zone`` was changed, the ``availability_zone`` will follow the"},{"line_number":31,"context_line":"snapshot volume\u0027s. This can cause back-end scheduling failures when create"},{"line_number":32,"context_line":"volume from snapshot, because the ``availability_zone`` not match."},{"line_number":33,"context_line":""},{"line_number":34,"context_line":"Use Cases"},{"line_number":35,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":36,"context_line":""}],"source_content_type":"text/x-rst","patch_set":21,"id":"fb158b8f_d6e639f6","line":33,"range":{"start_line":29,"start_character":0,"end_line":33,"end_character":0},"updated":"2022-09-15 12:48:14.000000000","message":"please check you context and tenses here. Doesn\u0027t make sense","commit_id":"8efa0495598058bc3aaed2b8dfb2d27b0fd4c7ab"},{"author":{"_account_id":13425,"name":"Simon Dodsley","email":"simon@purestorage.com","username":"sdodsley"},"change_message_id":"442059cd48d2c233c7ff307d88e2ae3e99bc5ad2","unresolved":true,"context_lines":[{"line_number":35,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":36,"context_line":""},{"line_number":37,"context_line":"As an operator, I would like to change the backend ``availability_zone``. At the"},{"line_number":38,"context_line":"same time, the ``availability_zone`` of the volume and backup can be updated by"},{"line_number":39,"context_line":"the cinder-manage tool"},{"line_number":40,"context_line":""},{"line_number":41,"context_line":"Proposed change"}],"source_content_type":"text/x-rst","patch_set":21,"id":"e480dcc2_42ec2dce","line":38,"range":{"start_line":38,"start_character":62,"end_line":38,"end_character":66},"updated":"2022-09-15 12:48:14.000000000","message":"should","commit_id":"8efa0495598058bc3aaed2b8dfb2d27b0fd4c7ab"},{"author":{"_account_id":13425,"name":"Simon Dodsley","email":"simon@purestorage.com","username":"sdodsley"},"change_message_id":"442059cd48d2c233c7ff307d88e2ae3e99bc5ad2","unresolved":true,"context_lines":[{"line_number":36,"context_line":""},{"line_number":37,"context_line":"As an operator, I would like to change the backend ``availability_zone``. At the"},{"line_number":38,"context_line":"same time, the ``availability_zone`` of the volume and backup can be updated by"},{"line_number":39,"context_line":"the cinder-manage tool"},{"line_number":40,"context_line":""},{"line_number":41,"context_line":"Proposed change"},{"line_number":42,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":21,"id":"6938283f_91785f6f","line":39,"updated":"2022-09-15 12:48:14.000000000","message":"add period","commit_id":"8efa0495598058bc3aaed2b8dfb2d27b0fd4c7ab"},{"author":{"_account_id":13425,"name":"Simon Dodsley","email":"simon@purestorage.com","username":"sdodsley"},"change_message_id":"442059cd48d2c233c7ff307d88e2ae3e99bc5ad2","unresolved":true,"context_lines":[{"line_number":41,"context_line":"Proposed change"},{"line_number":42,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":43,"context_line":""},{"line_number":44,"context_line":"This proposal would add operations in cinder-manage tool, Update when end user"},{"line_number":45,"context_line":"run the ``cinder-manage volume sync-az --backend-name \u003cstorage_backend_name\u003e``"},{"line_number":46,"context_line":"and  ``cinder-manage backup sync-az --backend-name \u003cstorage_backend_name\u003e``"},{"line_number":47,"context_line":"and  ``cinder-manage consistent-group sync-az --backend-name \u003cstorage_backend_name\u003e``"}],"source_content_type":"text/x-rst","patch_set":21,"id":"0ab3bb74_6a78809f","line":44,"range":{"start_line":44,"start_character":56,"end_line":44,"end_character":58},"updated":"2022-09-15 12:48:14.000000000","message":"nit: Period","commit_id":"8efa0495598058bc3aaed2b8dfb2d27b0fd4c7ab"},{"author":{"_account_id":13425,"name":"Simon Dodsley","email":"simon@purestorage.com","username":"sdodsley"},"change_message_id":"442059cd48d2c233c7ff307d88e2ae3e99bc5ad2","unresolved":true,"context_lines":[{"line_number":41,"context_line":"Proposed change"},{"line_number":42,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":43,"context_line":""},{"line_number":44,"context_line":"This proposal would add operations in cinder-manage tool, Update when end user"},{"line_number":45,"context_line":"run the ``cinder-manage volume sync-az --backend-name \u003cstorage_backend_name\u003e``"},{"line_number":46,"context_line":"and  ``cinder-manage backup sync-az --backend-name \u003cstorage_backend_name\u003e``"},{"line_number":47,"context_line":"and  ``cinder-manage consistent-group sync-az --backend-name \u003cstorage_backend_name\u003e``"}],"source_content_type":"text/x-rst","patch_set":21,"id":"46522d54_3df596b4","line":44,"range":{"start_line":44,"start_character":35,"end_line":44,"end_character":38},"updated":"2022-09-15 12:48:14.000000000","message":"nit: to the","commit_id":"8efa0495598058bc3aaed2b8dfb2d27b0fd4c7ab"},{"author":{"_account_id":13425,"name":"Simon Dodsley","email":"simon@purestorage.com","username":"sdodsley"},"change_message_id":"442059cd48d2c233c7ff307d88e2ae3e99bc5ad2","unresolved":true,"context_lines":[{"line_number":43,"context_line":""},{"line_number":44,"context_line":"This proposal would add operations in cinder-manage tool, Update when end user"},{"line_number":45,"context_line":"run the ``cinder-manage volume sync-az --backend-name \u003cstorage_backend_name\u003e``"},{"line_number":46,"context_line":"and  ``cinder-manage backup sync-az --backend-name \u003cstorage_backend_name\u003e``"},{"line_number":47,"context_line":"and  ``cinder-manage consistent-group sync-az --backend-name \u003cstorage_backend_name\u003e``"},{"line_number":48,"context_line":""},{"line_number":49,"context_line":"Whe run the update_az command, we must ensure that all volumes and the snapshots"}],"source_content_type":"text/x-rst","patch_set":21,"id":"1c2a9884_8feada3d","line":46,"range":{"start_line":46,"start_character":0,"end_line":46,"end_character":5},"updated":"2022-09-15 12:48:14.000000000","message":"nit: or","commit_id":"8efa0495598058bc3aaed2b8dfb2d27b0fd4c7ab"},{"author":{"_account_id":13425,"name":"Simon Dodsley","email":"simon@purestorage.com","username":"sdodsley"},"change_message_id":"442059cd48d2c233c7ff307d88e2ae3e99bc5ad2","unresolved":true,"context_lines":[{"line_number":44,"context_line":"This proposal would add operations in cinder-manage tool, Update when end user"},{"line_number":45,"context_line":"run the ``cinder-manage volume sync-az --backend-name \u003cstorage_backend_name\u003e``"},{"line_number":46,"context_line":"and  ``cinder-manage backup sync-az --backend-name \u003cstorage_backend_name\u003e``"},{"line_number":47,"context_line":"and  ``cinder-manage consistent-group sync-az --backend-name \u003cstorage_backend_name\u003e``"},{"line_number":48,"context_line":""},{"line_number":49,"context_line":"Whe run the update_az command, we must ensure that all volumes and the snapshots"},{"line_number":50,"context_line":"of it are available, because this will affect Nova\u0027s volume attach, as well as"}],"source_content_type":"text/x-rst","patch_set":21,"id":"78e81420_9e5009bb","line":47,"range":{"start_line":47,"start_character":0,"end_line":47,"end_character":5},"updated":"2022-09-15 12:48:14.000000000","message":"nit: or","commit_id":"8efa0495598058bc3aaed2b8dfb2d27b0fd4c7ab"},{"author":{"_account_id":13425,"name":"Simon Dodsley","email":"simon@purestorage.com","username":"sdodsley"},"change_message_id":"442059cd48d2c233c7ff307d88e2ae3e99bc5ad2","unresolved":true,"context_lines":[{"line_number":46,"context_line":"and  ``cinder-manage backup sync-az --backend-name \u003cstorage_backend_name\u003e``"},{"line_number":47,"context_line":"and  ``cinder-manage consistent-group sync-az --backend-name \u003cstorage_backend_name\u003e``"},{"line_number":48,"context_line":""},{"line_number":49,"context_line":"Whe run the update_az command, we must ensure that all volumes and the snapshots"},{"line_number":50,"context_line":"of it are available, because this will affect Nova\u0027s volume attach, as well as"},{"line_number":51,"context_line":"backups and consistent-group."},{"line_number":52,"context_line":""}],"source_content_type":"text/x-rst","patch_set":21,"id":"f98545ff_e7b93a8e","line":49,"range":{"start_line":49,"start_character":0,"end_line":49,"end_character":7},"updated":"2022-09-15 12:48:14.000000000","message":"When we run","commit_id":"8efa0495598058bc3aaed2b8dfb2d27b0fd4c7ab"},{"author":{"_account_id":13425,"name":"Simon Dodsley","email":"simon@purestorage.com","username":"sdodsley"},"change_message_id":"442059cd48d2c233c7ff307d88e2ae3e99bc5ad2","unresolved":true,"context_lines":[{"line_number":48,"context_line":""},{"line_number":49,"context_line":"Whe run the update_az command, we must ensure that all volumes and the snapshots"},{"line_number":50,"context_line":"of it are available, because this will affect Nova\u0027s volume attach, as well as"},{"line_number":51,"context_line":"backups and consistent-group."},{"line_number":52,"context_line":""},{"line_number":53,"context_line":"when we execute the command, first we will check the volumes\u0027s and snapshots\u0027s"},{"line_number":54,"context_line":"status, if all status are available, we will check whether the backend\u0027s az"}],"source_content_type":"text/x-rst","patch_set":21,"id":"a311a5bd_7235fcf5","line":51,"range":{"start_line":51,"start_character":12,"end_line":51,"end_character":22},"updated":"2022-09-15 12:48:14.000000000","message":"nit: consistency","commit_id":"8efa0495598058bc3aaed2b8dfb2d27b0fd4c7ab"},{"author":{"_account_id":13425,"name":"Simon Dodsley","email":"simon@purestorage.com","username":"sdodsley"},"change_message_id":"442059cd48d2c233c7ff307d88e2ae3e99bc5ad2","unresolved":true,"context_lines":[{"line_number":50,"context_line":"of it are available, because this will affect Nova\u0027s volume attach, as well as"},{"line_number":51,"context_line":"backups and consistent-group."},{"line_number":52,"context_line":""},{"line_number":53,"context_line":"when we execute the command, first we will check the volumes\u0027s and snapshots\u0027s"},{"line_number":54,"context_line":"status, if all status are available, we will check whether the backend\u0027s az"},{"line_number":55,"context_line":"consistent with the volumes\u0027s, if not, we update the volumes\u0027s."},{"line_number":56,"context_line":"as well as backups and consistent-group."}],"source_content_type":"text/x-rst","patch_set":21,"id":"a58d8b31_fffbdb11","line":53,"updated":"2022-09-15 12:48:14.000000000","message":"nit: Capital W","commit_id":"8efa0495598058bc3aaed2b8dfb2d27b0fd4c7ab"},{"author":{"_account_id":13425,"name":"Simon Dodsley","email":"simon@purestorage.com","username":"sdodsley"},"change_message_id":"442059cd48d2c233c7ff307d88e2ae3e99bc5ad2","unresolved":true,"context_lines":[{"line_number":52,"context_line":""},{"line_number":53,"context_line":"when we execute the command, first we will check the volumes\u0027s and snapshots\u0027s"},{"line_number":54,"context_line":"status, if all status are available, we will check whether the backend\u0027s az"},{"line_number":55,"context_line":"consistent with the volumes\u0027s, if not, we update the volumes\u0027s."},{"line_number":56,"context_line":"as well as backups and consistent-group."},{"line_number":57,"context_line":""},{"line_number":58,"context_line":"Alternatives"}],"source_content_type":"text/x-rst","patch_set":21,"id":"2d5bc4d1_a2ae68cb","line":55,"range":{"start_line":55,"start_character":0,"end_line":55,"end_character":11},"updated":"2022-09-15 12:48:14.000000000","message":"is consistent","commit_id":"8efa0495598058bc3aaed2b8dfb2d27b0fd4c7ab"},{"author":{"_account_id":13425,"name":"Simon Dodsley","email":"simon@purestorage.com","username":"sdodsley"},"change_message_id":"442059cd48d2c233c7ff307d88e2ae3e99bc5ad2","unresolved":true,"context_lines":[{"line_number":53,"context_line":"when we execute the command, first we will check the volumes\u0027s and snapshots\u0027s"},{"line_number":54,"context_line":"status, if all status are available, we will check whether the backend\u0027s az"},{"line_number":55,"context_line":"consistent with the volumes\u0027s, if not, we update the volumes\u0027s."},{"line_number":56,"context_line":"as well as backups and consistent-group."},{"line_number":57,"context_line":""},{"line_number":58,"context_line":"Alternatives"},{"line_number":59,"context_line":"------------"}],"source_content_type":"text/x-rst","patch_set":21,"id":"f455c9e8_f6f78f06","line":56,"range":{"start_line":56,"start_character":23,"end_line":56,"end_character":33},"updated":"2022-09-15 12:48:14.000000000","message":"nit: consistency-group","commit_id":"8efa0495598058bc3aaed2b8dfb2d27b0fd4c7ab"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"2f5f0729b11f2ae2f09e0b93145660f4e893c7f6","unresolved":true,"context_lines":[{"line_number":96,"context_line":"None"},{"line_number":97,"context_line":""},{"line_number":98,"context_line":"Other deployer impact"},{"line_number":99,"context_line":"---------------------"},{"line_number":100,"context_line":""},{"line_number":101,"context_line":"None"},{"line_number":102,"context_line":""}],"source_content_type":"text/x-rst","patch_set":21,"id":"64e0be60_e30062f9","line":99,"updated":"2026-02-23 13:58:38.000000000","message":"Don\u0027t forget about the volume-type issue I mentioned on PS15.","commit_id":"8efa0495598058bc3aaed2b8dfb2d27b0fd4c7ab"}]}