)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":36171,"name":"jayaanand borra","display_name":"jayaanand borra","email":"jayaanand.borra@netapp.com","username":"jayaanan","status":"netapp"},"change_message_id":"b2327e553015ab0b1e4c0a52b3ab0bf5e4198d07","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":24,"id":"52d0150f_311f58a5","updated":"2026-04-14 04:54:38.000000000","message":"LGTM only minor issue","commit_id":"bc7854418aa4848b894bbe01f9766d974dc384fd"}],"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":"b2327e553015ab0b1e4c0a52b3ab0bf5e4198d07","unresolved":true,"context_lines":[{"line_number":5064,"context_line":"        self.assertIn(\u0027initiator_target_map\u0027, conn_info[\u0027data\u0027])"},{"line_number":5065,"context_line":""},{"line_number":5066,"context_line":"        # Scenario 2 : user defined host location"},{"line_number":5067,"context_line":"        host_site \u003d {\u0027site1\u0027: \u0027ff00000000000000\u0027, \u0027site2\u0027: \u0027ff00000000000001\u0027}"},{"line_number":5068,"context_line":"        self.fc_driver.configuration.set_override("},{"line_number":5069,"context_line":"            \u0027storwize_preferred_host_site\u0027, host_site)"},{"line_number":5070,"context_line":"        conn_info2 \u003d self.fc_driver.initialize_connection(volume_fc, connector)"}],"source_content_type":"text/x-python","patch_set":24,"id":"d6ac7fc0_65fb69c9","line":5067,"updated":"2026-04-14 04:54:38.000000000","message":"The connector has ffff prefix for WWPNs, but storwize_preferred_host_site values are ff prefixed ffff000000000000 does NOT match ff00000000000000","commit_id":"bc7854418aa4848b894bbe01f9766d974dc384fd"},{"author":{"_account_id":37888,"name":"Akshay Ranade","display_name":"Akshay Ranade","email":"akshay.ranade@ibm.com","username":"rakshay"},"change_message_id":"b2a908b625b15943e473de92b5ed9465d72d15a6","unresolved":true,"context_lines":[{"line_number":5064,"context_line":"        self.assertIn(\u0027initiator_target_map\u0027, conn_info[\u0027data\u0027])"},{"line_number":5065,"context_line":""},{"line_number":5066,"context_line":"        # Scenario 2 : user defined host location"},{"line_number":5067,"context_line":"        host_site \u003d {\u0027site1\u0027: \u0027ff00000000000000\u0027, \u0027site2\u0027: \u0027ff00000000000001\u0027}"},{"line_number":5068,"context_line":"        self.fc_driver.configuration.set_override("},{"line_number":5069,"context_line":"            \u0027storwize_preferred_host_site\u0027, host_site)"},{"line_number":5070,"context_line":"        conn_info2 \u003d self.fc_driver.initialize_connection(volume_fc, connector)"}],"source_content_type":"text/x-python","patch_set":24,"id":"e139adc1_827d7dfb","line":5067,"in_reply_to":"d6ac7fc0_65fb69c9","updated":"2026-04-14 10:20:45.000000000","message":"Thanks @jayaanand.borra@netapp.com for reveiwing the patch.\nThe UT passed even with no matching wwpn in this case is a result of failback mechanism in src code. It went ahead with the case of no host site info provided.\nI could have added site specific asserts here to catch such mismatches.\nI will rather go ahead and sycn the wwps from connector so that the UT intentions are met.","commit_id":"bc7854418aa4848b894bbe01f9766d974dc384fd"}]}