)]}'
{"/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":"687c3a4eca0cb04ad47f8b3fea874b617f3b8f94","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"01615e11_10a8bf8c","updated":"2026-04-15 17:12:41.000000000","message":"LGTM","commit_id":"1cfbbd2dc0539d9907a13a0904735c81a46d2b9d"},{"author":{"_account_id":38059,"name":"Anoop Kumar Shukla","display_name":"Anoop Shukla","email":"anoop.shukla@netapp.com","username":"anoop2","status":"NetApp"},"change_message_id":"8a4a202e86147484e6de036e89505635710182b9","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"0a60dc15_c938127b","updated":"2026-04-17 14:38:02.000000000","message":"LGTM. I think this is the right fix. The bug talks about not being able to find device_paths and failing. Passing the find_controllers\u003dtrue seems to be doing that.","commit_id":"1cfbbd2dc0539d9907a13a0904735c81a46d2b9d"},{"author":{"_account_id":9236,"name":"Jon Bernard","email":"jobernar@redhat.com","username":"jbernard"},"change_message_id":"c467c7d3a6468e67ea9136a946d76f122bfbe3b5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"991afa63_03c87280","updated":"2026-08-17 17:57:11.000000000","message":"Logic looks okay to me.","commit_id":"1cfbbd2dc0539d9907a13a0904735c81a46d2b9d"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"c19320a124ccb69a9fc6ec0da733296989cf2f70","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"569500a1_a49f88d8","updated":"2026-08-21 15:42:39.000000000","message":"Nice fix.  Thank you!","commit_id":"1cfbbd2dc0539d9907a13a0904735c81a46d2b9d"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"f331eef9886afa581402a0b81ce56c88cc79a8d4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"0903da15_9a4bc3bc","updated":"2026-04-09 07:28:58.000000000","message":"we are using this patch in production against a netapp backend and it fixes the issue that we saw","commit_id":"1cfbbd2dc0539d9907a13a0904735c81a46d2b9d"}],"os_brick/initiator/connectors/nvmeof.py":[{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"c19320a124ccb69a9fc6ec0da733296989cf2f70","unresolved":true,"context_lines":[{"line_number":1222,"context_line":""},{"line_number":1223,"context_line":"        For replicated volumes, the RAID needs to be extended."},{"line_number":1224,"context_line":"        \"\"\""},{"line_number":1225,"context_line":"        # NOTE: Cannot use NVMeOFConnProps\u0027s decorator to create instance from"},{"line_number":1226,"context_line":"        # conn props because \"connect_volume_undo_prepare_result\" must be the"},{"line_number":1227,"context_line":"        # first decorator and it expects a dictionary."},{"line_number":1228,"context_line":"        conn_props \u003d NVMeOFConnProps(connection_properties,"}],"source_content_type":"text/x-python","patch_set":4,"id":"4a05ea9a_38020cba","line":1225,"range":{"start_line":1225,"start_character":16,"end_line":1225,"end_character":54},"updated":"2026-08-21 15:42:39.000000000","message":"If we had been able to use the decorator, the NVMeOFConnProps object would be created with find_controllers\u003dTrue (see line 696 above), so it makes sense for us to pass it at line 1229 below.","commit_id":"1cfbbd2dc0539d9907a13a0904735c81a46d2b9d"}]}
