)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":16137,"name":"Tobias Urdin","email":"tobias.urdin@binero.com","username":"tobasco"},"change_message_id":"8ee3d8bfd92a3d48bee84edf05f5794fd74bc23d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"4b151ae9_178ee013","updated":"2024-08-14 09:23:15.000000000","message":"rebased on the api-ref for os-availability-zones that has been missing for atleast 6 years – setting this as WIP because I need to update the api-ref in this patch","commit_id":"ab4d4208491fb6491ecdf0de73df49bdd9061411"},{"author":{"_account_id":16137,"name":"Tobias Urdin","email":"tobias.urdin@binero.com","username":"tobasco"},"change_message_id":"5caf37fe949674fbad187ac0495011db0d99ad06","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"ea3ba935_ace14cee","updated":"2024-08-22 08:05:58.000000000","message":"recheck releasenotes job fixed in openstack-zuul-jobs","commit_id":"e3ce8c0e6bc90515c44b8aed3bf591c4b8648e6b"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"56238def57108ee77de2feb7d83193f305b74628","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"0e85d2b6_140db491","updated":"2024-08-30 19:20:59.000000000","message":"I need to look at this more closely.  Not sure if there\u0027s a precedent for using a query parameter that way in the block storage API (not that it\u0027s necessarily bad, will just require a bit of thought).","commit_id":"60b93d96116b5c8470c21d4ca42f75a74cba1ca5"},{"author":{"_account_id":16137,"name":"Tobias Urdin","email":"tobias.urdin@binero.com","username":"tobasco"},"change_message_id":"69d32acf76ef0818518a7c916e82cf84dc3a2cae","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"73f7d845_f1390fca","in_reply_to":"0e85d2b6_140db491","updated":"2024-09-02 09:39:09.000000000","message":"Ack, let me know and I\u0027ll adjust accordingly if there is any feedback on that.","commit_id":"60b93d96116b5c8470c21d4ca42f75a74cba1ca5"},{"author":{"_account_id":9236,"name":"Jon Bernard","email":"jobernar@redhat.com","username":"jbernard"},"change_message_id":"10f1a4558076fe0e56a7e5c89e135fb049432451","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":12,"id":"40d5a1a3_cb6f9971","updated":"2025-08-07 16:00:02.000000000","message":"This looks okay to me.  Appears to be implemented properly, the fallback preserves existing behavior, tests look good, passes CI.","commit_id":"b74e8828e41885307618ea709cf7c51e3c2c83e4"},{"author":{"_account_id":38081,"name":"Anthony Galica","display_name":"agalica","email":"anthony.galica@hitachivantara.com","username":"agalica","status":"Hitachi Vantara"},"change_message_id":"d4a422a6832347c39d9d26d7f6038e6db288e949","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":13,"id":"f2841186_5b30d99d","updated":"2025-10-22 23:20:57.000000000","message":"This patch is looking good to me.  Just one minor question about the implementation.","commit_id":"460f599abddc1d644a3c7284efdad8d8dadb5faa"},{"author":{"_account_id":16137,"name":"Tobias Urdin","email":"tobias.urdin@binero.com","username":"tobasco"},"change_message_id":"d21a71aa3d353d3fb9980c24e2a4088fc04abfbd","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":13,"id":"f1a38e1b_07ed41f2","updated":"2025-09-01 14:19:48.000000000","message":"recheck post_failure","commit_id":"460f599abddc1d644a3c7284efdad8d8dadb5faa"},{"author":{"_account_id":16137,"name":"Tobias Urdin","email":"tobias.urdin@binero.com","username":"tobasco"},"change_message_id":"0133c6416e7b76f6be829ead491da9da2a63479b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":13,"id":"a7a5daff_7c8d2250","updated":"2025-09-03 06:41:54.000000000","message":"recheck unrelated neutron port tempest error","commit_id":"460f599abddc1d644a3c7284efdad8d8dadb5faa"},{"author":{"_account_id":16137,"name":"Tobias Urdin","email":"tobias.urdin@binero.com","username":"tobasco"},"change_message_id":"d834bbaf4b323acfff835a63bec7d14898d0c28b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":17,"id":"2685b2d2_1ff9ea16","updated":"2025-10-24 13:58:25.000000000","message":"fixed race condition in the FakeRequest class in testing","commit_id":"e3639748c0ab8884a31eecaf8c62371223fad6d2"},{"author":{"_account_id":13915,"name":"Silvan Kaiser","email":"silvan@quobyte.com","username":"kaisers"},"change_message_id":"778f984a7b753b79b74c40a3756253c37ee62358","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":20,"id":"97f5ece9_026db815","updated":"2026-02-19 08:54:24.000000000","message":"run-Quobyte CI","commit_id":"0e4175e53de71fcd258e2a8daa9d792187dffdda"}],"cinder/api/contrib/availability_zones.py":[{"author":{"_account_id":38081,"name":"Anthony Galica","display_name":"agalica","email":"anthony.galica@hitachivantara.com","username":"agalica","status":"Hitachi Vantara"},"change_message_id":"d4a422a6832347c39d9d26d7f6038e6db288e949","unresolved":true,"context_lines":[{"line_number":44,"context_line":""},{"line_number":45,"context_line":"        if service \u003d\u003d constants.BACKUP_TOPIC:"},{"line_number":46,"context_line":"            return self.backup_api.list_availability_zones()"},{"line_number":47,"context_line":"        else:"},{"line_number":48,"context_line":"            return self.volume_api.list_availability_zones()"},{"line_number":49,"context_line":""},{"line_number":50,"context_line":"    def index(self, req):"}],"source_content_type":"text/x-python","patch_set":13,"id":"87728b9b_c3b301fe","line":47,"updated":"2025-10-22 23:20:57.000000000","message":"Should we also check that it\u0027s the volume binary since we also handle service being None above, and throw an exception otherwise?  This way if new binaries are added we aren\u0027t doing something unexpected.","commit_id":"460f599abddc1d644a3c7284efdad8d8dadb5faa"},{"author":{"_account_id":16137,"name":"Tobias Urdin","email":"tobias.urdin@binero.com","username":"tobasco"},"change_message_id":"2624dfa0588d0a947daab2e208d375d5b1a20f77","unresolved":false,"context_lines":[{"line_number":44,"context_line":""},{"line_number":45,"context_line":"        if service \u003d\u003d constants.BACKUP_TOPIC:"},{"line_number":46,"context_line":"            return self.backup_api.list_availability_zones()"},{"line_number":47,"context_line":"        else:"},{"line_number":48,"context_line":"            return self.volume_api.list_availability_zones()"},{"line_number":49,"context_line":""},{"line_number":50,"context_line":"    def index(self, req):"}],"source_content_type":"text/x-python","patch_set":13,"id":"a110105e_8e086259","line":47,"in_reply_to":"87728b9b_c3b301fe","updated":"2025-10-23 08:00:08.000000000","message":"Thanks for the review, I\u0027ve updated it to be even more explicit.","commit_id":"460f599abddc1d644a3c7284efdad8d8dadb5faa"}]}