)]}'
{"/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":"00075ab2903580e835936a93eadbdcd5d7ab0c2d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"a069abd8_5dc133ae","updated":"2026-09-15 05:25:38.000000000","message":"also run NetApp CI to make sure no regression","commit_id":"47f7e7f9ad4433d6edcbcf9a2ee0203e14d9f648"},{"author":{"_account_id":38864,"name":"Sridhar Kumar Ettkepalli","display_name":"Sridhar","email":"sridharkumar.ettkepalli@windriver.com","username":"settkepa"},"change_message_id":"29efc67b4e963eb75dfd2c4f23fa7d0a74774229","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"051b23b7_0daaec0b","updated":"2026-09-14 19:02:03.000000000","message":"recheck","commit_id":"47f7e7f9ad4433d6edcbcf9a2ee0203e14d9f648"},{"author":{"_account_id":38081,"name":"Anthony Galica","display_name":"agalica","email":"anthony.galica@hitachivantara.com","username":"agalica","status":"Hitachi Vantara"},"change_message_id":"fa4b47513db32631a7f578712cbf5045ec8590f4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"04cd70f6_5dd69192","updated":"2026-09-17 20:46:49.000000000","message":"Need release notes and a bug report.","commit_id":"cc64c6a6a013072a6bf62ef2fd87c11a0fa33d59"},{"author":{"_account_id":38081,"name":"Anthony Galica","display_name":"agalica","email":"anthony.galica@hitachivantara.com","username":"agalica","status":"Hitachi Vantara"},"change_message_id":"dcf5b912efa9549392035d4a66df7783e51f5cc7","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"dfd78a31_22df4cc6","in_reply_to":"04cd70f6_5dd69192","updated":"2026-09-17 20:47:43.000000000","message":"Correction: Just release notes.","commit_id":"cc64c6a6a013072a6bf62ef2fd87c11a0fa33d59"},{"author":{"_account_id":38864,"name":"Sridhar Kumar Ettkepalli","display_name":"Sridhar","email":"sridharkumar.ettkepalli@windriver.com","username":"settkepa"},"change_message_id":"69044e0aa7e58101c984ca6972f9b0298d9d22da","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"67724351_34c592ee","in_reply_to":"dfd78a31_22df4cc6","updated":"2026-09-23 17:47:39.000000000","message":"Added the release note here in the new patchset per your ask.","commit_id":"cc64c6a6a013072a6bf62ef2fd87c11a0fa33d59"},{"author":{"_account_id":38864,"name":"Sridhar Kumar Ettkepalli","display_name":"Sridhar","email":"sridharkumar.ettkepalli@windriver.com","username":"settkepa"},"change_message_id":"bfa664cf85d0846320760e427b112afacbf66e31","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"01efdf51_c55ff4df","updated":"2026-09-25 13:03:03.000000000","message":"@anthony.galica@hitachivantara.com I see that zuul and Net-App CI is green now. Could you please help approve?","commit_id":"9393ce8c2ba1030970a5253e4a5a9bd63386191d"},{"author":{"_account_id":38081,"name":"Anthony Galica","display_name":"agalica","email":"anthony.galica@hitachivantara.com","username":"agalica","status":"Hitachi Vantara"},"change_message_id":"2351a6f3eb6e7f69cdf24f6a2497963a8c5e4cb5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"bcc4dd1d_ea7b46e0","updated":"2026-09-24 11:37:25.000000000","message":"Adjusting to -1 as the NetApp CI is also failing.","commit_id":"9393ce8c2ba1030970a5253e4a5a9bd63386191d"},{"author":{"_account_id":38059,"name":"Anoop Kumar Shukla","display_name":"Anoop Shukla","email":"anoop.shukla@netapp.com","username":"anoop2","status":"NetApp"},"change_message_id":"70f6e39c17e48c78fc6e27306eb1747ab11c08e1","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"484fe463_ab2b5b4b","updated":"2026-09-24 05:07:25.000000000","message":"Code LGTM. Will wait for NetApp CI runs to succeed before proceeding with the approvals.","commit_id":"9393ce8c2ba1030970a5253e4a5a9bd63386191d"},{"author":{"_account_id":38081,"name":"Anthony Galica","display_name":"agalica","email":"anthony.galica@hitachivantara.com","username":"agalica","status":"Hitachi Vantara"},"change_message_id":"27da8572c3fea01296acd21dfb9a1723bea5a203","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"af67fead_7209a72c","updated":"2026-09-24 11:36:48.000000000","message":"LGTM, but Zuul is still not happy.\n\nI don\u0027t think this is your issue.  Ping me when it\u0027s passing and I can change to a +2.","commit_id":"9393ce8c2ba1030970a5253e4a5a9bd63386191d"},{"author":{"_account_id":38059,"name":"Anoop Kumar Shukla","display_name":"Anoop Shukla","email":"anoop.shukla@netapp.com","username":"anoop2","status":"NetApp"},"change_message_id":"f5dc44f6ba8e546f240cff7a224cded87afd4e11","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"97f6f565_e4e8eed7","updated":"2026-09-30 05:28:29.000000000","message":"LGTM. Nit: The commit message is too long and talks about implementation. May be shorten it if possible.","commit_id":"9393ce8c2ba1030970a5253e4a5a9bd63386191d"},{"author":{"_account_id":38864,"name":"Sridhar Kumar Ettkepalli","display_name":"Sridhar","email":"sridharkumar.ettkepalli@windriver.com","username":"settkepa"},"change_message_id":"325c48856d54b7ddc8b053985f24807447caa126","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"02231a22_3511fb4d","updated":"2026-09-25 12:29:10.000000000","message":"Run-NetApp CI","commit_id":"9393ce8c2ba1030970a5253e4a5a9bd63386191d"},{"author":{"_account_id":38059,"name":"Anoop Kumar Shukla","display_name":"Anoop Shukla","email":"anoop.shukla@netapp.com","username":"anoop2","status":"NetApp"},"change_message_id":"0c2aae20d230baa51acb116d1cf2fc2b7844446f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"f3f2d84f_9b864c1e","updated":"2026-09-24 04:39:09.000000000","message":"Run-NetApp CI","commit_id":"9393ce8c2ba1030970a5253e4a5a9bd63386191d"},{"author":{"_account_id":38081,"name":"Anthony Galica","display_name":"agalica","email":"anthony.galica@hitachivantara.com","username":"agalica","status":"Hitachi Vantara"},"change_message_id":"5376419759a2ccdbceedfab6446880e374e602ff","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"d265975b_081158ac","updated":"2026-09-24 01:16:32.000000000","message":"recheck","commit_id":"9393ce8c2ba1030970a5253e4a5a9bd63386191d"},{"author":{"_account_id":38864,"name":"Sridhar Kumar Ettkepalli","display_name":"Sridhar","email":"sridharkumar.ettkepalli@windriver.com","username":"settkepa"},"change_message_id":"e3efec65056490f0396852efb3efda812e6a13ec","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"61417a9f_e11d6197","updated":"2026-09-24 19:38:23.000000000","message":"recheck\n\nrun-NetApp CI","commit_id":"9393ce8c2ba1030970a5253e4a5a9bd63386191d"},{"author":{"_account_id":38081,"name":"Anthony Galica","display_name":"agalica","email":"anthony.galica@hitachivantara.com","username":"agalica","status":"Hitachi Vantara"},"change_message_id":"f758ef2bf15a7e593006ce42d74801c7a9436449","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"45dfbe52_51cd7517","updated":"2026-09-24 11:34:20.000000000","message":"recheck pyslow not happy still","commit_id":"9393ce8c2ba1030970a5253e4a5a9bd63386191d"}],"cinder/releasenotes/notes/netapp-ontap-rest-perf-counter-2166908-352d36ac563ad4ce.yaml":[{"author":{"_account_id":38864,"name":"Sridhar Kumar Ettkepalli","display_name":"Sridhar","email":"sridharkumar.ettkepalli@windriver.com","username":"settkepa"},"change_message_id":"86caf30758bb3a19092aedc6c1336df7cba26726","unresolved":false,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"fixes:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    NetApp ONTAP driver: Fixed driver initialization failing with the"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"9c68436c_67e125bc","line":1,"updated":"2026-09-23 20:46:18.000000000","message":"recheck\n\nVerified-1 was tempest-ipv6-only failing a single test on an environment/SSH flake, unrelated to this driver change. The other red jobs are non-voting and pre-existing.","commit_id":"9393ce8c2ba1030970a5253e4a5a9bd63386191d"}],"cinder/volume/drivers/netapp/dataontap/client/client_cmode_rest.py":[{"author":{"_account_id":31779,"name":"Jean Pierre Roquesalane","display_name":"happystacker","email":"jeanpierre.roquesalane@dell.com","username":"happystacker"},"change_message_id":"518b4f56795af094b8806a4e11e0fdfba6cb6e57","unresolved":true,"context_lines":[{"line_number":1265,"context_line":"                \u0027base-counter\u0027: base_counter,"},{"line_number":1266,"context_line":"            }"},{"line_number":1267,"context_line":"        except netapp_api.NaApiError:"},{"line_number":1268,"context_line":"            # NOTE: Re-raise the original NaApiError rather than translating it"},{"line_number":1269,"context_line":"            # to exception.NotFound. The perf_cmode fallback in"},{"line_number":1270,"context_line":"            # _init_counter_info only catches NaApiError, and NotFound is not a"},{"line_number":1271,"context_line":"            # subclass of it; translating here defeats that fallback and makes"}],"source_content_type":"text/x-python","patch_set":3,"id":"d8aa852b_452e55ad","line":1268,"updated":"2026-09-16 07:07:14.000000000","message":"Nit: The debug message says the counter was “not found,” but the caught NaApiError could represent another REST failure, such as authorization or a transient server error.","commit_id":"47f7e7f9ad4433d6edcbcf9a2ee0203e14d9f648"},{"author":{"_account_id":38864,"name":"Sridhar Kumar Ettkepalli","display_name":"Sridhar","email":"sridharkumar.ettkepalli@windriver.com","username":"settkepa"},"change_message_id":"357844f8c18e19a4fe5dc86bcfe8bdb3908f6a55","unresolved":false,"context_lines":[{"line_number":1265,"context_line":"                \u0027base-counter\u0027: base_counter,"},{"line_number":1266,"context_line":"            }"},{"line_number":1267,"context_line":"        except netapp_api.NaApiError:"},{"line_number":1268,"context_line":"            # NOTE: Re-raise the original NaApiError rather than translating it"},{"line_number":1269,"context_line":"            # to exception.NotFound. The perf_cmode fallback in"},{"line_number":1270,"context_line":"            # _init_counter_info only catches NaApiError, and NotFound is not a"},{"line_number":1271,"context_line":"            # subclass of it; translating here defeats that fallback and makes"}],"source_content_type":"text/x-python","patch_set":3,"id":"d1a5b9ac_f4f97b6c","line":1268,"in_reply_to":"8855972b_151cd6b6","updated":"2026-09-16 20:23:34.000000000","message":"Done","commit_id":"47f7e7f9ad4433d6edcbcf9a2ee0203e14d9f648"},{"author":{"_account_id":38864,"name":"Sridhar Kumar Ettkepalli","display_name":"Sridhar","email":"sridharkumar.ettkepalli@windriver.com","username":"settkepa"},"change_message_id":"da4491bcf868931d13a16beeab23713b4ac18b48","unresolved":false,"context_lines":[{"line_number":1265,"context_line":"                \u0027base-counter\u0027: base_counter,"},{"line_number":1266,"context_line":"            }"},{"line_number":1267,"context_line":"        except netapp_api.NaApiError:"},{"line_number":1268,"context_line":"            # NOTE: Re-raise the original NaApiError rather than translating it"},{"line_number":1269,"context_line":"            # to exception.NotFound. The perf_cmode fallback in"},{"line_number":1270,"context_line":"            # _init_counter_info only catches NaApiError, and NotFound is not a"},{"line_number":1271,"context_line":"            # subclass of it; translating here defeats that fallback and makes"}],"source_content_type":"text/x-python","patch_set":3,"id":"434cad8b_96484eb2","line":1268,"in_reply_to":"b31c65ea_1c63b14c","updated":"2026-09-16 17:49:21.000000000","message":"Thanks — made the message generic (\"Could not retrieve performance counter %s\" instead of \"not found\", since the NaApiError can be auth/transient/etc.) and raised it to info.\n\nOne small deviation from the snippet: I logged counter_name (the method argument) rather than query.get(\u0027counters.name\u0027). The except covers two send_request calls -- the counter-table lookup (query has counter_schemas.name) and the rows lookup (query has counters.name) -- so query.get(\u0027counters.name\u0027) would be None when the first request is the one that failed. counter_name is unambiguous in both paths.","commit_id":"47f7e7f9ad4433d6edcbcf9a2ee0203e14d9f648"},{"author":{"_account_id":37173,"name":"Alex Fernandes Figueirêdo","display_name":"Alex Figueiredo","email":"alex.fernandesfigueiredo@windriver.com","username":"afernand"},"change_message_id":"f9ae31423ad2c88c1848f90fc0735ee080991b21","unresolved":true,"context_lines":[{"line_number":1265,"context_line":"                \u0027base-counter\u0027: base_counter,"},{"line_number":1266,"context_line":"            }"},{"line_number":1267,"context_line":"        except netapp_api.NaApiError:"},{"line_number":1268,"context_line":"            # NOTE: Re-raise the original NaApiError rather than translating it"},{"line_number":1269,"context_line":"            # to exception.NotFound. The perf_cmode fallback in"},{"line_number":1270,"context_line":"            # _init_counter_info only catches NaApiError, and NotFound is not a"},{"line_number":1271,"context_line":"            # subclass of it; translating here defeats that fallback and makes"}],"source_content_type":"text/x-python","patch_set":3,"id":"b31c65ea_1c63b14c","line":1268,"in_reply_to":"d8aa852b_452e55ad","updated":"2026-09-16 17:04:11.000000000","message":"Agreed. The message could be something more generic to cover unforeseen situations. In addition, I suggest raising the log level to info, at minimum. E.g.: \n```python\n            LOG.info(\u0027Couldn\u0027t query fields %s for performance counter %s\u0027,\n                      query.get(\u0027fields\u0027),\n                      query.get(\u0027counters.name\u0027))\n```","commit_id":"47f7e7f9ad4433d6edcbcf9a2ee0203e14d9f648"},{"author":{"_account_id":38864,"name":"Sridhar Kumar Ettkepalli","display_name":"Sridhar","email":"sridharkumar.ettkepalli@windriver.com","username":"settkepa"},"change_message_id":"da4491bcf868931d13a16beeab23713b4ac18b48","unresolved":true,"context_lines":[{"line_number":1265,"context_line":"                \u0027base-counter\u0027: base_counter,"},{"line_number":1266,"context_line":"            }"},{"line_number":1267,"context_line":"        except netapp_api.NaApiError:"},{"line_number":1268,"context_line":"            # NOTE: Re-raise the original NaApiError rather than translating it"},{"line_number":1269,"context_line":"            # to exception.NotFound. The perf_cmode fallback in"},{"line_number":1270,"context_line":"            # _init_counter_info only catches NaApiError, and NotFound is not a"},{"line_number":1271,"context_line":"            # subclass of it; translating here defeats that fallback and makes"}],"source_content_type":"text/x-python","patch_set":3,"id":"8855972b_151cd6b6","line":1268,"in_reply_to":"d8aa852b_452e55ad","updated":"2026-09-16 17:49:21.000000000","message":"Good point — fixed. The message is now generic so it isn\u0027t misleading for auth/transient failures, and I raised it to info.","commit_id":"47f7e7f9ad4433d6edcbcf9a2ee0203e14d9f648"}]}