)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":597,"name":"Pete Zaitcev","email":"zaitcev@kotori.zaitcev.us","username":"zaitcev"},"change_message_id":"53daf761a8f755f8ac3f8b09a3317f5594ea3b51","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"008587ee_a9c5f091","updated":"2024-03-15 14:44:36.000000000","message":"I\u0027m okay with this, but I have an extra suggestion:\nhttps://review.opendev.org/c/openstack/cinder/+/913391","commit_id":"939f93fda2c539b8fb86c05b6a9a569507f17cf8"},{"author":{"_account_id":25468,"name":"Michel Nederlof","email":"michel@nederlof.info","username":"pellucid"},"change_message_id":"67a318ed06955ddb4c258b53994c41b291936103","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"7272ce32_2b48ce1a","updated":"2024-03-15 15:01:37.000000000","message":"Thanks for you reviews!","commit_id":"939f93fda2c539b8fb86c05b6a9a569507f17cf8"},{"author":{"_account_id":25468,"name":"Michel Nederlof","email":"michel@nederlof.info","username":"pellucid"},"change_message_id":"f962f67c296896bf2f8f85731e8a952a0dc05710","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"a7cf6f9f_b43cdb71","updated":"2024-03-11 16:38:29.000000000","message":"recheck","commit_id":"939f93fda2c539b8fb86c05b6a9a569507f17cf8"},{"author":{"_account_id":25468,"name":"Michel Nederlof","email":"michel@nederlof.info","username":"pellucid"},"change_message_id":"35357b8d126316a0404d4745658e20323bb58319","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"e2b31519_575a4ff8","updated":"2024-03-07 12:36:03.000000000","message":"recheck","commit_id":"939f93fda2c539b8fb86c05b6a9a569507f17cf8"},{"author":{"_account_id":25468,"name":"Michel Nederlof","email":"michel@nederlof.info","username":"pellucid"},"change_message_id":"618a759bb22bb73f323f5ff13bfeb3b7b9e7b354","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"e3a1843f_1b15886f","updated":"2024-03-11 09:06:14.000000000","message":"recheck","commit_id":"939f93fda2c539b8fb86c05b6a9a569507f17cf8"},{"author":{"_account_id":25468,"name":"Michel Nederlof","email":"michel@nederlof.info","username":"pellucid"},"change_message_id":"a66c1596f7f9d83555e1d0a4aa07d5d3bb8068d4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"c8074019_236624df","updated":"2024-03-11 12:16:44.000000000","message":"recheck\n\nfailure seems not related","commit_id":"939f93fda2c539b8fb86c05b6a9a569507f17cf8"},{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"29c78bfa1061d58db98699648db53154b8962d46","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"8ab5be17_d11773ab","updated":"2024-03-15 14:39:59.000000000","message":"this requires a releasenote for the bugfix and a UT","commit_id":"939f93fda2c539b8fb86c05b6a9a569507f17cf8"},{"author":{"_account_id":25468,"name":"Michel Nederlof","email":"michel@nederlof.info","username":"pellucid"},"change_message_id":"67a318ed06955ddb4c258b53994c41b291936103","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"bda364ba_1d188f84","in_reply_to":"008587ee_a9c5f091","updated":"2024-03-15 15:01:37.000000000","message":"I am a little bit conflicted here, yes, it might make sense to use the value of what is stored in the database object, but the actual debug message is printed by the service, which itself is part of a specific AZ.\n\nI\u0027d opt to discuss this in the separate patch you provided and not include it in this patch.","commit_id":"939f93fda2c539b8fb86c05b6a9a569507f17cf8"},{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"c4ca914be00c8c4259e6e5c06c001ab82eb04f29","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"358cfdbd_59f5de3e","in_reply_to":"1a580f05_1a875092","updated":"2024-03-15 15:13:04.000000000","message":"sorry for not being verbose, with UT i mean unit test.","commit_id":"939f93fda2c539b8fb86c05b6a9a569507f17cf8"},{"author":{"_account_id":25468,"name":"Michel Nederlof","email":"michel@nederlof.info","username":"pellucid"},"change_message_id":"30c3e9f67d2ae9c0164081d73df3ccc736aedbf1","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"1dae7f97_eda2960c","in_reply_to":"358cfdbd_59f5de3e","updated":"2024-03-15 15:17:42.000000000","message":"Ah, yes, i will add a unit for this scenario.","commit_id":"939f93fda2c539b8fb86c05b6a9a569507f17cf8"},{"author":{"_account_id":25468,"name":"Michel Nederlof","email":"michel@nederlof.info","username":"pellucid"},"change_message_id":"67a318ed06955ddb4c258b53994c41b291936103","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"1a580f05_1a875092","in_reply_to":"8ab5be17_d11773ab","updated":"2024-03-15 15:01:37.000000000","message":"I will add the bugfix releasenote, but can you elaborate on what you mean with \u0027UT\u0027 ? I am unfamiliar with this abbreviation.","commit_id":"939f93fda2c539b8fb86c05b6a9a569507f17cf8"},{"author":{"_account_id":25468,"name":"Michel Nederlof","email":"michel@nederlof.info","username":"pellucid"},"change_message_id":"bcce7dd6fba622d93742f37085dd91f97e781757","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"f671db90_65edeebc","updated":"2024-08-16 09:34:51.000000000","message":"Body: b\u0027{\"forbidden\": {\"code\": 403, \"message\": \"Quota exceeded for instances: Requested 1, but already used 10 of 10 instances\"}}\u0027\n\nSo i guess we need to wait for someone to fix the cinder-plugin-ceph-tempest job..","commit_id":"9b7eaf2e328400b2ad00a93d8cd602ebcce63731"},{"author":{"_account_id":25468,"name":"Michel Nederlof","email":"michel@nederlof.info","username":"pellucid"},"change_message_id":"c6c8010d3790c55d90acfbc536373edbc42226ab","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"bb4b0fc3_54cd6a4a","updated":"2024-03-19 15:44:43.000000000","message":"Unittests and releasenote has been added.","commit_id":"9b7eaf2e328400b2ad00a93d8cd602ebcce63731"},{"author":{"_account_id":16137,"name":"Tobias Urdin","email":"tobias.urdin@binero.com","username":"tobasco"},"change_message_id":"ca081ffcd548478914f65e7e314d8f65bbb74fc6","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"5f6ae32b_ae518505","updated":"2024-04-22 07:59:28.000000000","message":"i guess one question here is, what is the intended design for cinder backup, should no AZ mean any AZ or only schedule to one AZ\n\nin nova no AZ means it can be used and moved from/to anywhere, but I agree with this patch that for Cinder Backup it should be \"give no AZ because you can place it anywhere, but then save where it is\" – it shouldn\u0027t be transparent when it comes to data safety \n\n[1] https://review.opendev.org/c/openstack/cinder/+/785099","commit_id":"9b7eaf2e328400b2ad00a93d8cd602ebcce63731"},{"author":{"_account_id":25468,"name":"Michel Nederlof","email":"michel@nederlof.info","username":"pellucid"},"change_message_id":"9770e57e1383d26a96399d259314cc733c2b133e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"ab1646bc_ccb46149","updated":"2024-08-20 07:17:39.000000000","message":"recheck","commit_id":"9b7eaf2e328400b2ad00a93d8cd602ebcce63731"},{"author":{"_account_id":25468,"name":"Michel Nederlof","email":"michel@nederlof.info","username":"pellucid"},"change_message_id":"dcf4021c34e08fa6bdfe3bcbaa964dbf1214cd6b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"f2e5607e_e113879b","updated":"2024-08-16 07:28:00.000000000","message":"recheck","commit_id":"9b7eaf2e328400b2ad00a93d8cd602ebcce63731"},{"author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"change_message_id":"5ea415ed8ea8c535d4697bfcd63c7b4fb134748d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"cd2bb064_ee7d4a64","updated":"2024-08-27 09:42:36.000000000","message":"recheck - tempest failure doesn\u0027t look related to this change.","commit_id":"9b7eaf2e328400b2ad00a93d8cd602ebcce63731"},{"author":{"_account_id":16137,"name":"Tobias Urdin","email":"tobias.urdin@binero.com","username":"tobasco"},"change_message_id":"d04ea6ca3ea751e348b194af5a3f63bd3480c9ba","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"7c57f62a_f994f389","in_reply_to":"5f6ae32b_ae518505","updated":"2024-04-22 08:00:57.000000000","message":"with [1] I was going to refer that selecting an AZ will now work and save where it stored","commit_id":"9b7eaf2e328400b2ad00a93d8cd602ebcce63731"},{"author":{"_account_id":25468,"name":"Michel Nederlof","email":"michel@nederlof.info","username":"pellucid"},"change_message_id":"38c9a9934631f1f8a84c86ec106e0364e254044f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"adcf4f18_999ab375","in_reply_to":"7c57f62a_f994f389","updated":"2024-04-23 09:06:43.000000000","message":"Yes i agree, though this patch has nothing to do with how backups are scheduled (that is where your referred link is for). \n\nThis patch just consistently stores the availability_zone property of the backup record, just like the availability_zone field is also always populated for volumes.\n\nThen, when a restore is requested, the appropriate backup service can be selected using the scheduler.","commit_id":"9b7eaf2e328400b2ad00a93d8cd602ebcce63731"}]}