)]}'
{"/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":"fda30c1ce2ec925aca75bbcc1a9848adcba5ba0b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"82c9e618_938f9874","updated":"2025-07-08 16:27:40.000000000","message":"Looks good.","commit_id":"46393aaca8c2271073526cb85e5713e74e786c50"},{"author":{"_account_id":36171,"name":"jayaanand borra","display_name":"jayaanand borra","email":"jayaanand.borra@netapp.com","username":"jayaanan","status":"netapp"},"change_message_id":"97e246c46fddfca78a6cd5ee9593f88b068c6b7e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"045ab9ac_5fc5cc18","updated":"2025-06-25 14:19:05.000000000","message":"recheck","commit_id":"46393aaca8c2271073526cb85e5713e74e786c50"},{"author":{"_account_id":5997,"name":"Walt","display_name":"Hemna","email":"waboring@hemna.com","username":"walter-boring","status":"SAP"},"change_message_id":"f1ad96ea9b17d68a9afcd2805e6ad1f17f5ff347","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"c1acc3b7_ca7c2c58","updated":"2025-07-16 13:47:32.000000000","message":"recheck","commit_id":"46393aaca8c2271073526cb85e5713e74e786c50"},{"author":{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},"change_message_id":"689d8af45a1df8acbd841e9d5c3f8b6229484c17","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"940d03bb_205d5382","updated":"2025-07-16 15:31:27.000000000","message":"I think it looks like this patch is close.  The wording in the one error message needs to be fixed and then I think I am +2 on this.","commit_id":"f88601cb2c8651282fc4d9857e6cc6d54dd193d0"},{"author":{"_account_id":36725,"name":"Nilesh Thathagar","display_name":"Nilesh Thathagar","email":"nilesh.thathagar@dell.com","username":"NileshT"},"change_message_id":"940dfbf39b76a6e6e0e55af6f1e98545a3c14a86","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"eb2a6fe8_74eb050c","updated":"2025-08-20 15:09:49.000000000","message":"Although the CI pipeline appears green, all jobs within it have actually failed.","commit_id":"11bc25c1267dd7f47ec9284e66c7d875a7511981"},{"author":{"_account_id":36171,"name":"jayaanand borra","display_name":"jayaanand borra","email":"jayaanand.borra@netapp.com","username":"jayaanan","status":"netapp"},"change_message_id":"f3b75d0ab6d78d3ee1c3bb0ffc5b09ab8dbee368","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"76857243_3d9bb5fd","updated":"2025-08-22 05:39:26.000000000","message":"Hi Nilesh, Can you please review again. NetApp CI is successful for iSCSI and NVMe-TCP. FCP we have recurring failures in all patches. also, FCP is non-voting.","commit_id":"11bc25c1267dd7f47ec9284e66c7d875a7511981"},{"author":{"_account_id":36171,"name":"jayaanand borra","display_name":"jayaanand borra","email":"jayaanand.borra@netapp.com","username":"jayaanan","status":"netapp"},"change_message_id":"57dc061e40826a2f24ad4869a8ba6566648fbb5f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"76e2aac1_53f530a4","updated":"2025-08-20 12:32:52.000000000","message":"Hi Nilesh, NetApp CI is green. can you please check","commit_id":"11bc25c1267dd7f47ec9284e66c7d875a7511981"},{"author":{"_account_id":38059,"name":"Anoop Kumar Shukla","display_name":"Anoop Shukla","email":"anoop.shukla@netapp.com","username":"anoop2","status":"NetApp"},"change_message_id":"b5254b4a1903dbdef2ff05e380c650c17131fcb3","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"0dd5748d_07185f2b","updated":"2025-08-07 13:54:31.000000000","message":"LGTM","commit_id":"11bc25c1267dd7f47ec9284e66c7d875a7511981"},{"author":{"_account_id":36725,"name":"Nilesh Thathagar","display_name":"Nilesh Thathagar","email":"nilesh.thathagar@dell.com","username":"NileshT"},"change_message_id":"84ace2c968f3f97fcfe3a4921a1c53ae24bd3a1a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"28fb1630_dadbc380","updated":"2025-08-18 05:54:31.000000000","message":"LGTM but the CI is getting failed.","commit_id":"11bc25c1267dd7f47ec9284e66c7d875a7511981"},{"author":{"_account_id":36725,"name":"Nilesh Thathagar","display_name":"Nilesh Thathagar","email":"nilesh.thathagar@dell.com","username":"NileshT"},"change_message_id":"5f8507270299d6b81e08ef7641de73e6588c23a6","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"2d3d8bff_800cddce","updated":"2025-08-27 14:53:19.000000000","message":"LGTM, CI is getting passed and there is one non voting fc CI is failed.","commit_id":"11bc25c1267dd7f47ec9284e66c7d875a7511981"},{"author":{"_account_id":13425,"name":"Simon Dodsley","email":"simon@everpuredata.com","username":"sdodsley"},"change_message_id":"b1b9428dee54b9f3852b998813efbb707dcdb80d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"b03eaf65_ca31b52c","updated":"2025-08-27 14:00:31.000000000","message":"Looks good","commit_id":"11bc25c1267dd7f47ec9284e66c7d875a7511981"},{"author":{"_account_id":36171,"name":"jayaanand borra","display_name":"jayaanand borra","email":"jayaanand.borra@netapp.com","username":"jayaanan","status":"netapp"},"change_message_id":"bda2379ba7de869fedddf3cb03c2d6eeb49bfcb3","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"da8d02d5_28bae8a1","updated":"2025-08-18 15:52:06.000000000","message":"NetApp CI is not in good shape. We are making progress to correct it. Please ignore NetApp CI result.","commit_id":"11bc25c1267dd7f47ec9284e66c7d875a7511981"},{"author":{"_account_id":38290,"deleted":true},"change_message_id":"bbdbda5723b9106d12dd6fabb90b9d6c7be95dae","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"58d127c6_543a4f2b","updated":"2025-08-21 14:18:18.000000000","message":"Run-NetApp CI","commit_id":"11bc25c1267dd7f47ec9284e66c7d875a7511981"},{"author":{"_account_id":36171,"name":"jayaanand borra","display_name":"jayaanand borra","email":"jayaanand.borra@netapp.com","username":"jayaanan","status":"netapp"},"change_message_id":"9d579a7d8b50995f5f922f7910fe3366fa80a660","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"c022962d_865cf55c","updated":"2025-08-20 04:57:21.000000000","message":"Run-NetApp CI","commit_id":"11bc25c1267dd7f47ec9284e66c7d875a7511981"},{"author":{"_account_id":38290,"deleted":true},"change_message_id":"bcffda3ec159c41b8ba0fe03a845cec423a4f288","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"c667eeb9_4752aaa5","updated":"2025-08-21 18:53:58.000000000","message":"Run-NetApp CI","commit_id":"11bc25c1267dd7f47ec9284e66c7d875a7511981"},{"author":{"_account_id":5997,"name":"Walt","display_name":"Hemna","email":"waboring@hemna.com","username":"walter-boring","status":"SAP"},"change_message_id":"96a270f8a46e83554c2321edc0f404e049992933","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"04601cc4_ed2fcc7f","updated":"2025-07-17 14:08:46.000000000","message":"recheck","commit_id":"11bc25c1267dd7f47ec9284e66c7d875a7511981"},{"author":{"_account_id":5997,"name":"Walt","display_name":"Hemna","email":"waboring@hemna.com","username":"walter-boring","status":"SAP"},"change_message_id":"2a759d725e6cd136ccfba0dd62540cfb4f2185f0","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"4ae59713_f0cf5212","updated":"2025-07-23 13:48:55.000000000","message":"recheck","commit_id":"11bc25c1267dd7f47ec9284e66c7d875a7511981"},{"author":{"_account_id":5997,"name":"Walt","display_name":"Hemna","email":"waboring@hemna.com","username":"walter-boring","status":"SAP"},"change_message_id":"6b2946d95cfc5ba253bd483e767bfa6d1e6912e8","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"71a490e2_ea9dcf19","updated":"2025-07-21 18:49:14.000000000","message":"recheck","commit_id":"11bc25c1267dd7f47ec9284e66c7d875a7511981"},{"author":{"_account_id":34034,"name":"Franciszek Przewoźny","display_name":"Franciszek Przewozny","email":"przewozny.franciszek@gmail.com","username":"fprzewozn","status":"OpenStack Engineer / SRE @ Opera Software"},"change_message_id":"7e4649dbb93b1fcf7fa6b0fd005944786aa1133e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"f74dcbf8_e58b0f95","updated":"2026-04-13 10:22:26.000000000","message":"recheck 2","commit_id":"adbdd8c8fde4cab00949cdeccf864d401168fe65"},{"author":{"_account_id":34034,"name":"Franciszek Przewoźny","display_name":"Franciszek Przewozny","email":"przewozny.franciszek@gmail.com","username":"fprzewozn","status":"OpenStack Engineer / SRE @ Opera Software"},"change_message_id":"256677804e634f02a2a51b8bf06ade8dccc797b1","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"926877fc_bc4b8e22","updated":"2026-04-13 07:42:31.000000000","message":"recheck: fatal: unable to access \u0027https://opendev.org/openstack/hacking/\u0027: The requested URL returned error: 500","commit_id":"adbdd8c8fde4cab00949cdeccf864d401168fe65"}],"cinder/volume/drivers/netapp/dataontap/nfs_base.py":[{"author":{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},"change_message_id":"689d8af45a1df8acbd841e9d5c3f8b6229484c17","unresolved":true,"context_lines":[{"line_number":972,"context_line":"                lock_err_msg \u003d \"another process using the image\""},{"line_number":973,"context_line":"                if volume_attached and ex.stderr and lock_err_msg in ex.stderr:"},{"line_number":974,"context_line":"                    LOG.debug(\"Failed to extend volume through Cinder, the\""},{"line_number":975,"context_line":"                              \"volume is locked by Nova service. Letting the \""},{"line_number":976,"context_line":"                              \"extend operation to Nova.\")"},{"line_number":977,"context_line":"                    ctxt.reraise \u003d False"},{"line_number":978,"context_line":"        except Exception as err:"},{"line_number":979,"context_line":"            exception_msg \u003d (_(\"Failed to extend volume \""}],"source_content_type":"text/x-python","patch_set":2,"id":"026770ac_06248145","line":976,"range":{"start_line":975,"start_character":65,"end_line":976,"end_character":56},"updated":"2025-07-16 15:31:27.000000000","message":"The last sentence here doesn\u0027t make sense.  I think it would make more sense to say \"Nova will attempt the extend operation.\"","commit_id":"f88601cb2c8651282fc4d9857e6cc6d54dd193d0"},{"author":{"_account_id":5997,"name":"Walt","display_name":"Hemna","email":"waboring@hemna.com","username":"walter-boring","status":"SAP"},"change_message_id":"bdd82b9da44ced4ab9ba076c0cffafe0b1ba4300","unresolved":false,"context_lines":[{"line_number":972,"context_line":"                lock_err_msg \u003d \"another process using the image\""},{"line_number":973,"context_line":"                if volume_attached and ex.stderr and lock_err_msg in ex.stderr:"},{"line_number":974,"context_line":"                    LOG.debug(\"Failed to extend volume through Cinder, the\""},{"line_number":975,"context_line":"                              \"volume is locked by Nova service. Letting the \""},{"line_number":976,"context_line":"                              \"extend operation to Nova.\")"},{"line_number":977,"context_line":"                    ctxt.reraise \u003d False"},{"line_number":978,"context_line":"        except Exception as err:"},{"line_number":979,"context_line":"            exception_msg \u003d (_(\"Failed to extend volume \""}],"source_content_type":"text/x-python","patch_set":2,"id":"81ed0fd7_0b90ca22","line":976,"range":{"start_line":975,"start_character":65,"end_line":976,"end_character":56},"in_reply_to":"026770ac_06248145","updated":"2025-07-16 19:38:32.000000000","message":"Acknowledged","commit_id":"f88601cb2c8651282fc4d9857e6cc6d54dd193d0"},{"author":{"_account_id":35676,"name":"Max","email":"max.lamprecht@digits.schwarz","username":"lamprechtm"},"change_message_id":"e39ed02bb1707a304526e17329a5130817130cc7","unresolved":true,"context_lines":[{"line_number":944,"context_line":"    def extend_volume(self, volume, new_size):"},{"line_number":945,"context_line":"        \"\"\"Extend an existing volume to the new size."},{"line_number":946,"context_line":""},{"line_number":947,"context_line":"           Encrypted volume extension is not supported"},{"line_number":948,"context_line":"        \"\"\""},{"line_number":949,"context_line":""},{"line_number":950,"context_line":"        LOG.info(\u0027Extending volume %s.\u0027, volume[\u0027name\u0027])"}],"source_content_type":"text/x-python","patch_set":3,"id":"790ddf6d_64406895","line":947,"updated":"2026-04-07 13:16:56.000000000","message":"Is there a reason why we don\u0027t support extend for encrypted volumes?","commit_id":"11bc25c1267dd7f47ec9284e66c7d875a7511981"},{"author":{"_account_id":34034,"name":"Franciszek Przewoźny","display_name":"Franciszek Przewozny","email":"przewozny.franciszek@gmail.com","username":"fprzewozn","status":"OpenStack Engineer / SRE @ Opera Software"},"change_message_id":"d36e74dc73411b610112d0525dfcc5ef49f586b1","unresolved":true,"context_lines":[{"line_number":944,"context_line":"    def extend_volume(self, volume, new_size):"},{"line_number":945,"context_line":"        \"\"\"Extend an existing volume to the new size."},{"line_number":946,"context_line":""},{"line_number":947,"context_line":"           Encrypted volume extension is not supported"},{"line_number":948,"context_line":"        \"\"\""},{"line_number":949,"context_line":""},{"line_number":950,"context_line":"        LOG.info(\u0027Extending volume %s.\u0027, volume[\u0027name\u0027])"}],"source_content_type":"text/x-python","patch_set":3,"id":"3c6e26e9_b572b0a4","line":947,"in_reply_to":"3f5bf9f8_1d529d60","updated":"2026-07-22 06:13:16.000000000","message":"@max.lamprecht@digits.schwarz As I wrote above - currently my setups allows only for testing non-encrypted volumes. I\u0027m not comfortable with proposing a change I cannot fully test. Once someone wth capable setup is interested in encrypted volumes extend, it will be much easier to go forward having the current change already in place.","commit_id":"11bc25c1267dd7f47ec9284e66c7d875a7511981"},{"author":{"_account_id":38059,"name":"Anoop Kumar Shukla","display_name":"Anoop Shukla","email":"anoop.shukla@netapp.com","username":"anoop2","status":"NetApp"},"change_message_id":"039e30ac5310709f3a681ebabbefb3a20a522e81","unresolved":true,"context_lines":[{"line_number":944,"context_line":"    def extend_volume(self, volume, new_size):"},{"line_number":945,"context_line":"        \"\"\"Extend an existing volume to the new size."},{"line_number":946,"context_line":""},{"line_number":947,"context_line":"           Encrypted volume extension is not supported"},{"line_number":948,"context_line":"        \"\"\""},{"line_number":949,"context_line":""},{"line_number":950,"context_line":"        LOG.info(\u0027Extending volume %s.\u0027, volume[\u0027name\u0027])"}],"source_content_type":"text/x-python","patch_set":3,"id":"f7bf0eeb_33c47d4e","line":947,"in_reply_to":"790ddf6d_64406895","updated":"2026-04-16 15:19:21.000000000","message":"+1 ONTAP supports expanding/extending the encrypted volumes for eg via NVE. Is this limitation coming due to attachment?","commit_id":"11bc25c1267dd7f47ec9284e66c7d875a7511981"},{"author":{"_account_id":35676,"name":"Max","email":"max.lamprecht@digits.schwarz","username":"lamprechtm"},"change_message_id":"83a3516512b831ca0e4db578b0f067d48b0f782f","unresolved":true,"context_lines":[{"line_number":944,"context_line":"    def extend_volume(self, volume, new_size):"},{"line_number":945,"context_line":"        \"\"\"Extend an existing volume to the new size."},{"line_number":946,"context_line":""},{"line_number":947,"context_line":"           Encrypted volume extension is not supported"},{"line_number":948,"context_line":"        \"\"\""},{"line_number":949,"context_line":""},{"line_number":950,"context_line":"        LOG.info(\u0027Extending volume %s.\u0027, volume[\u0027name\u0027])"}],"source_content_type":"text/x-python","patch_set":3,"id":"3f5bf9f8_1d529d60","line":947,"in_reply_to":"ac1ca13e_e5470052","updated":"2026-07-06 07:43:27.000000000","message":"At least we have a similar patch like this since multiple years in production. On our side this works also with \"encrypted volumes\". So I\u0027m not sure if we should introduce a limitation without mentioning why we need this.","commit_id":"11bc25c1267dd7f47ec9284e66c7d875a7511981"},{"author":{"_account_id":34034,"name":"Franciszek Przewoźny","display_name":"Franciszek Przewozny","email":"przewozny.franciszek@gmail.com","username":"fprzewozn","status":"OpenStack Engineer / SRE @ Opera Software"},"change_message_id":"d7495c78612cd7fa99ca66ce65d83d73d16b2236","unresolved":true,"context_lines":[{"line_number":944,"context_line":"    def extend_volume(self, volume, new_size):"},{"line_number":945,"context_line":"        \"\"\"Extend an existing volume to the new size."},{"line_number":946,"context_line":""},{"line_number":947,"context_line":"           Encrypted volume extension is not supported"},{"line_number":948,"context_line":"        \"\"\""},{"line_number":949,"context_line":""},{"line_number":950,"context_line":"        LOG.info(\u0027Extending volume %s.\u0027, volume[\u0027name\u0027])"}],"source_content_type":"text/x-python","patch_set":3,"id":"ac1ca13e_e5470052","line":947,"in_reply_to":"f7bf0eeb_33c47d4e","updated":"2026-06-24 10:19:43.000000000","message":"Hey @anoop.shukla@netapp.com change was initially prepared like this by @jayaanand.borra@netapp.com and for some reason this limitation was introduced. My setup currently allows me only to test this change as is (without encrypted volumes). Could we split the encrypted volumes expansion to a separate change done later? A number of deployments would benefit from this change even without encrypted volumes supported for now. Thanks","commit_id":"11bc25c1267dd7f47ec9284e66c7d875a7511981"}]}