)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":36171,"name":"jayaanand borra","display_name":"jayaanand borra","email":"jayaanand.borra@netapp.com","username":"jayaanan","status":"netapp"},"change_message_id":"58ed8ed1bd335628dd539f2de11866795e821fb2","unresolved":true,"context_lines":[{"line_number":4,"context_line":"Commit:     Doug Goldstein \u003ccardoe@cardoe.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2025-11-11 20:25:11 -0600"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"fix(netapp): respect the iniitalize_connection supplied os_type"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"The user or os-brick can supply the os_type which should be used but"},{"line_number":10,"context_line":"this was never used. Switch to respecting the value supplied and if no"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"b3419298_d71d25cd","line":7,"updated":"2025-11-25 11:46:49.000000000","message":"\"iniitalize\" → should be \"initialize\"","commit_id":"ea83d169368539f5482f350e4e24203ee386ab4c"}],"/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":"58ed8ed1bd335628dd539f2de11866795e821fb2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"09b22845_fe10c208","updated":"2025-11-25 11:46:49.000000000","message":"Can you add UTs to proposed changes and CI is failing","commit_id":"ea83d169368539f5482f350e4e24203ee386ab4c"},{"author":{"_account_id":38059,"name":"Anoop Kumar Shukla","display_name":"Anoop Shukla","email":"anoop.shukla@netapp.com","username":"anoop2","status":"NetApp"},"change_message_id":"8b1e236a73b3c087ad0c8633537594163c7d00b9","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"ddc8aaab_289d181b","updated":"2025-12-18 17:31:27.000000000","message":"Run-NetApp CI","commit_id":"ea83d169368539f5482f350e4e24203ee386ab4c"},{"author":{"_account_id":38059,"name":"Anoop Kumar Shukla","display_name":"Anoop Shukla","email":"anoop.shukla@netapp.com","username":"anoop2","status":"NetApp"},"change_message_id":"09a140a5b7e8be43aef414ba98849c0cdb416b99","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"f8083f72_d3c07e57","updated":"2025-12-18 17:31:08.000000000","message":"UT changes are required.","commit_id":"ea83d169368539f5482f350e4e24203ee386ab4c"}],"cinder/volume/drivers/netapp/dataontap/block_base.py":[{"author":{"_account_id":36171,"name":"jayaanand borra","display_name":"jayaanand borra","email":"jayaanand.borra@netapp.com","username":"jayaanan","status":"netapp"},"change_message_id":"27c554744404d112392aab26e7f5158dea9eaf2e","unresolved":true,"context_lines":[{"line_number":442,"context_line":"            discovered_lun \u003d self._extract_lun_info(lun)"},{"line_number":443,"context_line":"            self._add_lun_to_table(discovered_lun)"},{"line_number":444,"context_line":""},{"line_number":445,"context_line":"    def _map_lun(self,"},{"line_number":446,"context_line":"                 name: str,"},{"line_number":447,"context_line":"                 initiator_list: list,"},{"line_number":448,"context_line":"                 initiator_type,"}],"source_content_type":"text/x-python","patch_set":1,"id":"f79d8926_0535b69b","line":445,"updated":"2026-02-20 08:25:08.000000000","message":"os_type \u003d connector.get(\"os_type\") or self.host_type\nif os_type not in self.ALLOWED_IGROUP_HOST_TYPES:\n    raise exception.VolumeBackendAPIException(...)\n\nBut the call is still the old form:\nlun_id \u003d self._map_lun(name, [initiator_name], \u0027iscsi\u0027, None)\n\ngoes to os_type, not lun_id!  The positional call self._map_lun(name, \n[initiator_name], \u0027iscsi\u0027, None) maps:\n\n\u0027iscsi\u0027 → initiator_type \nNone → os_type  (None instead of the computed variable)\nlun_id → uses default None (may be correct but os_type\u003dNone breaks the method)","commit_id":"ea83d169368539f5482f350e4e24203ee386ab4c"},{"author":{"_account_id":5890,"name":"Doug Goldstein","email":"cardoe@cardoe.com","username":"cardoe"},"change_message_id":"9711c2297407db7e898d4656ecb601334604378d","unresolved":true,"context_lines":[{"line_number":442,"context_line":"            discovered_lun \u003d self._extract_lun_info(lun)"},{"line_number":443,"context_line":"            self._add_lun_to_table(discovered_lun)"},{"line_number":444,"context_line":""},{"line_number":445,"context_line":"    def _map_lun(self,"},{"line_number":446,"context_line":"                 name: str,"},{"line_number":447,"context_line":"                 initiator_list: list,"},{"line_number":448,"context_line":"                 initiator_type,"}],"source_content_type":"text/x-python","patch_set":1,"id":"77386e8d_f88204c5","line":445,"in_reply_to":"f79d8926_0535b69b","updated":"2026-03-20 22:22:13.000000000","message":"Yeah the final implementation ended up being a bit different.","commit_id":"ea83d169368539f5482f350e4e24203ee386ab4c"}]}