)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"eca52bc711cc61799f73955a4f732a5124329b62","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":4,"id":"1289724a_e56e9696","updated":"2026-09-04 23:26:15.000000000","message":"I\u0027m trying to wrap my head around what is going on here.  Change I3358109b1c6997961c1cb73e114dd938fb9ff1e2 (in hibiscus, not yet released) added cross-pool clone support for NetApp iscsi/fc and made the driver report clone_across_pools: True.  But apparently it doesn\u0027t work well, so this patch has the driver report clone_across_pools: False.  There is another change I4941197a2df741f80c317fb9e270f29f6fb085b2 (in the cinder main code, not driver code) currently under review that introduces an option to control whether drivers who report they can clone_across_pools will actually do it.  Is there a patch that turns clone_across_pools on for any netapp drivers?\n\nI\u0027m asking because I share Rajat\u0027s concerns about the release note.  See comments inline.\n\nI\u0027m also unclear on what exactly the change in client_cmode.py is for.  It seems to be fixing some kind of bug, but it\u0027s not clear from the commit message what exactly the issue is.","commit_id":"a9bf30f7b6293b3113af7141aeabeca42da8ade4"},{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"5f6c456e18552df6ed8d0a6ce6d92c528f7dd9ed","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"a086aa3b_8f1140e5","updated":"2026-09-03 11:52:24.000000000","message":"I\u0027ve concerns noted inline, i still feel [1] is more straightforward i.e. to not pass vserver details in case of clone across pools.\n\n[1] https://review.opendev.org/c/openstack/cinder/+/1003076","commit_id":"a9bf30f7b6293b3113af7141aeabeca42da8ade4"},{"author":{"_account_id":36171,"name":"jayaanand borra","display_name":"jayaanand borra","email":"jayaanand.borra@netapp.com","username":"jayaanan","status":"netapp"},"change_message_id":"c49b31261afb82df4d9256c7b7c52b89c233c3d8","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"6ec84c04_4a4a179e","updated":"2026-09-03 07:44:29.000000000","message":"we will enable cross-pool cache capability as part of V2 release","commit_id":"a9bf30f7b6293b3113af7141aeabeca42da8ade4"}],"cinder/volume/drivers/netapp/dataontap/block_cmode.py":[{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"5f6c456e18552df6ed8d0a6ce6d92c528f7dd9ed","unresolved":true,"context_lines":[{"line_number":679,"context_line":"        # Used for service state report"},{"line_number":680,"context_line":"        data[\u0027replication_enabled\u0027] \u003d self.replication_enabled"},{"line_number":681,"context_line":""},{"line_number":682,"context_line":"        data[\u0027clone_across_pools\u0027] \u003d False"},{"line_number":683,"context_line":""},{"line_number":684,"context_line":"        # Surface disaggregated\\-platform flag at the backend level"},{"line_number":685,"context_line":"        data[\u0027netapp_disaggregated_platform\u0027] \u003d ("}],"source_content_type":"text/x-python","patch_set":4,"id":"163a9292_ff00d1d2","line":682,"range":{"start_line":682,"start_character":8,"end_line":682,"end_character":42},"updated":"2026-09-03 11:52:24.000000000","message":"why are we disabling it here? this will cause the optimized path to never get triggered, am i missing something?","commit_id":"a9bf30f7b6293b3113af7141aeabeca42da8ade4"},{"author":{"_account_id":38059,"name":"Anoop Kumar Shukla","display_name":"Anoop Shukla","email":"anoop.shukla@netapp.com","username":"anoop2","status":"NetApp"},"change_message_id":"4e56557239d5dfaf047dd8b1e5cb1458c854a8de","unresolved":true,"context_lines":[{"line_number":679,"context_line":"        # Used for service state report"},{"line_number":680,"context_line":"        data[\u0027replication_enabled\u0027] \u003d self.replication_enabled"},{"line_number":681,"context_line":""},{"line_number":682,"context_line":"        data[\u0027clone_across_pools\u0027] \u003d False"},{"line_number":683,"context_line":""},{"line_number":684,"context_line":"        # Surface disaggregated\\-platform flag at the backend level"},{"line_number":685,"context_line":"        data[\u0027netapp_disaggregated_platform\u0027] \u003d ("}],"source_content_type":"text/x-python","patch_set":4,"id":"8f223f33_18abb54b","line":682,"range":{"start_line":682,"start_character":8,"end_line":682,"end_character":42},"in_reply_to":"029526e2_6d7b6037","updated":"2026-09-03 14:16:33.000000000","message":"Customers are used to warm the pool with cache manually before creating the VMs. Forcing them to go to copy path for all the VM creations, ends up taking time for bulk volume provisioning. Cross pool cache functionality for storage solutions like NetApp where the FlexVolumes can come from aggregates sitting on different node needs cloning solution. This goes back to the manual image copy operations done in older releases using Copy Offload tool. So IMO, unless we optimize the cross pool copy to use clones, forcing customers to go via a copy operation does not help.","commit_id":"a9bf30f7b6293b3113af7141aeabeca42da8ade4"},{"author":{"_account_id":38059,"name":"Anoop Kumar Shukla","display_name":"Anoop Shukla","email":"anoop.shukla@netapp.com","username":"anoop2","status":"NetApp"},"change_message_id":"f250edf97ecd36b6a30114b56c02e8135eefb2f5","unresolved":true,"context_lines":[{"line_number":679,"context_line":"        # Used for service state report"},{"line_number":680,"context_line":"        data[\u0027replication_enabled\u0027] \u003d self.replication_enabled"},{"line_number":681,"context_line":""},{"line_number":682,"context_line":"        data[\u0027clone_across_pools\u0027] \u003d False"},{"line_number":683,"context_line":""},{"line_number":684,"context_line":"        # Surface disaggregated\\-platform flag at the backend level"},{"line_number":685,"context_line":"        data[\u0027netapp_disaggregated_platform\u0027] \u003d ("}],"source_content_type":"text/x-python","patch_set":4,"id":"77ee6a6f_cc42bdcb","line":682,"range":{"start_line":682,"start_character":8,"end_line":682,"end_character":42},"in_reply_to":"163a9292_ff00d1d2","updated":"2026-09-03 12:18:18.000000000","message":"Cross Pool Caching feature from NetApp driver needs clone optimization. We would like to enable it as part of the upstream patch: https://review.opendev.org/c/openstack/cinder/+/988315 merge.","commit_id":"a9bf30f7b6293b3113af7141aeabeca42da8ade4"},{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"4b75e775d46e1ea0e169b24b87a02386dc33fe39","unresolved":true,"context_lines":[{"line_number":679,"context_line":"        # Used for service state report"},{"line_number":680,"context_line":"        data[\u0027replication_enabled\u0027] \u003d self.replication_enabled"},{"line_number":681,"context_line":""},{"line_number":682,"context_line":"        data[\u0027clone_across_pools\u0027] \u003d False"},{"line_number":683,"context_line":""},{"line_number":684,"context_line":"        # Surface disaggregated\\-platform flag at the backend level"},{"line_number":685,"context_line":"        data[\u0027netapp_disaggregated_platform\u0027] \u003d ("}],"source_content_type":"text/x-python","patch_set":4,"id":"00363684_4b5f5c21","line":682,"range":{"start_line":682,"start_character":8,"end_line":682,"end_character":42},"in_reply_to":"693a9bda_7fb0a5c7","updated":"2026-09-04 04:16:28.000000000","message":"Please note that this patch is in upstream space and the decisions are made taking community into mind so I would kindly ask to refrain using RH and customer centric wording.\nFrom the upstream perspective, if there are any issues in a feature, we fix the issues rather than disabling it hence I am personally against the idea here.\nHaving said that, this is the NetApp driver and the vendor makes the final decision so I don\u0027t mind going forward with this.","commit_id":"a9bf30f7b6293b3113af7141aeabeca42da8ade4"},{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"49d00a82eaab08892978b4b7dfd543749a70f994","unresolved":true,"context_lines":[{"line_number":679,"context_line":"        # Used for service state report"},{"line_number":680,"context_line":"        data[\u0027replication_enabled\u0027] \u003d self.replication_enabled"},{"line_number":681,"context_line":""},{"line_number":682,"context_line":"        data[\u0027clone_across_pools\u0027] \u003d False"},{"line_number":683,"context_line":""},{"line_number":684,"context_line":"        # Surface disaggregated\\-platform flag at the backend level"},{"line_number":685,"context_line":"        data[\u0027netapp_disaggregated_platform\u0027] \u003d ("}],"source_content_type":"text/x-python","patch_set":4,"id":"029526e2_6d7b6037","line":682,"range":{"start_line":682,"start_character":8,"end_line":682,"end_character":42},"in_reply_to":"77ee6a6f_cc42bdcb","updated":"2026-09-03 13:06:08.000000000","message":"I would disagree here since the feature as it is works, its not as efficient as cloning is but surely is faster than image download path.\nIf we merge this and the second patch sits for months (ideally it shouldn\u0027t but that\u0027s how things work upstream), i feel we are disabling the functionality for the best case scenario but ending up with the worst case scenario in the current code.","commit_id":"a9bf30f7b6293b3113af7141aeabeca42da8ade4"},{"author":{"_account_id":38106,"name":"Venkatesh","display_name":"venkythegreat","email":"Venkatesh.Varadan@netapp.com","username":"venkythegreat"},"change_message_id":"28344e639b98f8d3425ee4f8bd1e3b9ad041a6fe","unresolved":true,"context_lines":[{"line_number":679,"context_line":"        # Used for service state report"},{"line_number":680,"context_line":"        data[\u0027replication_enabled\u0027] \u003d self.replication_enabled"},{"line_number":681,"context_line":""},{"line_number":682,"context_line":"        data[\u0027clone_across_pools\u0027] \u003d False"},{"line_number":683,"context_line":""},{"line_number":684,"context_line":"        # Surface disaggregated\\-platform flag at the backend level"},{"line_number":685,"context_line":"        data[\u0027netapp_disaggregated_platform\u0027] \u003d ("}],"source_content_type":"text/x-python","patch_set":4,"id":"693a9bda_7fb0a5c7","line":682,"range":{"start_line":682,"start_character":8,"end_line":682,"end_character":42},"in_reply_to":"8f223f33_18abb54b","updated":"2026-09-03 14:29:34.000000000","message":"@rajatdhasmana@gmail.com\n\nw.r.t your comment \u0027If we merge this and the second patch sits for months (ideally it shouldn\u0027t but that\u0027s how things work upstream)\u0027\n\nthe second patch with optimization was presented in PTG in detail citing the limitations of the first patch, back in Apr 2026 and this review PR has been available for the last three months already with upstream review with very little progess and alignment. As you suggested, if this is going to take more months, then it is ideal to have an option to disable this functionality as providing a comprehensive solution is better than providing an non efficient solution. \n\nBesides the customer we are working with, is aligned to get this disabled and asked us to raise this patch and colloborate with RH in getting this approved at the earliest. \n\nso pls consider this request from NetApp to disable this functionality in our driver code, considering the customer ask.","commit_id":"a9bf30f7b6293b3113af7141aeabeca42da8ade4"},{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"5f6c456e18552df6ed8d0a6ce6d92c528f7dd9ed","unresolved":true,"context_lines":[{"line_number":750,"context_line":"                self.configuration.safe_get(\u0027netapp_disaggregated_platform\u0027)"},{"line_number":751,"context_line":"            )"},{"line_number":752,"context_line":"            pool[\u0027reserved_percentage\u0027] \u003d self.reserved_percentage"},{"line_number":753,"context_line":"            pool[\u0027clone_across_pools\u0027] \u003d False"},{"line_number":754,"context_line":"            pool[\u0027max_over_subscription_ratio\u0027] \u003d ("},{"line_number":755,"context_line":"                self.max_over_subscription_ratio)"},{"line_number":756,"context_line":"            if self.configuration.netapp_disaggregated_platform:"}],"source_content_type":"text/x-python","patch_set":4,"id":"3c7c3930_8f0d1e48","line":753,"range":{"start_line":753,"start_character":0,"end_line":753,"end_character":46},"updated":"2026-09-03 11:52:24.000000000","message":"same","commit_id":"a9bf30f7b6293b3113af7141aeabeca42da8ade4"}],"cinder/volume/drivers/netapp/dataontap/client/client_cmode.py":[{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"5f6c456e18552df6ed8d0a6ce6d92c528f7dd9ed","unresolved":true,"context_lines":[{"line_number":825,"context_line":"            dest_lun_name \u003d lun_name"},{"line_number":826,"context_line":""},{"line_number":827,"context_line":"        api_args \u003d {"},{"line_number":828,"context_line":"            \u0027source-vserver\u0027: src_vserver,"},{"line_number":829,"context_line":"            \u0027paths\u0027: [{"},{"line_number":830,"context_line":"                \u0027lun-path-pair\u0027: {"},{"line_number":831,"context_line":"                    \u0027destination-path\u0027: \u0027/vol/%s/%s\u0027 % (dest_ontap_volume,"}],"source_content_type":"text/x-python","patch_set":4,"id":"fb3410f2_af9b5f21","line":828,"range":{"start_line":828,"start_character":12,"end_line":828,"end_character":42},"updated":"2026-09-03 11:52:24.000000000","message":"is the issue only with destination vserver? is source-vserver acceptable with the SVM scoped tokens?","commit_id":"a9bf30f7b6293b3113af7141aeabeca42da8ade4"},{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"49d00a82eaab08892978b4b7dfd543749a70f994","unresolved":false,"context_lines":[{"line_number":825,"context_line":"            dest_lun_name \u003d lun_name"},{"line_number":826,"context_line":""},{"line_number":827,"context_line":"        api_args \u003d {"},{"line_number":828,"context_line":"            \u0027source-vserver\u0027: src_vserver,"},{"line_number":829,"context_line":"            \u0027paths\u0027: [{"},{"line_number":830,"context_line":"                \u0027lun-path-pair\u0027: {"},{"line_number":831,"context_line":"                    \u0027destination-path\u0027: \u0027/vol/%s/%s\u0027 % (dest_ontap_volume,"}],"source_content_type":"text/x-python","patch_set":4,"id":"44815dfe_07c4f6e1","line":828,"range":{"start_line":828,"start_character":12,"end_line":828,"end_character":42},"in_reply_to":"1060c4a2_0acdfa15","updated":"2026-09-03 13:06:08.000000000","message":"Acknowledged","commit_id":"a9bf30f7b6293b3113af7141aeabeca42da8ade4"},{"author":{"_account_id":38059,"name":"Anoop Kumar Shukla","display_name":"Anoop Shukla","email":"anoop.shukla@netapp.com","username":"anoop2","status":"NetApp"},"change_message_id":"d3e1dc7c8f51b9a33df76c67c45b1c8f27eb6699","unresolved":true,"context_lines":[{"line_number":825,"context_line":"            dest_lun_name \u003d lun_name"},{"line_number":826,"context_line":""},{"line_number":827,"context_line":"        api_args \u003d {"},{"line_number":828,"context_line":"            \u0027source-vserver\u0027: src_vserver,"},{"line_number":829,"context_line":"            \u0027paths\u0027: [{"},{"line_number":830,"context_line":"                \u0027lun-path-pair\u0027: {"},{"line_number":831,"context_line":"                    \u0027destination-path\u0027: \u0027/vol/%s/%s\u0027 % (dest_ontap_volume,"}],"source_content_type":"text/x-python","patch_set":4,"id":"1060c4a2_0acdfa15","line":828,"range":{"start_line":828,"start_character":12,"end_line":828,"end_character":42},"in_reply_to":"fb3410f2_af9b5f21","updated":"2026-09-03 12:00:51.000000000","message":"Yes. Source vserver is accepted by the API when on SVM scope. It does not take the destination as the API scoping is to the single SVM.","commit_id":"a9bf30f7b6293b3113af7141aeabeca42da8ade4"},{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"5f6c456e18552df6ed8d0a6ce6d92c528f7dd9ed","unresolved":true,"context_lines":[{"line_number":839,"context_line":"        # context, so the field is extra input (ONTAP 13115). Intra-SVM"},{"line_number":840,"context_line":"        # copies still need it for cluster-admin (ONTAP 13006)."},{"line_number":841,"context_line":"        # https://docs.netapp.com/us-en/ontap-cli/lun-copy-start.html"},{"line_number":842,"context_line":"        if dest_vserver and (self.features.CLUSTER_CREDENTIALS or"},{"line_number":843,"context_line":"                             dest_vserver !\u003d src_vserver):"},{"line_number":844,"context_line":"            api_args[\u0027destination-vserver\u0027] \u003d dest_vserver"},{"line_number":845,"context_line":"        result \u003d self.connection.send_request(\u0027lun-copy-start\u0027, api_args,"},{"line_number":846,"context_line":"                                              enable_tunneling\u003dFalse)"}],"source_content_type":"text/x-python","patch_set":4,"id":"ce8dc4c0_7a5e725e","line":843,"range":{"start_line":842,"start_character":0,"end_line":843,"end_character":58},"updated":"2026-09-03 11:52:24.000000000","message":"I\u0027m unsure if this is the right way to handle this.\nLet me run through some scenarios:\n1. if we\u0027re in the migration path from one backend (SVM1) to another (SVM2) - this requires both conditions\na) cluster credentials\nb) dest_vserver is not equal to src_vserver\nso this might fail in the case when I\u0027m requesting a migration and cluster credentials are not set but dst vserver !\u003d src vserver -- this condition evaluates to true but will fail in schema validation later\n\n2. clone across pool currently only works within same SVM: if we have cluster credentials and dest_vserver is passed, what change does it bring for the clone across pool scenario? we know it\u0027s always within same SVM so either SVM scoped or cluster scoped, we should not be passing destination-vserver parameter.","commit_id":"a9bf30f7b6293b3113af7141aeabeca42da8ade4"},{"author":{"_account_id":38059,"name":"Anoop Kumar Shukla","display_name":"Anoop Shukla","email":"anoop.shukla@netapp.com","username":"anoop2","status":"NetApp"},"change_message_id":"d3e1dc7c8f51b9a33df76c67c45b1c8f27eb6699","unresolved":true,"context_lines":[{"line_number":839,"context_line":"        # context, so the field is extra input (ONTAP 13115). Intra-SVM"},{"line_number":840,"context_line":"        # copies still need it for cluster-admin (ONTAP 13006)."},{"line_number":841,"context_line":"        # https://docs.netapp.com/us-en/ontap-cli/lun-copy-start.html"},{"line_number":842,"context_line":"        if dest_vserver and (self.features.CLUSTER_CREDENTIALS or"},{"line_number":843,"context_line":"                             dest_vserver !\u003d src_vserver):"},{"line_number":844,"context_line":"            api_args[\u0027destination-vserver\u0027] \u003d dest_vserver"},{"line_number":845,"context_line":"        result \u003d self.connection.send_request(\u0027lun-copy-start\u0027, api_args,"},{"line_number":846,"context_line":"                                              enable_tunneling\u003dFalse)"}],"source_content_type":"text/x-python","patch_set":4,"id":"eec6fc8e_1ce86a7f","line":843,"range":{"start_line":842,"start_character":0,"end_line":843,"end_character":58},"in_reply_to":"ce8dc4c0_7a5e725e","updated":"2026-09-03 12:00:51.000000000","message":"The fix is done based on the API behaviour. Even in case of migration, SVM scoped credentials should not add the destination vserver as it will never be supported at SVM scope. So ideally, a migration to a different SVM using SVM credential should not be supported, and we may need to except before we reach here.","commit_id":"a9bf30f7b6293b3113af7141aeabeca42da8ade4"},{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"49d00a82eaab08892978b4b7dfd543749a70f994","unresolved":false,"context_lines":[{"line_number":839,"context_line":"        # context, so the field is extra input (ONTAP 13115). Intra-SVM"},{"line_number":840,"context_line":"        # copies still need it for cluster-admin (ONTAP 13006)."},{"line_number":841,"context_line":"        # https://docs.netapp.com/us-en/ontap-cli/lun-copy-start.html"},{"line_number":842,"context_line":"        if dest_vserver and (self.features.CLUSTER_CREDENTIALS or"},{"line_number":843,"context_line":"                             dest_vserver !\u003d src_vserver):"},{"line_number":844,"context_line":"            api_args[\u0027destination-vserver\u0027] \u003d dest_vserver"},{"line_number":845,"context_line":"        result \u003d self.connection.send_request(\u0027lun-copy-start\u0027, api_args,"},{"line_number":846,"context_line":"                                              enable_tunneling\u003dFalse)"}],"source_content_type":"text/x-python","patch_set":4,"id":"c935ee0b_05f9506f","line":843,"range":{"start_line":842,"start_character":0,"end_line":843,"end_character":58},"in_reply_to":"eec6fc8e_1ce86a7f","updated":"2026-09-03 13:06:08.000000000","message":"Acknowledged","commit_id":"a9bf30f7b6293b3113af7141aeabeca42da8ade4"},{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"eed48b79806d8fffbb3ecafc763e132cf090ae2a","unresolved":true,"context_lines":[{"line_number":839,"context_line":"        # context, so the field is extra input (ONTAP 13115). Intra-SVM"},{"line_number":840,"context_line":"        # copies still need it for cluster-admin (ONTAP 13006)."},{"line_number":841,"context_line":"        # https://docs.netapp.com/us-en/ontap-cli/lun-copy-start.html"},{"line_number":842,"context_line":"        if dest_vserver and (self.features.CLUSTER_CREDENTIALS or"},{"line_number":843,"context_line":"                             dest_vserver !\u003d src_vserver):"},{"line_number":844,"context_line":"            api_args[\u0027destination-vserver\u0027] \u003d dest_vserver"},{"line_number":845,"context_line":"        result \u003d self.connection.send_request(\u0027lun-copy-start\u0027, api_args,"},{"line_number":846,"context_line":"                                              enable_tunneling\u003dFalse)"}],"source_content_type":"text/x-python","patch_set":5,"id":"e134400e_eec79c51","line":843,"range":{"start_line":842,"start_character":0,"end_line":843,"end_character":58},"updated":"2026-09-07 19:10:54.000000000","message":"I still don\u0027t understand this condition completely\nIf we have dest_vserver and dest_vserver !\u003d src_vserver, the request will pass \u0027destination-vserver\u0027 even if we have SVM scoped credentials which is supposed to fail right?","commit_id":"3029d36062402ff77baa70f931bed222e5597c58"}],"releasenotes/notes/netapp-cross-pool-clone-6fe47467ffe9a7a2.yaml":[{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"95e8753b3e6e067a4e297a6c36c1149dd04cf2eb","unresolved":true,"context_lines":[{"line_number":22,"context_line":"    * **Same FlexVol**: Uses efficient FlexClone (instant clone)"},{"line_number":23,"context_line":"    * **Cross-aggregate**: Uses ONTAP LUN copy operation"},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"    This feature works with both ZAPI and REST API clients. The"},{"line_number":26,"context_line":"    ``clone_across_pools`` capability is reported as ``False``, so the"},{"line_number":27,"context_line":"    image-volume cache and Glance volume-backed images do not clone"},{"line_number":28,"context_line":"    across pools. Same-pool FlexClone is unchanged."},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"    For more details about the clone_across_pools capability, please see"},{"line_number":31,"context_line":"    `bp clone_across_pools"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"c774c118_e0e3ff31","line":28,"range":{"start_line":25,"start_character":60,"end_line":28,"end_character":51},"updated":"2026-09-04 05:51:02.000000000","message":"there is no mention of why this is set to disabled.","commit_id":"a9bf30f7b6293b3113af7141aeabeca42da8ade4"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"eca52bc711cc61799f73955a4f732a5124329b62","unresolved":true,"context_lines":[{"line_number":22,"context_line":"    * **Same FlexVol**: Uses efficient FlexClone (instant clone)"},{"line_number":23,"context_line":"    * **Cross-aggregate**: Uses ONTAP LUN copy operation"},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"    This feature works with both ZAPI and REST API clients. The"},{"line_number":26,"context_line":"    ``clone_across_pools`` capability is reported as ``False``, so the"},{"line_number":27,"context_line":"    image-volume cache and Glance volume-backed images do not clone"},{"line_number":28,"context_line":"    across pools. Same-pool FlexClone is unchanged."},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"    For more details about the clone_across_pools capability, please see"},{"line_number":31,"context_line":"    `bp clone_across_pools"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"27acb7fc_b304db49","line":28,"range":{"start_line":25,"start_character":60,"end_line":28,"end_character":51},"in_reply_to":"c774c118_e0e3ff31","updated":"2026-09-04 23:26:15.000000000","message":"Yes, it seems like this feature does NOT work with either ZAPI or REST API clients!  The capability is reported as False, and there is no way to change it other than hacking the driver code, right?\n\nPlease correct me if I\u0027m wrong, but it looks to me like setting clone_across_pools: False in the driver means that in this patch, you are withdrawing everything mentioned on lines 4-27 in the Base patch.  So basically, change I3358109b1c6997961c1cb73e114dd938fb9ff1e2 added some code related to cloning across pools, but with this patch, the driver no longer reports that it can do it, and thus none of the cinder main code that checks for the capability will use it.\n\nIf some of the code from I3358109b1c6997961c1cb73e114dd938fb9ff1e is used internally by the driver, you can mention and explain that in the release note, but otherwise I think this patch should just remove the release note entirely.","commit_id":"a9bf30f7b6293b3113af7141aeabeca42da8ade4"},{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"95e8753b3e6e067a4e297a6c36c1149dd04cf2eb","unresolved":true,"context_lines":[{"line_number":30,"context_line":"    For more details about the clone_across_pools capability, please see"},{"line_number":31,"context_line":"    `bp clone_across_pools"},{"line_number":32,"context_line":"    \u003chttps://blueprints.launchpad.net/cinder/+spec/clone-across-pools\u003e`__"},{"line_number":33,"context_line":"upgrade:"},{"line_number":34,"context_line":"  - |"},{"line_number":35,"context_line":"    NetApp ONTAP iSCSI and FC drivers: The ``clone_across_pools``"},{"line_number":36,"context_line":"    capability is reported as ``False``. Creating a volume from an"},{"line_number":37,"context_line":"    image-volume cache entry or a volume-backed Glance image that lives"},{"line_number":38,"context_line":"    in a different storage pool (FlexVol) downloads the image from"},{"line_number":39,"context_line":"    Glance instead of using ONTAP LUN copy. Same-pool FlexClone is"},{"line_number":40,"context_line":"    unchanged."}],"source_content_type":"text/x-yaml","patch_set":4,"id":"78d0e05c_7cdc3465","line":40,"range":{"start_line":33,"start_character":0,"end_line":40,"end_character":14},"updated":"2026-09-04 05:51:02.000000000","message":"the feature itself isn\u0027t released yet so this is very confusing. it was not in 2026.1 and 2026.2 is not released yet so operators don\u0027t really know about this feature and there shouldn\u0027t be any upgrade impact.","commit_id":"a9bf30f7b6293b3113af7141aeabeca42da8ade4"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"eca52bc711cc61799f73955a4f732a5124329b62","unresolved":true,"context_lines":[{"line_number":30,"context_line":"    For more details about the clone_across_pools capability, please see"},{"line_number":31,"context_line":"    `bp clone_across_pools"},{"line_number":32,"context_line":"    \u003chttps://blueprints.launchpad.net/cinder/+spec/clone-across-pools\u003e`__"},{"line_number":33,"context_line":"upgrade:"},{"line_number":34,"context_line":"  - |"},{"line_number":35,"context_line":"    NetApp ONTAP iSCSI and FC drivers: The ``clone_across_pools``"},{"line_number":36,"context_line":"    capability is reported as ``False``. Creating a volume from an"},{"line_number":37,"context_line":"    image-volume cache entry or a volume-backed Glance image that lives"},{"line_number":38,"context_line":"    in a different storage pool (FlexVol) downloads the image from"},{"line_number":39,"context_line":"    Glance instead of using ONTAP LUN copy. Same-pool FlexClone is"},{"line_number":40,"context_line":"    unchanged."}],"source_content_type":"text/x-yaml","patch_set":4,"id":"37b73971_7763fc22","line":40,"range":{"start_line":33,"start_character":0,"end_line":40,"end_character":14},"in_reply_to":"78d0e05c_7cdc3465","updated":"2026-09-04 23:26:15.000000000","message":"Agree with Rajat, the feature did not exist before the Hibiscus release, there is no way to turn it on or off using the config file, so it looks to me like there is really no upgrade impact.","commit_id":"a9bf30f7b6293b3113af7141aeabeca42da8ade4"}]}
