)]}'
{"/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":"92239eebb1b6f179d1a114ee43fd622ae1eab999","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"43bb7245_3aecab3a","updated":"2026-11-11 07:10:03.000000000","message":"Please take a look at the minor comment.","commit_id":"9b9e244062d9dbb274334aadcf166cc0bd359f34"},{"author":{"_account_id":36179,"name":"Saikumar Pulluri","display_name":"Saikumar Pulluri","email":"saikumar1016@gmail.com","username":"pulluri"},"change_message_id":"d842ec5dcffeff4e97eab3c4ad69b92717fbf3c3","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"beb2a8db_7a2ddff2","updated":"2026-11-10 16:04:46.000000000","message":"run-NetApp CI","commit_id":"9b9e244062d9dbb274334aadcf166cc0bd359f34"},{"author":{"_account_id":38059,"name":"Anoop Kumar Shukla","display_name":"Anoop Shukla","email":"anoop.shukla@netapp.com","username":"anoop2","status":"NetApp"},"change_message_id":"d9b0f012cc3126a5740df47e706ba2670773b286","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"dc93e6e5_84890f47","updated":"2026-11-12 05:43:09.000000000","message":"LGTM.","commit_id":"ebc0eca4a658ac6ab51839f95d1669fe8f3f52b5"},{"author":{"_account_id":36180,"name":"Gireesh Awasthi","display_name":"Gireesh","email":"gawasthi2010@gmail.com","username":"agireesh","status":"NetApp"},"change_message_id":"8fc3dd21906b60ea370593f08d67f9abbc5e5044","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"8c2ab87b_1a53ccc2","updated":"2026-11-12 14:28:07.000000000","message":"Thanks @saikumar1016@gmail.com, for working on this. I have added few comments, please take a look.","commit_id":"ebc0eca4a658ac6ab51839f95d1669fe8f3f52b5"},{"author":{"_account_id":36179,"name":"Saikumar Pulluri","display_name":"Saikumar Pulluri","email":"saikumar1016@gmail.com","username":"pulluri"},"change_message_id":"098de2cd725b68b2073dc556c439b96ef647aaf6","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"c1d7d27a_1c31ca9c","updated":"2026-11-11 16:23:40.000000000","message":"Thanks Anoop. Addressed the comment.","commit_id":"ebc0eca4a658ac6ab51839f95d1669fe8f3f52b5"}],"cinder/volume/drivers/netapp/dataontap/client/client_cmode_rest.py":[{"author":{"_account_id":38059,"name":"Anoop Kumar Shukla","display_name":"Anoop Shukla","email":"anoop.shukla@netapp.com","username":"anoop2","status":"NetApp"},"change_message_id":"92239eebb1b6f179d1a114ee43fd622ae1eab999","unresolved":true,"context_lines":[{"line_number":422,"context_line":"        except netapp_api.NaApiError as ex:"},{"line_number":423,"context_line":"            if ex.code in (None, \u0027unknown\u0027):"},{"line_number":424,"context_line":"                raise"},{"line_number":425,"context_line":"            LOG.debug(\u0027Cannot use cluster API %(api)s: %(error)s\u0027,"},{"line_number":426,"context_line":"                      {\u0027api\u0027: api, \u0027error\u0027: ex})"},{"line_number":427,"context_line":"            return False"},{"line_number":428,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"1d96ee6b_5c61828e","line":425,"updated":"2026-11-11 07:10:03.000000000","message":"The needs to be error log.","commit_id":"9b9e244062d9dbb274334aadcf166cc0bd359f34"},{"author":{"_account_id":36179,"name":"Saikumar Pulluri","display_name":"Saikumar Pulluri","email":"saikumar1016@gmail.com","username":"pulluri"},"change_message_id":"098de2cd725b68b2073dc556c439b96ef647aaf6","unresolved":false,"context_lines":[{"line_number":422,"context_line":"        except netapp_api.NaApiError as ex:"},{"line_number":423,"context_line":"            if ex.code in (None, \u0027unknown\u0027):"},{"line_number":424,"context_line":"                raise"},{"line_number":425,"context_line":"            LOG.debug(\u0027Cannot use cluster API %(api)s: %(error)s\u0027,"},{"line_number":426,"context_line":"                      {\u0027api\u0027: api, \u0027error\u0027: ex})"},{"line_number":427,"context_line":"            return False"},{"line_number":428,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"7d00a924_5ae60a37","line":425,"in_reply_to":"1d96ee6b_5c61828e","updated":"2026-11-11 16:23:40.000000000","message":"Done, Thanks Anoop.","commit_id":"9b9e244062d9dbb274334aadcf166cc0bd359f34"},{"author":{"_account_id":36180,"name":"Gireesh Awasthi","display_name":"Gireesh","email":"gawasthi2010@gmail.com","username":"agireesh","status":"NetApp"},"change_message_id":"8fc3dd21906b60ea370593f08d67f9abbc5e5044","unresolved":true,"context_lines":[{"line_number":422,"context_line":"        except netapp_api.NaApiError as ex:"},{"line_number":423,"context_line":"            if ex.code in (None, \u0027unknown\u0027):"},{"line_number":424,"context_line":"                raise"},{"line_number":425,"context_line":"            LOG.error(\u0027Cannot use cluster API %(api)s: %(error)s\u0027,"},{"line_number":426,"context_line":"                      {\u0027api\u0027: api, \u0027error\u0027: ex})"},{"line_number":427,"context_line":"            return False"},{"line_number":428,"context_line":""},{"line_number":429,"context_line":"        return True"}],"source_content_type":"text/x-python","patch_set":2,"id":"b2506a4a_c566b514","line":426,"range":{"start_line":425,"start_character":11,"end_line":426,"end_character":48},"updated":"2026-11-12 14:28:07.000000000","message":"Change it LOG.debug or LOG.warning. Other can comment on this but for me this should be either debug to make it less noisy otherwise warning also fine instead of error.","commit_id":"ebc0eca4a658ac6ab51839f95d1669fe8f3f52b5"},{"author":{"_account_id":36180,"name":"Gireesh Awasthi","display_name":"Gireesh","email":"gawasthi2010@gmail.com","username":"agireesh","status":"NetApp"},"change_message_id":"8fc3dd21906b60ea370593f08d67f9abbc5e5044","unresolved":true,"context_lines":[{"line_number":424,"context_line":"                raise"},{"line_number":425,"context_line":"            LOG.error(\u0027Cannot use cluster API %(api)s: %(error)s\u0027,"},{"line_number":426,"context_line":"                      {\u0027api\u0027: api, \u0027error\u0027: ex})"},{"line_number":427,"context_line":"            return False"},{"line_number":428,"context_line":""},{"line_number":429,"context_line":"        return True"},{"line_number":430,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"5600ae01_b358d3c4","line":427,"range":{"start_line":427,"start_character":12,"end_line":427,"end_character":24},"updated":"2026-11-12 14:28:07.000000000","message":"Previously we are returning based on condtion ex.code \u003d\u003d netapp_api.REST_UNAUTHORIZED, which looks good approach, here also we need to add the specific error code to return False as False is broader than before as it can return base on other code also.","commit_id":"ebc0eca4a658ac6ab51839f95d1669fe8f3f52b5"}]}