)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":36439,"name":"Vivek Pandey","email":"vivek.pandey11@ibm.com","username":"vivek.pandey11"},"change_message_id":"d4939a53dbfb14e93db86fd6b622982b18f64efd","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":26,"id":"34e1d3ff_a2d47d63","updated":"2026-02-11 11:04:17.000000000","message":"LGTM","commit_id":"e42ece463bad83876cb0a90850a828175ca68327"}],"cinder/tests/unit/volume/drivers/ibm/test_storwize_svc.py":[{"author":{"_account_id":36171,"name":"jayaanand borra","display_name":"jayaanand borra","email":"jayaanand.borra@netapp.com","username":"jayaanan","status":"netapp"},"change_message_id":"eb9eea94bb77be042269aa176f24dc224eac0791","unresolved":true,"context_lines":[{"line_number":6026,"context_line":"        else:"},{"line_number":6027,"context_line":"            self.assertFalse(get_hs_storage_state.called)"},{"line_number":6028,"context_line":""},{"line_number":6029,"context_line":"    def test_update_volume_stats_pbha(self):"},{"line_number":6030,"context_line":"        self.driver.partition_name \u003d \"test\""},{"line_number":6031,"context_line":"        self.driver.is_pbha_partition \u003d True"},{"line_number":6032,"context_line":"        self.driver._update_volume_stats()"}],"source_content_type":"text/x-python","patch_set":29,"id":"1f9aec27_2d1178bd","line":6029,"updated":"2026-04-08 09:24:41.000000000","message":"can you add tests for - preferred_management_site\n,active_management_site,ha_status and link_status","commit_id":"0df0650d48a30a36706e54ef2e04e27f66c8e14a"}],"cinder/volume/drivers/ibm/storwize_svc/storwize_svc_common.py":[{"author":{"_account_id":36171,"name":"jayaanand borra","display_name":"jayaanand borra","email":"jayaanand.borra@netapp.com","username":"jayaanan","status":"netapp"},"change_message_id":"eb9eea94bb77be042269aa176f24dc224eac0791","unresolved":true,"context_lines":[{"line_number":4228,"context_line":"            return partition[0][\u0027location2_system_id\u0027]"},{"line_number":4229,"context_line":"        else:"},{"line_number":4230,"context_line":"            return partition[0][\u0027location1_system_id\u0027]"},{"line_number":4231,"context_line":""},{"line_number":4232,"context_line":"    def get_partition_details(self, partition_name):"},{"line_number":4233,"context_line":"        return self.ssh.lspartition(partition_name)"},{"line_number":4234,"context_line":""}],"source_content_type":"text/x-python","patch_set":29,"id":"f9785cf1_a08f711f","line":4231,"updated":"2026-04-08 09:24:41.000000000","message":"pls add tests for VolumeBackendAPIException","commit_id":"0df0650d48a30a36706e54ef2e04e27f66c8e14a"},{"author":{"_account_id":36171,"name":"jayaanand borra","display_name":"jayaanand borra","email":"jayaanand.borra@netapp.com","username":"jayaanan","status":"netapp"},"change_message_id":"eb9eea94bb77be042269aa176f24dc224eac0791","unresolved":true,"context_lines":[{"line_number":8245,"context_line":"                            \"Exception: %(ex)s.\", {\u0027ex\u0027: exc.msg})"},{"line_number":8246,"context_line":""},{"line_number":8247,"context_line":"        if self.is_pbha_partition:"},{"line_number":8248,"context_line":"            data[\u00272-site-ha_enabled\u0027] \u003d True"},{"line_number":8249,"context_line":"            try:"},{"line_number":8250,"context_line":"                partition \u003d self._helpers.get_partition_details("},{"line_number":8251,"context_line":"                    self.partition_name)"}],"source_content_type":"text/x-python","patch_set":29,"id":"9a58e9ea_655b0bd6","line":8248,"updated":"2026-04-08 09:24:41.000000000","message":"data[\u00272-site-ha_enabled\u0027] \u003d True is set unconditionally when self.is_pbha_partition is True, even if the subsequent get_partition_details() call fails. This means:\n- The scheduler/API will show 2-site-ha_enabled: True even when the actual PBHA partition details cannot be retrieved?","commit_id":"0df0650d48a30a36706e54ef2e04e27f66c8e14a"}]}
