)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"860961bf0d5ea9f91f1920f4eed4e828faec7de6","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"97b0c156_8c816986","updated":"2026-04-27 19:19:39.000000000","message":"This is similar to the code that\u0027s used for volumes (as opposed to backups), which is fine, but I want to mention that there\u0027s a patch up addressing a possible flaw in the volume code: https://review.opendev.org/c/openstack/cinder/+/979641\n\nThat patch is still under discussion, but you might want to take a look and see if you think the issue might apply here, since the code is so similar.","commit_id":"40d9b4dd5141bca2294626a3eca7d71bf83bc430"},{"author":{"_account_id":32666,"name":"Damian Dąbrowski","email":"damian@dabrowski.cloud","username":"ddabrowski"},"change_message_id":"26f5deef34efe243edf18c56963018aacafca484","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"5d73d86c_23eb77d3","updated":"2026-03-11 21:58:28.000000000","message":"recheck\n\n- cinder-plugin-ceph-tempest failed on: capture-performance-data: Generate statistics\n- devstack-plugin-nfs-tempest-full failed on tempest tests(a lot of `No valid backend was found. No weighed backends available` in logs)","commit_id":"40d9b4dd5141bca2294626a3eca7d71bf83bc430"},{"author":{"_account_id":32666,"name":"Damian Dąbrowski","email":"damian@dabrowski.cloud","username":"ddabrowski"},"change_message_id":"b2362e4ea9154568ba0bd5f531a0c0a064bf8cb9","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"d5913593_52f58c55","in_reply_to":"97b0c156_8c816986","updated":"2026-05-12 13:41:49.000000000","message":"thanks, I looks like #979641 is invalid.\nDespite this, I tried to reproduce the issue mentioned there but I couldn\u0027t.","commit_id":"40d9b4dd5141bca2294626a3eca7d71bf83bc430"}],"cinder/tests/unit/scheduler/test_host_manager.py":[{"author":{"_account_id":36171,"name":"jayaanand borra","display_name":"jayaanand borra","email":"jayaanand.borra@netapp.com","username":"jayaanan","status":"netapp"},"change_message_id":"db214fd4c063223949066f99ab9770dcfae76e8f","unresolved":true,"context_lines":[{"line_number":1285,"context_line":"                \u0027get_all_by_topic\u0027)"},{"line_number":1286,"context_line":"    @mock.patch(\u0027cinder.scheduler.host_manager.cinder_context.\u0027"},{"line_number":1287,"context_line":"                \u0027get_admin_context\u0027)"},{"line_number":1288,"context_line":"    def test_list_backup_services_filters_down_services("},{"line_number":1289,"context_line":"            self, mock_get_admin_context, mock_get_all_by_topic):"},{"line_number":1290,"context_line":"        mock_get_admin_context.return_value \u003d mock.sentinel.context"},{"line_number":1291,"context_line":"        self.host_manager.backup_service_states \u003d {"}],"source_content_type":"text/x-python","patch_set":1,"id":"90e0bef9_c7738177","line":1288,"updated":"2026-04-08 15:24:03.000000000","message":"missing tests for capabilities is None and if backend_key in active_backends: continue. other wise LGTM","commit_id":"40d9b4dd5141bca2294626a3eca7d71bf83bc430"},{"author":{"_account_id":32666,"name":"Damian Dąbrowski","email":"damian@dabrowski.cloud","username":"ddabrowski"},"change_message_id":"b2362e4ea9154568ba0bd5f531a0c0a064bf8cb9","unresolved":false,"context_lines":[{"line_number":1285,"context_line":"                \u0027get_all_by_topic\u0027)"},{"line_number":1286,"context_line":"    @mock.patch(\u0027cinder.scheduler.host_manager.cinder_context.\u0027"},{"line_number":1287,"context_line":"                \u0027get_admin_context\u0027)"},{"line_number":1288,"context_line":"    def test_list_backup_services_filters_down_services("},{"line_number":1289,"context_line":"            self, mock_get_admin_context, mock_get_all_by_topic):"},{"line_number":1290,"context_line":"        mock_get_admin_context.return_value \u003d mock.sentinel.context"},{"line_number":1291,"context_line":"        self.host_manager.backup_service_states \u003d {"}],"source_content_type":"text/x-python","patch_set":1,"id":"bce38343_77fc005d","line":1288,"in_reply_to":"90e0bef9_c7738177","updated":"2026-05-12 13:41:49.000000000","message":"thanks, I added two tests:\n\n```\ncinder.tests.unit.scheduler.test_host_manager.HostManagerTestCase.test_update_backup_backend_state_map_skips_missing_capabilities\ncinder.tests.unit.scheduler.test_host_manager.HostManagerTestCase.test_update_backup_backend_state_map_skips_duplicate_backend\n```","commit_id":"40d9b4dd5141bca2294626a3eca7d71bf83bc430"}]}