)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":38059,"name":"Anoop Kumar Shukla","display_name":"Anoop Shukla","email":"anoop.shukla@netapp.com","username":"anoop2","status":"NetApp"},"change_message_id":"f039e7efe8fde2d3c3c1695d2b5a257a1c69fbcc","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"584dc855_05079632","updated":"2026-06-16 16:11:31.000000000","message":"LGTM","commit_id":"d033507013b8b497e446395f2b81c5f406a55855"},{"author":{"_account_id":36180,"name":"Gireesh Awasthi","display_name":"Gireesh","email":"gawasthi2010@gmail.com","username":"agireesh","status":"NetApp"},"change_message_id":"bf3f2afb7d557e979cdf835ad92218418a76a3a7","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"1b3b7c06_4d7d5fe9","updated":"2026-06-17 05:14:43.000000000","message":"LGTM ..!","commit_id":"d033507013b8b497e446395f2b81c5f406a55855"},{"author":{"_account_id":38371,"name":"Kumar Tadepalli","display_name":"Kumar Tadepalli","email":"kumart.lead@gmail.com","username":"kumart","status":"NetApp"},"change_message_id":"03de22abb54cff931511cca3f00be39290285e8d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"16986fd6_7adf120a","updated":"2026-06-17 05:20:56.000000000","message":"Please address this comment.","commit_id":"d033507013b8b497e446395f2b81c5f406a55855"},{"author":{"_account_id":38059,"name":"Anoop Kumar Shukla","display_name":"Anoop Shukla","email":"anoop.shukla@netapp.com","username":"anoop2","status":"NetApp"},"change_message_id":"aa4c229e0db64221b85678d4e18e0ea73d1d768b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"bcb19cf4_be917181","updated":"2026-06-18 16:31:45.000000000","message":"I think we need to prioritize this fix as it is a regression fix. LGTM.","commit_id":"30b4f66de77d7950a96882d71f2958ede120a477"},{"author":{"_account_id":38371,"name":"Kumar Tadepalli","display_name":"Kumar Tadepalli","email":"kumart.lead@gmail.com","username":"kumart","status":"NetApp"},"change_message_id":"52c429574cfe63a7fa723a5db57840c78a9068ee","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"30b1d1c9_83678441","updated":"2026-06-17 06:10:44.000000000","message":"LGTM.","commit_id":"30b4f66de77d7950a96882d71f2958ede120a477"},{"author":{"_account_id":39161,"name":"Medha Choudhary (netapp)","display_name":"Medha Choudhary (netapp)","email":"medhac@netapp.com","username":"medhac","status":"Netapp"},"change_message_id":"8b7e3778922b6e8b6fddd5cadc63892ce4c91349","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"9277044c_1b94d8c5","updated":"2026-06-18 04:14:59.000000000","message":"recheck","commit_id":"30b4f66de77d7950a96882d71f2958ede120a477"}],"manila/share/drivers/netapp/dataontap/client/client_cmode_rest.py":[{"author":{"_account_id":38371,"name":"Kumar Tadepalli","display_name":"Kumar Tadepalli","email":"kumart.lead@gmail.com","username":"kumart","status":"NetApp"},"change_message_id":"03de22abb54cff931511cca3f00be39290285e8d","unresolved":true,"context_lines":[{"line_number":4114,"context_line":"            cluster_peer \u003d {"},{"line_number":4115,"context_line":"                # REST only exposes the remote cluster\u0027s addresses, so there"},{"line_number":4116,"context_line":"                # is no separate \"active\" (local) address list as in ZAPI."},{"line_number":4117,"context_line":"                \u0027active-addresses\u0027: [],"},{"line_number":4118,"context_line":"                \u0027peer-addresses\u0027: remote.get(\u0027ip_addresses\u0027, []),"},{"line_number":4119,"context_line":"                \u0027availability\u0027: peer.get(\u0027status\u0027, {}).get(\u0027state\u0027),"},{"line_number":4120,"context_line":"                \u0027cluster-name\u0027: peer.get(\u0027name\u0027),"}],"source_content_type":"text/x-python","patch_set":1,"id":"66b0d559_15eab767","line":4117,"updated":"2026-06-17 05:20:56.000000000","message":"```suggestion\n                \u0027active-addresses\u0027: remote.get(\u0027ip_addresses, []),\n```\n\nAs per ZAPI REST mapper, both are representing peer ipaddresses\nhttps://docs.netapp.com/us-en/ontap-restmap/cluster-peer.html#cluster-peer-get-iter.\nYou can remove the comment in above line as well.","commit_id":"d033507013b8b497e446395f2b81c5f406a55855"},{"author":{"_account_id":39161,"name":"Medha Choudhary (netapp)","display_name":"Medha Choudhary (netapp)","email":"medhac@netapp.com","username":"medhac","status":"Netapp"},"change_message_id":"4f42fa67a6204678607b373ad89ae2e0add7b52b","unresolved":false,"context_lines":[{"line_number":4114,"context_line":"            cluster_peer \u003d {"},{"line_number":4115,"context_line":"                # REST only exposes the remote cluster\u0027s addresses, so there"},{"line_number":4116,"context_line":"                # is no separate \"active\" (local) address list as in ZAPI."},{"line_number":4117,"context_line":"                \u0027active-addresses\u0027: [],"},{"line_number":4118,"context_line":"                \u0027peer-addresses\u0027: remote.get(\u0027ip_addresses\u0027, []),"},{"line_number":4119,"context_line":"                \u0027availability\u0027: peer.get(\u0027status\u0027, {}).get(\u0027state\u0027),"},{"line_number":4120,"context_line":"                \u0027cluster-name\u0027: peer.get(\u0027name\u0027),"}],"source_content_type":"text/x-python","patch_set":1,"id":"b39b01ac_12b94385","line":4117,"in_reply_to":"66b0d559_15eab767","updated":"2026-06-17 05:43:39.000000000","message":"Done","commit_id":"d033507013b8b497e446395f2b81c5f406a55855"}]}