)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"619918c9a029fb6f7713e10ed2f1422ed324e465","unresolved":true,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"NetApp ONTAP: Fix volume attachment with multiple HA pairs"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This patch fixes the volume attachement operation filtering"},{"line_number":10,"context_line":"the WWPNs that are passed to os-brick according to the HA pair"},{"line_number":11,"context_line":"where the volume is created."},{"line_number":12,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"f17b944b_046e14d4","line":9,"range":{"start_line":9,"start_character":28,"end_line":9,"end_character":39},"updated":"2023-08-28 10:47:53.000000000","message":"nit: attachment","commit_id":"3af828738a07ed520cd889674b14c310d809fb75"},{"author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"change_message_id":"4db864f761b844bb8b42b293a8f6933494785f46","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"NetApp ONTAP: Fix volume attachment with multiple HA pairs"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This patch fixes the volume attachement operation filtering"},{"line_number":10,"context_line":"the WWPNs that are passed to os-brick according to the HA pair"},{"line_number":11,"context_line":"where the volume is created."},{"line_number":12,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"69d44363_896625cf","line":9,"range":{"start_line":9,"start_character":28,"end_line":9,"end_character":39},"in_reply_to":"f17b944b_046e14d4","updated":"2024-12-10 21:19:04.000000000","message":"Done","commit_id":"3af828738a07ed520cd889674b14c310d809fb75"},{"author":{"_account_id":38059,"name":"Anoop Kumar Shukla","display_name":"Anoop Shukla","email":"anoop.shukla@netapp.com","username":"anoop2","status":"NetApp"},"change_message_id":"8dd3eee14e7bce53230fb916063968c5f950ecf8","unresolved":true,"context_lines":[{"line_number":7,"context_line":"NetApp ONTAP: Fix volume attachment with multiple HA pairs"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This patch fixes the volume attachment operation filtering"},{"line_number":10,"context_line":"the WWPNs that are passed to os-brick according to the HA pair"},{"line_number":11,"context_line":"where the volume is created."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Closes-Bug: #1990846"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"aa9fde15_fcde9948","line":10,"updated":"2026-02-26 12:09:13.000000000","message":"Can you please add additional detail to the commit message and release notes on why the attachment can fail if all WWPNs are passed?","commit_id":"583741adfcc13472767c7487ea1780e680f16492"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":30615,"name":"Tushar Trambak Gite","email":"tushargite96@gmail.com","username":"tushargite96"},"change_message_id":"d52e9cd70d55496031a1616aa3343dc59d984515","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"2eda3dfe_a26d961b","updated":"2023-02-16 09:25:48.000000000","message":"LGTM","commit_id":"3af828738a07ed520cd889674b14c310d809fb75"},{"author":{"_account_id":33756,"name":"Luisa","email":"luisarfamaral@gmail.com","username":"rfluisa"},"change_message_id":"5963a2fc0b70e9a1b4b60fe1ea7995d190787194","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"e78b5572_59cebbd6","updated":"2023-02-14 16:49:38.000000000","message":"Looks good to me :) ","commit_id":"3af828738a07ed520cd889674b14c310d809fb75"},{"author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"change_message_id":"4db864f761b844bb8b42b293a8f6933494785f46","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"f637a752_4b6922cf","updated":"2024-12-10 21:19:04.000000000","message":"Needs a rebase -- I\u0027ll do that to get this moving again.","commit_id":"3af828738a07ed520cd889674b14c310d809fb75"},{"author":{"_account_id":35586,"name":"Renan Vitor","email":"renanv@netapp.com","username":"renan_vitor"},"change_message_id":"b628d7ec08ce8d2af90a150981436406eba0340f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"ab7b1487_8d7517ea","updated":"2023-07-03 18:46:12.000000000","message":"Thanks Nahim, LGTM!","commit_id":"3af828738a07ed520cd889674b14c310d809fb75"},{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"619918c9a029fb6f7713e10ed2f1422ed324e465","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"32fb21f8_69e3e3aa","updated":"2023-08-28 10:47:53.000000000","message":"The downvote is for the assert in the tests, and before +2-ing the patch I would need to know the answer to the 2 inline questions.","commit_id":"3af828738a07ed520cd889674b14c310d809fb75"},{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"616bae52260937529b558a3aec75c5be2e768dfe","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"bb16ed54_1eecf8a8","updated":"2023-08-28 06:01:33.000000000","message":"one nit noted inline, else looks good.\nNetApp FC CI is passing.\nThe change can also be made as a followup so won\u0027t block this patch for it.","commit_id":"3af828738a07ed520cd889674b14c310d809fb75"},{"author":{"_account_id":33648,"name":"Nahim Alves de Souza","email":"nahimsouza@outlook.com","username":"nahimsouza"},"change_message_id":"30a9c651445fe828a9f6b37862cd3bcdadbdd47b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"81aae6ef_c8eafc72","updated":"2022-11-23 20:25:54.000000000","message":"recheck - failure not related to the patch","commit_id":"3af828738a07ed520cd889674b14c310d809fb75"},{"author":{"_account_id":33648,"name":"Nahim Alves de Souza","email":"nahimsouza@outlook.com","username":"nahimsouza"},"change_message_id":"3a7e27b8d55cdb36d0de024e6a81403e7ac962ef","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"3a07a7dc_30975501","updated":"2023-01-31 13:01:22.000000000","message":"run-NetApp CI","commit_id":"3af828738a07ed520cd889674b14c310d809fb75"},{"author":{"_account_id":33648,"name":"Nahim Alves de Souza","email":"nahimsouza@outlook.com","username":"nahimsouza"},"change_message_id":"fd3b97c1d635f37d2cb0a872d5b697b10143ee5a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"977bc85d_32cfdb62","updated":"2022-11-24 10:45:09.000000000","message":"run-NetApp CI","commit_id":"3af828738a07ed520cd889674b14c310d809fb75"},{"author":{"_account_id":33648,"name":"Nahim Alves de Souza","email":"nahimsouza@outlook.com","username":"nahimsouza"},"change_message_id":"7350a4a0003c6f6768506df9d27276bddf241e5e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"f2702be7_207bd785","updated":"2022-11-23 20:26:17.000000000","message":"run-NetApp CI","commit_id":"3af828738a07ed520cd889674b14c310d809fb75"},{"author":{"_account_id":33648,"name":"Nahim Alves de Souza","email":"nahimsouza@outlook.com","username":"nahimsouza"},"change_message_id":"99b8c5ecdd4159241f4a9a9560a2c298bf37f83d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"d7b2edf0_d7d6d25e","updated":"2022-11-24 14:36:07.000000000","message":"run-NetApp CI\n","commit_id":"3af828738a07ed520cd889674b14c310d809fb75"},{"author":{"_account_id":38059,"name":"Anoop Kumar Shukla","display_name":"Anoop Shukla","email":"anoop.shukla@netapp.com","username":"anoop2","status":"NetApp"},"change_message_id":"1895cff67ba844016ee4c8a7cba5e56f4fbe1a1d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"25a85e57_6c01f447","updated":"2026-02-26 11:55:17.000000000","message":"Adding NetApp reviewers.","commit_id":"583741adfcc13472767c7487ea1780e680f16492"},{"author":{"_account_id":38059,"name":"Anoop Kumar Shukla","display_name":"Anoop Shukla","email":"anoop.shukla@netapp.com","username":"anoop2","status":"NetApp"},"change_message_id":"8dd3eee14e7bce53230fb916063968c5f950ecf8","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"e5e45487_0452b13c","updated":"2026-02-26 12:09:13.000000000","message":"Please look at the comments.","commit_id":"583741adfcc13472767c7487ea1780e680f16492"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"a40e3d7a95cf6967b6acc021a465c00573948ff9","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"d60251f7_e6490d03","updated":"2025-01-13 18:32:55.000000000","message":"recheck cinder-plugin-ceph-tempest","commit_id":"583741adfcc13472767c7487ea1780e680f16492"}],"cinder/tests/unit/volume/drivers/netapp/dataontap/client/test_client_cmode.py":[{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"616bae52260937529b558a3aec75c5be2e768dfe","unresolved":true,"context_lines":[{"line_number":4561,"context_line":"        volume_name \u003d fake_client.LUN_NAME"},{"line_number":4562,"context_line":"        wwpns \u003d self.client.get_fc_target_wwpns(volume_name\u003dvolume_name)"},{"line_number":4563,"context_line":""},{"line_number":4564,"context_line":"        self.assertEqual(wwpns, expected_wwpns)"},{"line_number":4565,"context_line":""},{"line_number":4566,"context_line":"        args1 \u003d {"},{"line_number":4567,"context_line":"            \u0027query\u0027: {"}],"source_content_type":"text/x-python","patch_set":4,"id":"7c329627_37f7adb4","line":4564,"range":{"start_line":4564,"start_character":8,"end_line":4564,"end_character":47},"updated":"2023-08-28 06:01:33.000000000","message":"nit: the format we follow here is assert(Expected, Result)","commit_id":"3af828738a07ed520cd889674b14c310d809fb75"},{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"619918c9a029fb6f7713e10ed2f1422ed324e465","unresolved":true,"context_lines":[{"line_number":4561,"context_line":"        volume_name \u003d fake_client.LUN_NAME"},{"line_number":4562,"context_line":"        wwpns \u003d self.client.get_fc_target_wwpns(volume_name\u003dvolume_name)"},{"line_number":4563,"context_line":""},{"line_number":4564,"context_line":"        self.assertEqual(wwpns, expected_wwpns)"},{"line_number":4565,"context_line":""},{"line_number":4566,"context_line":"        args1 \u003d {"},{"line_number":4567,"context_line":"            \u0027query\u0027: {"}],"source_content_type":"text/x-python","patch_set":4,"id":"c253951b_e3dbb910","line":4564,"range":{"start_line":4564,"start_character":8,"end_line":4564,"end_character":47},"in_reply_to":"7c329627_37f7adb4","updated":"2023-08-28 10:47:53.000000000","message":"-1: Sorry, that is a reason to downvote for me.","commit_id":"3af828738a07ed520cd889674b14c310d809fb75"},{"author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"change_message_id":"4db864f761b844bb8b42b293a8f6933494785f46","unresolved":false,"context_lines":[{"line_number":4561,"context_line":"        volume_name \u003d fake_client.LUN_NAME"},{"line_number":4562,"context_line":"        wwpns \u003d self.client.get_fc_target_wwpns(volume_name\u003dvolume_name)"},{"line_number":4563,"context_line":""},{"line_number":4564,"context_line":"        self.assertEqual(wwpns, expected_wwpns)"},{"line_number":4565,"context_line":""},{"line_number":4566,"context_line":"        args1 \u003d {"},{"line_number":4567,"context_line":"            \u0027query\u0027: {"}],"source_content_type":"text/x-python","patch_set":4,"id":"ed864425_ff3af421","line":4564,"range":{"start_line":4564,"start_character":8,"end_line":4564,"end_character":47},"in_reply_to":"c253951b_e3dbb910","updated":"2024-12-10 21:19:04.000000000","message":"Done","commit_id":"3af828738a07ed520cd889674b14c310d809fb75"},{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"616bae52260937529b558a3aec75c5be2e768dfe","unresolved":true,"context_lines":[{"line_number":4617,"context_line":"        volume_name \u003d None"},{"line_number":4618,"context_line":"        wwpns \u003d self.client.get_fc_target_wwpns(volume_name\u003dvolume_name)"},{"line_number":4619,"context_line":""},{"line_number":4620,"context_line":"        self.assertEqual(wwpns, expected_wwpns)"},{"line_number":4621,"context_line":""},{"line_number":4622,"context_line":"        mock_send_request_iter.assert_has_calls(["},{"line_number":4623,"context_line":"            mock.call(\u0027fcp-interface-get-iter\u0027, {}),"}],"source_content_type":"text/x-python","patch_set":4,"id":"c4f5bae3_760a0820","line":4620,"range":{"start_line":4620,"start_character":8,"end_line":4620,"end_character":47},"updated":"2023-08-28 06:01:33.000000000","message":"same","commit_id":"3af828738a07ed520cd889674b14c310d809fb75"},{"author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"change_message_id":"4db864f761b844bb8b42b293a8f6933494785f46","unresolved":false,"context_lines":[{"line_number":4617,"context_line":"        volume_name \u003d None"},{"line_number":4618,"context_line":"        wwpns \u003d self.client.get_fc_target_wwpns(volume_name\u003dvolume_name)"},{"line_number":4619,"context_line":""},{"line_number":4620,"context_line":"        self.assertEqual(wwpns, expected_wwpns)"},{"line_number":4621,"context_line":""},{"line_number":4622,"context_line":"        mock_send_request_iter.assert_has_calls(["},{"line_number":4623,"context_line":"            mock.call(\u0027fcp-interface-get-iter\u0027, {}),"}],"source_content_type":"text/x-python","patch_set":4,"id":"85fc0dd4_e3ac336c","line":4620,"range":{"start_line":4620,"start_character":8,"end_line":4620,"end_character":47},"in_reply_to":"c4f5bae3_760a0820","updated":"2024-12-10 21:19:04.000000000","message":"Done","commit_id":"3af828738a07ed520cd889674b14c310d809fb75"},{"author":{"_account_id":35586,"name":"Renan Vitor","email":"renanv@netapp.com","username":"renan_vitor"},"change_message_id":"b628d7ec08ce8d2af90a150981436406eba0340f","unresolved":true,"context_lines":[{"line_number":4635,"context_line":""},{"line_number":4636,"context_line":"        expected_wwpns \u003d []"},{"line_number":4637,"context_line":"        volume_name \u003d None"},{"line_number":4638,"context_line":"        wwpns \u003d self.client.get_fc_target_wwpns(volume_name\u003dvolume_name)"},{"line_number":4639,"context_line":""},{"line_number":4640,"context_line":"        self.assertEqual(wwpns, expected_wwpns)"},{"line_number":4641,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"ae101643_af425d09","line":4638,"updated":"2023-07-03 18:46:12.000000000","message":"nit: Do you think should be a test case where the tested function is called without the volume_name? to simulate the old behavior","commit_id":"3af828738a07ed520cd889674b14c310d809fb75"},{"author":{"_account_id":33648,"name":"Nahim Alves de Souza","email":"nahimsouza@outlook.com","username":"nahimsouza"},"change_message_id":"0edce686bad642d015617211ac1d8f3268565bbe","unresolved":false,"context_lines":[{"line_number":4635,"context_line":""},{"line_number":4636,"context_line":"        expected_wwpns \u003d []"},{"line_number":4637,"context_line":"        volume_name \u003d None"},{"line_number":4638,"context_line":"        wwpns \u003d self.client.get_fc_target_wwpns(volume_name\u003dvolume_name)"},{"line_number":4639,"context_line":""},{"line_number":4640,"context_line":"        self.assertEqual(wwpns, expected_wwpns)"},{"line_number":4641,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"bc095faf_4e60f71f","line":4638,"in_reply_to":"ae101643_af425d09","updated":"2023-07-19 19:18:01.000000000","message":"Thanks for the suggestion. But I think since in this case `volume_name` has a default value, it would be redundant to make a test passing a `None` value (as it is now) or test a call without passing any parameter.","commit_id":"3af828738a07ed520cd889674b14c310d809fb75"},{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"616bae52260937529b558a3aec75c5be2e768dfe","unresolved":true,"context_lines":[{"line_number":4637,"context_line":"        volume_name \u003d None"},{"line_number":4638,"context_line":"        wwpns \u003d self.client.get_fc_target_wwpns(volume_name\u003dvolume_name)"},{"line_number":4639,"context_line":""},{"line_number":4640,"context_line":"        self.assertEqual(wwpns, expected_wwpns)"},{"line_number":4641,"context_line":""},{"line_number":4642,"context_line":"        mock_send_request_iter.assert_has_calls(["},{"line_number":4643,"context_line":"            mock.call(\u0027fcp-interface-get-iter\u0027, {}),"}],"source_content_type":"text/x-python","patch_set":4,"id":"3d408550_269af2b0","line":4640,"range":{"start_line":4640,"start_character":8,"end_line":4640,"end_character":47},"updated":"2023-08-28 06:01:33.000000000","message":"same","commit_id":"3af828738a07ed520cd889674b14c310d809fb75"},{"author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"change_message_id":"4db864f761b844bb8b42b293a8f6933494785f46","unresolved":false,"context_lines":[{"line_number":4637,"context_line":"        volume_name \u003d None"},{"line_number":4638,"context_line":"        wwpns \u003d self.client.get_fc_target_wwpns(volume_name\u003dvolume_name)"},{"line_number":4639,"context_line":""},{"line_number":4640,"context_line":"        self.assertEqual(wwpns, expected_wwpns)"},{"line_number":4641,"context_line":""},{"line_number":4642,"context_line":"        mock_send_request_iter.assert_has_calls(["},{"line_number":4643,"context_line":"            mock.call(\u0027fcp-interface-get-iter\u0027, {}),"}],"source_content_type":"text/x-python","patch_set":4,"id":"10982e89_5b5e37e5","line":4640,"range":{"start_line":4640,"start_character":8,"end_line":4640,"end_character":47},"in_reply_to":"3d408550_269af2b0","updated":"2024-12-10 21:19:04.000000000","message":"Done","commit_id":"3af828738a07ed520cd889674b14c310d809fb75"}],"cinder/volume/drivers/netapp/dataontap/client/client_cmode.py":[{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"619918c9a029fb6f7713e10ed2f1422ed324e465","unresolved":true,"context_lines":[{"line_number":471,"context_line":""},{"line_number":472,"context_line":"        result \u003d self.send_iter_request(\u0027fcp-interface-get-iter\u0027,"},{"line_number":473,"context_line":"                                        api_args_wwpn)"},{"line_number":474,"context_line":"        if self._has_records(result):"},{"line_number":475,"context_line":"            for info in result.get_child_by_name("},{"line_number":476,"context_line":"                    \u0027attributes-list\u0027).get_children():"},{"line_number":477,"context_line":"                wwpn \u003d info.get_child_content(\u0027port-name\u0027).lower()"}],"source_content_type":"text/x-python","patch_set":4,"id":"42f9b79f_163e3e13","line":474,"range":{"start_line":474,"start_character":8,"end_line":474,"end_character":37},"updated":"2023-08-28 10:47:53.000000000","message":"?: Why are we no longer setting max-records to 100 in the request like we were doing in L414?","commit_id":"3af828738a07ed520cd889674b14c310d809fb75"},{"author":{"_account_id":38059,"name":"Anoop Kumar Shukla","display_name":"Anoop Shukla","email":"anoop.shukla@netapp.com","username":"anoop2","status":"NetApp"},"change_message_id":"8dd3eee14e7bce53230fb916063968c5f950ecf8","unresolved":true,"context_lines":[{"line_number":412,"context_line":"        \"\"\""},{"line_number":413,"context_line":""},{"line_number":414,"context_line":"        node_name \u003d None"},{"line_number":415,"context_line":"        if volume_name:"},{"line_number":416,"context_line":"            api_args \u003d {"},{"line_number":417,"context_line":"                \u0027query\u0027: {"},{"line_number":418,"context_line":"                    \u0027lun-info\u0027: {"},{"line_number":419,"context_line":"                        \u0027path\u0027: \u0027*\u0027 + volume_name,"},{"line_number":420,"context_line":"                    }"},{"line_number":421,"context_line":"                },"},{"line_number":422,"context_line":"                \u0027desired-attributes\u0027: {"},{"line_number":423,"context_line":"                    \u0027lun-info\u0027: {"},{"line_number":424,"context_line":"                        \u0027node\u0027: None,"},{"line_number":425,"context_line":"                    }"},{"line_number":426,"context_line":"                },"},{"line_number":427,"context_line":"            }"},{"line_number":428,"context_line":""},{"line_number":429,"context_line":"            result \u003d self.send_iter_request(\u0027lun-get-iter\u0027, api_args)"},{"line_number":430,"context_line":""}],"source_content_type":"text/x-python","patch_set":5,"id":"b8b4b433_f8c9c179","line":427,"range":{"start_line":415,"start_character":8,"end_line":427,"end_character":13},"updated":"2026-02-26 12:09:13.000000000","message":"To make sure it does not do an open ended query, can we also pass the vserver as query param? We do not want the search to happen on all LUNs which can be too many.\n\nYou can also pass the FlexVol name to narrow the query to be looking into specific FlexVol.","commit_id":"583741adfcc13472767c7487ea1780e680f16492"},{"author":{"_account_id":38059,"name":"Anoop Kumar Shukla","display_name":"Anoop Shukla","email":"anoop.shukla@netapp.com","username":"anoop2","status":"NetApp"},"change_message_id":"8dd3eee14e7bce53230fb916063968c5f950ecf8","unresolved":true,"context_lines":[{"line_number":468,"context_line":""},{"line_number":469,"context_line":"        result \u003d self.send_iter_request(\u0027fcp-interface-get-iter\u0027,"},{"line_number":470,"context_line":"                                        api_args_wwpn)"},{"line_number":471,"context_line":"        if self._has_records(result):"},{"line_number":472,"context_line":"            for info in result.get_child_by_name("},{"line_number":473,"context_line":"                    \u0027attributes-list\u0027).get_children():"},{"line_number":474,"context_line":"                wwpn \u003d info.get_child_content(\u0027port-name\u0027).lower()"}],"source_content_type":"text/x-python","patch_set":5,"id":"7b595a6c_dd874c1e","line":471,"updated":"2026-02-26 12:09:13.000000000","message":"In case no wwpns are found, can we throw an error and log an error?","commit_id":"583741adfcc13472767c7487ea1780e680f16492"}],"cinder/volume/drivers/netapp/dataontap/client/client_cmode_rest.py":[{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"619918c9a029fb6f7713e10ed2f1422ed324e465","unresolved":true,"context_lines":[{"line_number":1510,"context_line":"        query \u003d {"},{"line_number":1511,"context_line":"            \u0027fields\u0027: \u0027wwpn\u0027"},{"line_number":1512,"context_line":"        }"},{"line_number":1513,"context_line":"        response \u003d self.send_request(\u0027/network/fc/interfaces\u0027,"},{"line_number":1514,"context_line":"                                     \u0027get\u0027, query\u003dquery)"},{"line_number":1515,"context_line":""},{"line_number":1516,"context_line":"        records \u003d response.get(\u0027records\u0027)"}],"source_content_type":"text/x-python","patch_set":4,"id":"cdb552ac_4b3c1b4e","side":"PARENT","line":1513,"updated":"2023-08-28 10:47:53.000000000","message":"?: Why is there no implementation here?","commit_id":"a9b429ab6bb36561ea5aad38abaaea648effa1e8"},{"author":{"_account_id":38059,"name":"Anoop Kumar Shukla","display_name":"Anoop Shukla","email":"anoop.shukla@netapp.com","username":"anoop2","status":"NetApp"},"change_message_id":"8dd3eee14e7bce53230fb916063968c5f950ecf8","unresolved":true,"context_lines":[{"line_number":1510,"context_line":""},{"line_number":1511,"context_line":"        return map_list"},{"line_number":1512,"context_line":""},{"line_number":1513,"context_line":"    def get_fc_target_wwpns(self, volume_name\u003dNone):"},{"line_number":1514,"context_line":"        \"\"\"Gets the FC target details.\"\"\""},{"line_number":1515,"context_line":"        wwpns \u003d []"},{"line_number":1516,"context_line":"        query \u003d {"}],"source_content_type":"text/x-python","patch_set":5,"id":"19accd2a_9f1ec4a7","line":1513,"updated":"2026-02-26 12:09:13.000000000","message":"This method needs to also be updated with the logic used in ZAPIs.","commit_id":"583741adfcc13472767c7487ea1780e680f16492"}]}