)]}'
{"/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":"081fa9d1baaa0244428938f299fe9f695b2b2bac","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"1ffb98b2_73c90b28","updated":"2026-10-20 08:49:48.000000000","message":"Run-NetApp CI","commit_id":"aa3ada0e3ee98b888d20277777676f765764d4ed"},{"author":{"_account_id":38059,"name":"Anoop Kumar Shukla","display_name":"Anoop Shukla","email":"anoop.shukla@netapp.com","username":"anoop2","status":"NetApp"},"change_message_id":"7843e1a651b4330171e0a2382ec0ec221786a847","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"3e15fc50_e0f91da0","updated":"2026-10-27 18:07:02.000000000","message":"Run-NetApp CI","commit_id":"aa3ada0e3ee98b888d20277777676f765764d4ed"},{"author":{"_account_id":39225,"name":"Prudhvi","display_name":"Prudhvi","email":"prudhvi.openstack@gmail.com","username":"Prudhvi"},"change_message_id":"c61e9c2c8cea12a80a88a6d54788d4c8230bdd7e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"78b7000b_1fcdb59c","updated":"2026-10-30 10:44:06.000000000","message":"Run-NetApp CI","commit_id":"aa3ada0e3ee98b888d20277777676f765764d4ed"},{"author":{"_account_id":38059,"name":"Anoop Kumar Shukla","display_name":"Anoop Shukla","email":"anoop.shukla@netapp.com","username":"anoop2","status":"NetApp"},"change_message_id":"8c93a2977011a775caba39c15550eaed3103d461","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"e6cd62b9_fd9b71e2","updated":"2026-10-30 06:49:20.000000000","message":"Run-NetApp CI","commit_id":"aa3ada0e3ee98b888d20277777676f765764d4ed"},{"author":{"_account_id":31779,"name":"Jean Pierre Roquesalane","display_name":"happystacker","email":"jeanpierre.roquesalane@dell.com","username":"happystacker"},"change_message_id":"84515bad7f55b1ac0798669a557c28b76588c9d4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"209e37d3_48d71056","updated":"2026-10-20 06:51:38.000000000","message":"This looks correct to me. Both source and destination FlexGroup cases now fall back to generic host-assisted migration, while same-backend destinations can be resolved through the local SSC. The accompanying tests cover the relevant paths","commit_id":"aa3ada0e3ee98b888d20277777676f765764d4ed"},{"author":{"_account_id":38623,"name":"Sanketh Shetty","display_name":"Sanketh Shetty","email":"sanketh.shetty@netapp.com","username":"sanketh-netapp","status":"NetApp"},"change_message_id":"19daefe081c7a91d54c5376f0a769f6ec810d1d3","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"c7dc0251_6edab2a0","updated":"2026-11-04 10:59:09.000000000","message":"Looks good.","commit_id":"49215c58af9d3efda82c1c251f612d69d339d9c1"},{"author":{"_account_id":36180,"name":"Gireesh Awasthi","display_name":"Gireesh","email":"gawasthi2010@gmail.com","username":"agireesh","status":"NetApp"},"change_message_id":"3d16251c00e131df4323c80dfe7cbdab33dde417","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"2465eddb_c89433b6","updated":"2026-11-11 10:13:37.000000000","message":"Thanks @anoop.shukla@netapp.com for working on this. Added few comments, please take a look.","commit_id":"49215c58af9d3efda82c1c251f612d69d339d9c1"},{"author":{"_account_id":38059,"name":"Anoop Kumar Shukla","display_name":"Anoop Shukla","email":"anoop.shukla@netapp.com","username":"anoop2","status":"NetApp"},"change_message_id":"b4e6535b71c13df13a815eb20797d686c16bd9f5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"b6de8114_74c6e9e2","updated":"2026-11-04 14:18:09.000000000","message":"recheck","commit_id":"49215c58af9d3efda82c1c251f612d69d339d9c1"}],"cinder/volume/drivers/netapp/dataontap/nfs_cmode.py":[{"author":{"_account_id":38623,"name":"Sanketh Shetty","display_name":"Sanketh Shetty","email":"sanketh.shetty@netapp.com","username":"sanketh-netapp","status":"NetApp"},"change_message_id":"5cb558b3fa47167d2ba5d99058000ed766ce4e63","unresolved":true,"context_lines":[{"line_number":1360,"context_line":"                     \"assisted migration.\")"},{"line_number":1361,"context_line":"            return False, {}"},{"line_number":1362,"context_line":""},{"line_number":1363,"context_line":"        if self._is_flexgroup_dest(host):"},{"line_number":1364,"context_line":"            LOG.info(\"Storage assisted migration is not supported to a \""},{"line_number":1365,"context_line":"                     \"FlexGroup pool. Falling back to host assisted \""},{"line_number":1366,"context_line":"                     \"migration.\")"}],"source_content_type":"text/x-python","patch_set":1,"id":"86490652_a5d3655e","line":1363,"updated":"2026-11-02 14:43:28.000000000","message":"Did not understand why we did not directly use _is_flexgroup(host\u003dhost) here ?","commit_id":"aa3ada0e3ee98b888d20277777676f765764d4ed"},{"author":{"_account_id":38059,"name":"Anoop Kumar Shukla","display_name":"Anoop Shukla","email":"anoop.shukla@netapp.com","username":"anoop2","status":"NetApp"},"change_message_id":"18d1048dce2d9597f752ee43308a9745587c44f7","unresolved":false,"context_lines":[{"line_number":1360,"context_line":"                     \"assisted migration.\")"},{"line_number":1361,"context_line":"            return False, {}"},{"line_number":1362,"context_line":""},{"line_number":1363,"context_line":"        if self._is_flexgroup_dest(host):"},{"line_number":1364,"context_line":"            LOG.info(\"Storage assisted migration is not supported to a \""},{"line_number":1365,"context_line":"                     \"FlexGroup pool. Falling back to host assisted \""},{"line_number":1366,"context_line":"                     \"migration.\")"}],"source_content_type":"text/x-python","patch_set":1,"id":"4883c0d7_f6df5e57","line":1363,"in_reply_to":"86490652_a5d3655e","updated":"2026-11-03 11:37:46.000000000","message":"Fixed now.","commit_id":"aa3ada0e3ee98b888d20277777676f765764d4ed"},{"author":{"_account_id":36180,"name":"Gireesh Awasthi","display_name":"Gireesh","email":"gawasthi2010@gmail.com","username":"agireesh","status":"NetApp"},"change_message_id":"3d16251c00e131df4323c80dfe7cbdab33dde417","unresolved":true,"context_lines":[{"line_number":1184,"context_line":"        backend. The local SSC catalog is consulted only for this backend."},{"line_number":1185,"context_line":"        \"\"\""},{"line_number":1186,"context_line":"        capabilities \u003d capabilities or {}"},{"line_number":1187,"context_line":"        if capabilities.get(\u0027netapp_is_flexgroup\u0027) \u003d\u003d \u0027true\u0027:"},{"line_number":1188,"context_line":"            return True"},{"line_number":1189,"context_line":""},{"line_number":1190,"context_line":"        backend \u003d volume_utils.extract_host(host, level\u003d\u0027backend\u0027)"},{"line_number":1191,"context_line":"        _, _, backend_name \u003d backend.partition(\u0027@\u0027)"}],"source_content_type":"text/x-python","patch_set":2,"id":"e5d044f7_34012606","line":1188,"range":{"start_line":1187,"start_character":8,"end_line":1188,"end_character":23},"updated":"2026-11-11 10:13:37.000000000","message":"Change this code as below for safer side.\n return str(capabilities[\u0027netapp_is_flexgroup\u0027]).lower() \u003d\u003d \u0027true\u0027","commit_id":"49215c58af9d3efda82c1c251f612d69d339d9c1"},{"author":{"_account_id":36180,"name":"Gireesh Awasthi","display_name":"Gireesh","email":"gawasthi2010@gmail.com","username":"agireesh","status":"NetApp"},"change_message_id":"3d16251c00e131df4323c80dfe7cbdab33dde417","unresolved":true,"context_lines":[{"line_number":1189,"context_line":""},{"line_number":1190,"context_line":"        backend \u003d volume_utils.extract_host(host, level\u003d\u0027backend\u0027)"},{"line_number":1191,"context_line":"        _, _, backend_name \u003d backend.partition(\u0027@\u0027)"},{"line_number":1192,"context_line":"        if backend_name !\u003d self.backend_name:"},{"line_number":1193,"context_line":"            return False"},{"line_number":1194,"context_line":"        return self._is_flexgroup(host\u003dhost)"},{"line_number":1195,"context_line":""},{"line_number":1196,"context_line":"    def _is_flexgroup_clone_file_supported(self):"}],"source_content_type":"text/x-python","patch_set":2,"id":"0b92bc66_83e5a3cd","line":1193,"range":{"start_line":1192,"start_character":8,"end_line":1193,"end_character":24},"updated":"2026-11-11 10:13:37.000000000","message":"Are we setting netapp_is_flexgroup capability always ..? if capability is missing and in this case this method will always return false for below condition for migration","commit_id":"49215c58af9d3efda82c1c251f612d69d339d9c1"},{"author":{"_account_id":38059,"name":"Anoop Kumar Shukla","display_name":"Anoop Shukla","email":"anoop.shukla@netapp.com","username":"anoop2","status":"NetApp"},"change_message_id":"16f08a36123aecd163128a6adbf61485e13ac299","unresolved":true,"context_lines":[{"line_number":1189,"context_line":""},{"line_number":1190,"context_line":"        backend \u003d volume_utils.extract_host(host, level\u003d\u0027backend\u0027)"},{"line_number":1191,"context_line":"        _, _, backend_name \u003d backend.partition(\u0027@\u0027)"},{"line_number":1192,"context_line":"        if backend_name !\u003d self.backend_name:"},{"line_number":1193,"context_line":"            return False"},{"line_number":1194,"context_line":"        return self._is_flexgroup(host\u003dhost)"},{"line_number":1195,"context_line":""},{"line_number":1196,"context_line":"    def _is_flexgroup_clone_file_supported(self):"}],"source_content_type":"text/x-python","patch_set":2,"id":"c15f870a_8a4d364d","line":1193,"range":{"start_line":1192,"start_character":8,"end_line":1193,"end_character":24},"in_reply_to":"0b92bc66_83e5a3cd","updated":"2026-11-11 10:32:04.000000000","message":"If the pool is not a flexgroup - its supposed to always return false. so both the cases of missing capability and it being false is addressed right?","commit_id":"49215c58af9d3efda82c1c251f612d69d339d9c1"},{"author":{"_account_id":36180,"name":"Gireesh Awasthi","display_name":"Gireesh","email":"gawasthi2010@gmail.com","username":"agireesh","status":"NetApp"},"change_message_id":"3d16251c00e131df4323c80dfe7cbdab33dde417","unresolved":true,"context_lines":[{"line_number":1357,"context_line":"                     \"migration.\")"},{"line_number":1358,"context_line":"            return False, {}"},{"line_number":1359,"context_line":""},{"line_number":1360,"context_line":"        if self._is_flexgroup_pool(volume.host):"},{"line_number":1361,"context_line":"            LOG.info(\"Storage assisted migration is not supported for \""},{"line_number":1362,"context_line":"                     \"volumes on a FlexGroup pool. Falling back to host \""},{"line_number":1363,"context_line":"                     \"assisted migration.\")"}],"source_content_type":"text/x-python","patch_set":2,"id":"d73cd659_8a2914ca","line":1360,"range":{"start_line":1360,"start_character":16,"end_line":1360,"end_character":34},"updated":"2026-11-11 10:13:37.000000000","message":"We added 3 newly method, but which file you are calling these. I am not seeing any code which call these methods.","commit_id":"49215c58af9d3efda82c1c251f612d69d339d9c1"},{"author":{"_account_id":38059,"name":"Anoop Kumar Shukla","display_name":"Anoop Shukla","email":"anoop.shukla@netapp.com","username":"anoop2","status":"NetApp"},"change_message_id":"16f08a36123aecd163128a6adbf61485e13ac299","unresolved":false,"context_lines":[{"line_number":1357,"context_line":"                     \"migration.\")"},{"line_number":1358,"context_line":"            return False, {}"},{"line_number":1359,"context_line":""},{"line_number":1360,"context_line":"        if self._is_flexgroup_pool(volume.host):"},{"line_number":1361,"context_line":"            LOG.info(\"Storage assisted migration is not supported for \""},{"line_number":1362,"context_line":"                     \"volumes on a FlexGroup pool. Falling back to host \""},{"line_number":1363,"context_line":"                     \"assisted migration.\")"}],"source_content_type":"text/x-python","patch_set":2,"id":"0abde9ab_65de0234","line":1360,"range":{"start_line":1360,"start_character":16,"end_line":1360,"end_character":34},"in_reply_to":"d73cd659_8a2914ca","updated":"2026-11-11 10:32:04.000000000","message":"Its being called by migrate_volume from this method. There is only one new method added.","commit_id":"49215c58af9d3efda82c1c251f612d69d339d9c1"}]}