)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":36725,"name":"Nilesh Thathagar","display_name":"Nilesh Thathagar","email":"nilesh.thathagar@dell.com","username":"NileshT"},"change_message_id":"3b8583007637d1eefa19d088138c992e86ee4a52","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"ede5c875_bc86bb22","updated":"2025-09-01 08:34:37.000000000","message":"run-DellEMC PowerFlex CI","commit_id":"89182fee1c72151484e56b1a667854d95e9bf04f"},{"author":{"_account_id":36725,"name":"Nilesh Thathagar","display_name":"Nilesh Thathagar","email":"nilesh.thathagar@dell.com","username":"NileshT"},"change_message_id":"2c66e24c584f9a8dac0a2c454230fc4a583a8e5a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"05cd0297_032a5c2b","updated":"2025-09-01 09:50:55.000000000","message":"run-DellEMC PowerFlex CI","commit_id":"aef78ee063a116da86e21113382a82f618c75ab0"},{"author":{"_account_id":36725,"name":"Nilesh Thathagar","display_name":"Nilesh Thathagar","email":"nilesh.thathagar@dell.com","username":"NileshT"},"change_message_id":"2b3687469339a10276f9e63c04cd36d70f76ae88","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"d479f6a1_04d3c9c1","updated":"2025-09-01 11:42:06.000000000","message":"Now looking ok. Please do a review.","commit_id":"bb373df52c0b042af55df58d92f7a49ef6825a98"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"17dfc8f7fbfd9390e02fbbc4b3a980b8aebe1b1e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"b06ca65a_162e6320","updated":"2025-09-02 17:37:30.000000000","message":"The revert originally happened because it was blocking the config file from being read.  The connector still needs to be able to read the config file during upgrades, when os-brick has been updated to 6.13.0+ but the cinder driver is still \u003c3.6.0.  Please indicate in the commit message whether this patch will affect the ability of the connector to read the config file.","commit_id":"bb373df52c0b042af55df58d92f7a49ef6825a98"},{"author":{"_account_id":36725,"name":"Nilesh Thathagar","display_name":"Nilesh Thathagar","email":"nilesh.thathagar@dell.com","username":"NileshT"},"change_message_id":"138591b7b3ab8af8ee69d0f96b2a2afc0a4efb9c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"3663fda9_0013116e","updated":"2025-09-01 10:32:15.000000000","message":"run-DellEMC PowerFlex CI","commit_id":"bb373df52c0b042af55df58d92f7a49ef6825a98"},{"author":{"_account_id":36725,"name":"Nilesh Thathagar","display_name":"Nilesh Thathagar","email":"nilesh.thathagar@dell.com","username":"NileshT"},"change_message_id":"3908d6e2213284bd227751e0abcacbda417a5d33","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"dc368c8d_d62891e3","in_reply_to":"b06ca65a_162e6320","updated":"2025-09-02 18:20:39.000000000","message":"Done. Added details in commit message.","commit_id":"bb373df52c0b042af55df58d92f7a49ef6825a98"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"52a51e7169907e36a46582afd573e8bd66c44c51","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"1067e5e3_6793bade","updated":"2026-02-17 17:59:08.000000000","message":"What is completely confusing about your commit message is that this patch is NOT a revert of a revert.  Line 40 in os_brick/privileged/__init__.py is different in this patch than it was in the revert [0] and of course it is different from the original patch [1].  This is very important because it\u0027s not like something external to the patch has changed that suddenly makes it work, it\u0027s that the original patch was incorrect and now you are fixing that.  So basically, this patch right here is the one that fixes the bug.  You can mention the one that was merged and withdrawn if you like, but this is the patch that actually fixes the original issue.\n\n[0] https://review.opendev.org/c/openstack/os-brick/+/896713/1/os_brick/privileged/__init__.py\n[1] https://review.opendev.org/c/openstack/os-brick/+/885979/6/os_brick/privileged/__init__.py","commit_id":"33de9c39425a5b39e077f0a92096d7d367208579"},{"author":{"_account_id":36725,"name":"Nilesh Thathagar","display_name":"Nilesh Thathagar","email":"nilesh.thathagar@dell.com","username":"NileshT"},"change_message_id":"8204d9420019a6b5ff1fee9a9cc725a5623fa7c3","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"24312e0c_aae7cb42","updated":"2026-02-13 14:08:06.000000000","message":"run-DellEMC PowerFlex CI","commit_id":"33de9c39425a5b39e077f0a92096d7d367208579"},{"author":{"_account_id":36725,"name":"Nilesh Thathagar","display_name":"Nilesh Thathagar","email":"nilesh.thathagar@dell.com","username":"NileshT"},"change_message_id":"ed9cea945bd2d1d47c65d6cb03e01cfb407d9303","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"6197cdb9_16ea9c57","updated":"2026-02-13 16:10:38.000000000","message":"run-DellEMC PowerFlex CI","commit_id":"33de9c39425a5b39e077f0a92096d7d367208579"},{"author":{"_account_id":36725,"name":"Nilesh Thathagar","display_name":"Nilesh Thathagar","email":"nilesh.thathagar@dell.com","username":"NileshT"},"change_message_id":"f77358dc58882b61444dadbce26d20fdcd3de300","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"87c9dd0d_f8782097","updated":"2026-02-13 14:16:18.000000000","message":"run-DellEMC PowerFlex CI","commit_id":"33de9c39425a5b39e077f0a92096d7d367208579"},{"author":{"_account_id":36725,"name":"Nilesh Thathagar","display_name":"Nilesh Thathagar","email":"nilesh.thathagar@dell.com","username":"NileshT"},"change_message_id":"67fe399b5e0dc646f386b048e704c8626b8701cc","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"ac3b7b3f_e6b3771a","updated":"2026-02-17 18:13:50.000000000","message":"@rosmaita.fossdev@gmail.com thanks for the review.\n\nUpdated the commit message as not reverting but fix the issue.","commit_id":"e3998b7d92a900ea704be135ded1030e9705710b"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"8065b8c3caf0e3a4ad32f749eb072739c1648f79","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"d15d05bb_55c68705","updated":"2026-02-17 19:25:01.000000000","message":"Revision LGTM.","commit_id":"e3998b7d92a900ea704be135ded1030e9705710b"},{"author":{"_account_id":36725,"name":"Nilesh Thathagar","display_name":"Nilesh Thathagar","email":"nilesh.thathagar@dell.com","username":"NileshT"},"change_message_id":"817f5c08f355a8b7ccad253c723ce0fa88550530","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"b32b2d89_80f70c43","updated":"2026-02-17 19:30:18.000000000","message":"thanks @eharney@redhat.com and @rosmaita.fossdev@gmail.com for the review.","commit_id":"e3998b7d92a900ea704be135ded1030e9705710b"}]}
