)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":35676,"name":"Max","email":"max.lamprecht@digits.schwarz","username":"lamprechtm"},"change_message_id":"a4b969a31f3846941710fba1999a08e118dfbd6e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"0fdc946c_ecc4100c","updated":"2025-04-29 21:08:43.000000000","message":"recheck","commit_id":"bdafcadaf5dd2a253c4ef873b67de553d2703ded"},{"author":{"_account_id":38059,"name":"Anoop Kumar Shukla","display_name":"Anoop Shukla","email":"anoop.shukla@netapp.com","username":"anoop2","status":"NetApp"},"change_message_id":"c9ded8ca3495c88bce6fbb10bd2327e5b94bab86","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"b7cd5d76_d6c57c85","updated":"2025-11-21 07:29:26.000000000","message":"Have run NetApp CI for the review. changes look good to me.","commit_id":"773f08ad2f9b08ec517d2d1f46360f773e629919"},{"author":{"_account_id":38059,"name":"Anoop Kumar Shukla","display_name":"Anoop Shukla","email":"anoop.shukla@netapp.com","username":"anoop2","status":"NetApp"},"change_message_id":"94bf439bbafdbf162ec7d57432c1cf6398e8fc28","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"575c69ea_773915b6","updated":"2025-12-16 12:54:36.000000000","message":"LGTM","commit_id":"773f08ad2f9b08ec517d2d1f46360f773e629919"},{"author":{"_account_id":36180,"name":"Gireesh Awasthi","display_name":"Gireesh","email":"gawasthi2010@gmail.com","username":"agireesh","status":"NetApp"},"change_message_id":"28e6ff132a7acff401d418119325c12fb1224484","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"26196c86_9dbf2e8a","updated":"2026-03-31 09:45:20.000000000","message":"LGTM ..!","commit_id":"773f08ad2f9b08ec517d2d1f46360f773e629919"},{"author":{"_account_id":38059,"name":"Anoop Kumar Shukla","display_name":"Anoop Shukla","email":"anoop.shukla@netapp.com","username":"anoop2","status":"NetApp"},"change_message_id":"0fa8c51a553ef20196573259f4892e106a34f939","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"710de278_1c1ee8d1","updated":"2026-04-01 07:16:41.000000000","message":"Please add release notes for the patch.\n\n@jobernar@redhat.com, @rajatdhasmana@gmail.com - We need some attention to this issue. Multiple customers are facing this and we have received queries asking for the fix. Can the core team please take a look at this bug fix?","commit_id":"773f08ad2f9b08ec517d2d1f46360f773e629919"},{"author":{"_account_id":38059,"name":"Anoop Kumar Shukla","display_name":"Anoop Shukla","email":"anoop.shukla@netapp.com","username":"anoop2","status":"NetApp"},"change_message_id":"5829107feb4ea11f3c6b3561a884bc988818e6ea","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"1889dfbb_ae113fbc","updated":"2025-11-21 07:26:01.000000000","message":"Run-NetApp CI","commit_id":"773f08ad2f9b08ec517d2d1f46360f773e629919"},{"author":{"_account_id":35676,"name":"Max","email":"max.lamprecht@digits.schwarz","username":"lamprechtm"},"change_message_id":"b8ebab3980fe772456c14c175b14cf1adccd1e06","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":3,"id":"bff546e4_0b97fe7c","in_reply_to":"710de278_1c1ee8d1","updated":"2026-04-01 11:49:14.000000000","message":"@anoop.shukla@netapp.com I have added a release note :)\n\nWould be great to get this merged. From an user perspective the rest driver has a lot of flaws currently. Would be great if you also could check out this:\nhttps://review.opendev.org/c/openstack/cinder/+/967734/comments/d0fe3efc_cfc3f8ea","commit_id":"773f08ad2f9b08ec517d2d1f46360f773e629919"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"5e0a098437d8dc0ac5bc52463a6b38af7e6104c9","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":4,"id":"2b66bd8a_f5bf9a3e","updated":"2026-04-01 14:42:59.000000000","message":"This looks OK to me, but it would be good to have an explicit acknowledgement from one of the NetApp developers that they endorse this change as correct.  Because intuitively, with this change you won\u0027t get an error from picking the \"wrong\" vserver any more, but you might still get an error if \"your\" vserver is busy.  But I am completely ignorant about the netapp backends, so it would be good to have an explicit endorsement.\n\nThat way, the commit message can be revised to include something about addressing nahimsouza\u0027s TODO from change I67eb7f6264cf5eea94c0a364082b530b9cdc1ae3.\n\nExcept for the above, I\u0027m +2 on this patch.","commit_id":"555c374a53968e49ab3334504383fc7d686edd21"},{"author":{"_account_id":38059,"name":"Anoop Kumar Shukla","display_name":"Anoop Shukla","email":"anoop.shukla@netapp.com","username":"anoop2","status":"NetApp"},"change_message_id":"b878c5bd2932ca7f1bcfdc5da856d129ab72fe19","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":4,"id":"210e4a1d_0fec4032","in_reply_to":"14a1df66_e96940a0","updated":"2026-04-02 16:42:48.000000000","message":"Hi Brian,\n\nWe looked at this change and also discussed internally. This is the recommended fix. The original error due to not going to the right SVM was \"volume not found\". But this fixes that problem. I am not sure why the SVM lookup was added in the original implementation, but it is actually not required.","commit_id":"555c374a53968e49ab3334504383fc7d686edd21"},{"author":{"_account_id":35676,"name":"Max","email":"max.lamprecht@digits.schwarz","username":"lamprechtm"},"change_message_id":"4a12164b07c6bb51c89d24d5eb943afbbebdb557","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":4,"id":"889e3937_977e075c","in_reply_to":"2b66bd8a_f5bf9a3e","updated":"2026-04-01 15:51:40.000000000","message":"Hi Brian,\nIt seems to be wrong to use a random one instead of the vserver that got specified in the backend(cinder.conf).\n\nWe already had 4 +1 before the commit msg change(2 of the approves were from netapp developers).\n\nI included nahimsouza\u0027s TODO in the commit msg","commit_id":"555c374a53968e49ab3334504383fc7d686edd21"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"534345ddee03b63073524dcd33faf7cd96f7048c","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":4,"id":"14a1df66_e96940a0","in_reply_to":"889e3937_977e075c","updated":"2026-04-01 16:09:07.000000000","message":"@Max thanks for updating the commit message.  I realize that there have been a bunch of +1s with no comment from netapp developers; what I want to see is an explicit acknowledgement from the vendor that this is the recommended fix.","commit_id":"555c374a53968e49ab3334504383fc7d686edd21"},{"author":{"_account_id":38081,"name":"Anthony Galica","display_name":"agalica","email":"anthony.galica@hitachivantara.com","username":"agalica","status":"Hitachi Vantara"},"change_message_id":"60f63763aebf853b78478fdcb67c06229bee4356","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"5e5e64e6_2eba03cd","updated":"2026-04-17 15:46:16.000000000","message":"LGTM - seems to be the desired fix by the NetApp team, and code removal looks fine.","commit_id":"ea7aff3689a901a8290b5d3442d41eb2233ddc97"},{"author":{"_account_id":36180,"name":"Gireesh Awasthi","display_name":"Gireesh","email":"gawasthi2010@gmail.com","username":"agireesh","status":"NetApp"},"change_message_id":"23e463ba442e0459c1b58a900f8ea42d9e800935","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"ebd7ac78_dcc92023","updated":"2026-04-17 15:59:45.000000000","message":"LGTM ..!","commit_id":"ea7aff3689a901a8290b5d3442d41eb2233ddc97"},{"author":{"_account_id":9236,"name":"Jon Bernard","email":"jobernar@redhat.com","username":"jbernard"},"change_message_id":"1fe9b52ad6c489ae9a39979e03ca658d32a8b2a7","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"5c40cd95_8f16d03e","updated":"2026-04-17 14:58:24.000000000","message":"Looks okay to me.","commit_id":"ea7aff3689a901a8290b5d3442d41eb2233ddc97"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"b0b369d0307a82985685b168c51983b1c3204199","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"f0f0f2ac_52751e11","updated":"2026-04-18 00:36:15.000000000","message":"Thanks Max for the updates, and thanks to Anoop for the ack for NetApp.","commit_id":"ea7aff3689a901a8290b5d3442d41eb2233ddc97"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"b460c9b9fb9c917106fdee2912e95c2d3a0db48d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"9e2662cb_0a911801","updated":"2026-04-01 21:32:22.000000000","message":"recheck tempest-integrated-storage :  tempest.api.compute.servers.test_server_actions.ServerActionsTestOtherA.test_resize_volume_backed_server_confirm - could not ssh into the instance","commit_id":"ea7aff3689a901a8290b5d3442d41eb2233ddc97"}]}
