)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"cbdcb879909e4dba682f66bd6ede20d1e4ba8cfa","unresolved":true,"context_lines":[{"line_number":4,"context_line":"Commit:     Damian Dabrowski \u003cdamian.dabrowski@cleura.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2026-02-26 21:30:12 +0100"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Implement backup_default_availability_zone variable"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"When creating backup, cinder-scheduler reads `availability_zone`"},{"line_number":10,"context_line":"parameter(if it\u0027s defined) and looks for cinder-backup service in that"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"84efb90d_5d16c11a","line":7,"updated":"2026-06-02 19:01:17.000000000","message":"This would be better as:\n\nAdd backup_default_availability_zone config opt","commit_id":"3dcda844ae73ddb06993c2bbcffd9829f52f216d"},{"author":{"_account_id":32666,"name":"Damian Dąbrowski","email":"damian@dabrowski.cloud","username":"ddabrowski"},"change_message_id":"a1b66f5611091500b595461e1153a4da63f7906b","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Damian Dabrowski \u003cdamian.dabrowski@cleura.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2026-02-26 21:30:12 +0100"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Implement backup_default_availability_zone variable"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"When creating backup, cinder-scheduler reads `availability_zone`"},{"line_number":10,"context_line":"parameter(if it\u0027s defined) and looks for cinder-backup service in that"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"501d8713_feb5fa29","line":7,"in_reply_to":"84efb90d_5d16c11a","updated":"2026-06-03 10:01:35.000000000","message":"Done","commit_id":"3dcda844ae73ddb06993c2bbcffd9829f52f216d"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"cbdcb879909e4dba682f66bd6ede20d1e4ba8cfa","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"a8755c65_ac989df9","updated":"2026-06-02 19:01:17.000000000","message":"Looks good! A few minor points noted inline, but otherwise this is fine.","commit_id":"3dcda844ae73ddb06993c2bbcffd9829f52f216d"},{"author":{"_account_id":32666,"name":"Damian Dąbrowski","email":"damian@dabrowski.cloud","username":"ddabrowski"},"change_message_id":"686985894911f77764ea1e7f8c07b260dd1e1e3a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"bb380aee_cfdd3315","updated":"2026-03-06 15:55:25.000000000","message":"recheck - now cinder-plugin-ceph-tempest failed (it didn\u0027t fail before)","commit_id":"3dcda844ae73ddb06993c2bbcffd9829f52f216d"},{"author":{"_account_id":32666,"name":"Damian Dąbrowski","email":"damian@dabrowski.cloud","username":"ddabrowski"},"change_message_id":"38bb2417c6644ddf23d3387910d33b960b78a53d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"b4f463e6_4d081049","updated":"2026-03-06 13:06:29.000000000","message":"recheck - tempest failed in a really unexpected way...it doesn\u0027t seem related to this patch. In logs I see msgs like:\n\n```\nFeb 26 22:11:05.627659 np8ba8b88085fb4 cinder-scheduler[88212]: ERROR cinder.scheduler.flows.create_volume [None req-7e7e5f01-81db-4622-9a67-bd2d4f386542 tempest-VolumeAndVolumeTypeFromImageTest-1494382889 None] Failed to run task cinder.scheduler.flows.create_volume.ScheduleCreateVolumeTask;volume:create: No valid backend was found. No weighed backends available: cinder.exception.NoValidBackend: No valid backend was found. No weighed backends available\n```\n\n```\nFeb 26 21:31:42.030620 np8ba8b88085fb4 devstack@c-api.service[87434]: ERROR cinder.volume.api [None req-b94bd1c9-0fba-40d7-92c6-34efb021a457 tempest-VolumeQuotasNegativeTestJSON-2009305400 tempest-VolumeQuotasNegativeTestJSON-2009305400-project-member] Quota exceeded for fe6371e4ac184f7ab75c361ada49e303, tried to extend volume by 1G, (1G of 1G already consumed).: cinder.exception.OverQuota: Quota exceeded for resources: [\u0027gigabytes\u0027]\n```","commit_id":"3dcda844ae73ddb06993c2bbcffd9829f52f216d"},{"author":{"_account_id":32666,"name":"Damian Dąbrowski","email":"damian@dabrowski.cloud","username":"ddabrowski"},"change_message_id":"a1b66f5611091500b595461e1153a4da63f7906b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"e14eaf39_289c69e9","in_reply_to":"a8755c65_ac989df9","updated":"2026-06-03 10:01:35.000000000","message":"thanks for the review! I addressed all your comments.","commit_id":"3dcda844ae73ddb06993c2bbcffd9829f52f216d"},{"author":{"_account_id":9236,"name":"Jon Bernard","email":"jobernar@redhat.com","username":"jbernard"},"change_message_id":"e710be644f52a96ad8954bb5afeab59ad9d4b5eb","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"2ac0a87d_8be10ed7","updated":"2026-06-03 18:47:56.000000000","message":"looks ok to me.","commit_id":"b90e8808b65b72c149ed4b2c2ec21dace2d73ecd"}],"cinder/tests/unit/scheduler/test_scheduler.py":[{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"cbdcb879909e4dba682f66bd6ede20d1e4ba8cfa","unresolved":true,"context_lines":[{"line_number":675,"context_line":"        self.assertEqual(\u0027backup_default_az\u0027, az)"},{"line_number":676,"context_line":""},{"line_number":677,"context_line":"    def test_get_az_falls_back_to_volume_az(self):"},{"line_number":678,"context_line":"        self.flags(backup_default_availability_zone\u003dNone)"},{"line_number":679,"context_line":"        volume \u003d fake_volume.fake_volume_obj("},{"line_number":680,"context_line":"            self.context, availability_zone\u003d\u0027volume_az\u0027)"},{"line_number":681,"context_line":"        hm \u003d fake_scheduler.FakeHostManager()"}],"source_content_type":"text/x-python","patch_set":1,"id":"f419ed73_492332ba","line":678,"range":{"start_line":678,"start_character":8,"end_line":678,"end_character":57},"updated":"2026-06-02 19:01:17.000000000","message":"This is a good test, but instead of setting the value to None, it might be better to leave it unset and just use the default.  That way, if someone monkeys with the default value, this test will break, which is good, because we want the default to be None to preserve the standard behavior.  (Because you\u0027re using the \"flags\" function in the previous test, the change in value there won\u0027t affect this test.)","commit_id":"3dcda844ae73ddb06993c2bbcffd9829f52f216d"},{"author":{"_account_id":32666,"name":"Damian Dąbrowski","email":"damian@dabrowski.cloud","username":"ddabrowski"},"change_message_id":"a1b66f5611091500b595461e1153a4da63f7906b","unresolved":false,"context_lines":[{"line_number":675,"context_line":"        self.assertEqual(\u0027backup_default_az\u0027, az)"},{"line_number":676,"context_line":""},{"line_number":677,"context_line":"    def test_get_az_falls_back_to_volume_az(self):"},{"line_number":678,"context_line":"        self.flags(backup_default_availability_zone\u003dNone)"},{"line_number":679,"context_line":"        volume \u003d fake_volume.fake_volume_obj("},{"line_number":680,"context_line":"            self.context, availability_zone\u003d\u0027volume_az\u0027)"},{"line_number":681,"context_line":"        hm \u003d fake_scheduler.FakeHostManager()"}],"source_content_type":"text/x-python","patch_set":1,"id":"e00b9b30_4391a5e4","line":678,"range":{"start_line":678,"start_character":8,"end_line":678,"end_character":57},"in_reply_to":"f419ed73_492332ba","updated":"2026-06-03 10:01:35.000000000","message":"Done","commit_id":"3dcda844ae73ddb06993c2bbcffd9829f52f216d"}],"releasenotes/notes/backup_default_availability_zone-895d6743a6d61328.yaml":[{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"cbdcb879909e4dba682f66bd6ede20d1e4ba8cfa","unresolved":true,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"features:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    Added ``backup_default_availability_zone`` variable that can be used to"},{"line_number":5,"context_line":"    define default availability zone for backups."}],"source_content_type":"text/x-yaml","patch_set":1,"id":"be4acc12_630059e8","line":4,"range":{"start_line":4,"start_character":47,"end_line":4,"end_character":55},"updated":"2026-06-02 19:01:17.000000000","message":"s/variable/configuration option/","commit_id":"3dcda844ae73ddb06993c2bbcffd9829f52f216d"},{"author":{"_account_id":32666,"name":"Damian Dąbrowski","email":"damian@dabrowski.cloud","username":"ddabrowski"},"change_message_id":"a1b66f5611091500b595461e1153a4da63f7906b","unresolved":false,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"features:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    Added ``backup_default_availability_zone`` variable that can be used to"},{"line_number":5,"context_line":"    define default availability zone for backups."}],"source_content_type":"text/x-yaml","patch_set":1,"id":"dbfe76e6_0f628fb4","line":4,"range":{"start_line":4,"start_character":47,"end_line":4,"end_character":55},"in_reply_to":"be4acc12_630059e8","updated":"2026-06-03 10:01:35.000000000","message":"Done","commit_id":"3dcda844ae73ddb06993c2bbcffd9829f52f216d"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"cbdcb879909e4dba682f66bd6ede20d1e4ba8cfa","unresolved":true,"context_lines":[{"line_number":2,"context_line":"features:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    Added ``backup_default_availability_zone`` variable that can be used to"},{"line_number":5,"context_line":"    define default availability zone for backups."}],"source_content_type":"text/x-yaml","patch_set":1,"id":"1fcca293_e7518379","line":5,"updated":"2026-06-02 19:01:17.000000000","message":"I suggest adding: for deployments where volume availability zones do not match backup availability zones.  The default value is ``None``, which respects the current behavior.\n\n(you can elaborate more if you want, this is the text that operators will see, so you want them to know both that the new option exists and what it\u0027s for)","commit_id":"3dcda844ae73ddb06993c2bbcffd9829f52f216d"},{"author":{"_account_id":32666,"name":"Damian Dąbrowski","email":"damian@dabrowski.cloud","username":"ddabrowski"},"change_message_id":"a1b66f5611091500b595461e1153a4da63f7906b","unresolved":false,"context_lines":[{"line_number":2,"context_line":"features:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    Added ``backup_default_availability_zone`` variable that can be used to"},{"line_number":5,"context_line":"    define default availability zone for backups."}],"source_content_type":"text/x-yaml","patch_set":1,"id":"808bc434_af6d5382","line":5,"in_reply_to":"1fcca293_e7518379","updated":"2026-06-03 10:01:35.000000000","message":"Done","commit_id":"3dcda844ae73ddb06993c2bbcffd9829f52f216d"}]}