)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":38059,"name":"Anoop Kumar Shukla","display_name":"Anoop Shukla","email":"anoop.shukla@netapp.com","username":"anoop2","status":"NetApp"},"change_message_id":"6bb89f87c9426215005ec7eecf1ed7ae0b9cdcac","unresolved":true,"context_lines":[{"line_number":15,"context_line":"receives a 401 challenge from ONTAP, then retries with valid"},{"line_number":16,"context_line":"credentials. This causes massive 401 Unauthorized audit log noise"},{"line_number":17,"context_line":"on ONTAP (~30k entries/min reported by customers), filling the"},{"line_number":18,"context_line":"AuditM.log buffer and obscuring real issues."},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"The requests library\u0027s HTTPBasicAuth sends the Authorization header"},{"line_number":21,"context_line":"preemptively on the first request, eliminating the 401 round-trip"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"eac9a189_93948c92","line":18,"range":{"start_line":18,"start_character":0,"end_line":18,"end_character":10},"updated":"2026-06-09 12:24:46.000000000","message":"nit: ONTAP audit log","commit_id":"3678582b3c91af135e436e898eaa7073a219f1b8"}],"/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":"6bb89f87c9426215005ec7eecf1ed7ae0b9cdcac","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"fb0415c2_fd326f74","updated":"2026-06-09 12:24:46.000000000","message":"Please take a look at the comments.","commit_id":"3678582b3c91af135e436e898eaa7073a219f1b8"}],"cinder/volume/drivers/netapp/dataontap/client/api.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":"6bb89f87c9426215005ec7eecf1ed7ae0b9cdcac","unresolved":true,"context_lines":[{"line_number":298,"context_line":"        \"\"\"Builds a requests.Session with auth and retry logic.\"\"\""},{"line_number":299,"context_line":"        self._session \u003d requests.Session()"},{"line_number":300,"context_line":""},{"line_number":301,"context_line":"        max_retries \u003d Retry(total\u003d5, connect\u003d5, read\u003d2, backoff_factor\u003d1)"},{"line_number":302,"context_line":"        adapter \u003d HTTPAdapter(max_retries\u003dmax_retries)"},{"line_number":303,"context_line":"        self._session.mount(\u0027%s://\u0027 % self._protocol, adapter)"},{"line_number":304,"context_line":"        self._session.headers.update("}],"source_content_type":"text/x-python","patch_set":1,"id":"4311bfa1_2820cd05","line":301,"updated":"2026-06-09 12:24:46.000000000","message":"Shall we keep the retry config configurable?","commit_id":"3678582b3c91af135e436e898eaa7073a219f1b8"},{"author":{"_account_id":38059,"name":"Anoop Kumar Shukla","display_name":"Anoop Shukla","email":"anoop.shukla@netapp.com","username":"anoop2","status":"NetApp"},"change_message_id":"6bb89f87c9426215005ec7eecf1ed7ae0b9cdcac","unresolved":true,"context_lines":[{"line_number":315,"context_line":"        else:"},{"line_number":316,"context_line":"            self._session.auth \u003d auth.HTTPBasicAuth("},{"line_number":317,"context_line":"                self._username, self._password)"},{"line_number":318,"context_line":"            self._session.verify \u003d False"},{"line_number":319,"context_line":""},{"line_number":320,"context_line":"        self._refresh_conn \u003d False"},{"line_number":321,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"0289ce10_a9b9ec1d","line":318,"updated":"2026-06-09 12:24:46.000000000","message":"Do we need to set this to ssl_verify? Keeping it false will compromise with TLS session verify?\n\nself._session.verify \u003d self._ssl_verify","commit_id":"3678582b3c91af135e436e898eaa7073a219f1b8"}]}