)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":36171,"name":"jayaanand borra","display_name":"jayaanand borra","email":"jayaanand.borra@netapp.com","username":"jayaanan","status":"netapp"},"change_message_id":"e0678df1584f49a644bf82b8dd0416ace4bd93f1","unresolved":true,"context_lines":[{"line_number":4,"context_line":"Commit:     wanght26 \u003cwanght26@lenovo.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2026-06-11 15:33:47 +0800"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Fix retype failure between DM and Huawei storage"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"When retyping a volume between DM and Huawei storage backends,"},{"line_number":10,"context_line":"the operation fails due to missing handling in the driver."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"1d9f9fa3_e0d2f310","line":7,"updated":"2026-06-14 06:03:36.000000000","message":"Vague Commit Message. Can you please elaborate.","commit_id":"a40aac7691899f2e3816bfbb0ab731842a76aa1e"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":36537,"name":"wanghongtao","display_name":"wanghongtao","email":"wanght26@lenovo.com","username":"wanght26"},"change_message_id":"c36ee12202a46092d3f46661e0a01f5019222d51","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"c1a576f2_6e87911c","updated":"2026-06-11 08:21:23.000000000","message":"Third-party CI logs are not accessible from my environment.\nRequesting recheck.","commit_id":"a40aac7691899f2e3816bfbb0ab731842a76aa1e"},{"author":{"_account_id":36537,"name":"wanghongtao","display_name":"wanghongtao","email":"wanght26@lenovo.com","username":"wanght26"},"change_message_id":"c06765e2fc48df273700a8fadf814b781c369f20","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"a8af8817_dc661216","updated":"2026-06-11 08:25:47.000000000","message":"recheck","commit_id":"a40aac7691899f2e3816bfbb0ab731842a76aa1e"},{"author":{"_account_id":36537,"name":"wanghongtao","display_name":"wanghongtao","email":"wanght26@lenovo.com","username":"wanght26"},"change_message_id":"a610a3a0ea0d93ada78bd7fdfe6d24f70ed80ea6","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"10dbc29f_c6e3e404","updated":"2026-06-17 06:23:35.000000000","message":"recheck","commit_id":"975bf39d123339aefc7604b16f1eeafaca2ab7bb"}],"cinder/volume/drivers/netapp/dataontap/utils/data_motion.py":[{"author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"change_message_id":"f02bcc2ae51c536545487ca8144952e303096eb5","unresolved":true,"context_lines":[{"line_number":1742,"context_line":"            dest_backend_config \u003d config_utils.get_backend_configuration("},{"line_number":1743,"context_line":"                dest_backend_name)"},{"line_number":1744,"context_line":""},{"line_number":1745,"context_line":"            dest_volume_driver \u003d dest_backend_config.safe_get(\u0027volume_driver\u0027)"},{"line_number":1746,"context_line":"            if (not dest_volume_driver or"},{"line_number":1747,"context_line":"                    not dest_volume_driver.startswith("},{"line_number":1748,"context_line":"                        \u0027cinder.volume.drivers.netapp.\u0027)):"}],"source_content_type":"text/x-python","patch_set":2,"id":"40e79a1c_5df06d8d","line":1745,"updated":"2026-06-12 13:48:23.000000000","message":"I think this assumes that the destination is running on the same host? Pulling this info from config seems questionable at best -- this approach needs more explanation/documentation.","commit_id":"a40aac7691899f2e3816bfbb0ab731842a76aa1e"},{"author":{"_account_id":36537,"name":"wanghongtao","display_name":"wanghongtao","email":"wanght26@lenovo.com","username":"wanght26"},"change_message_id":"a610a3a0ea0d93ada78bd7fdfe6d24f70ed80ea6","unresolved":true,"context_lines":[{"line_number":1742,"context_line":"            dest_backend_config \u003d config_utils.get_backend_configuration("},{"line_number":1743,"context_line":"                dest_backend_name)"},{"line_number":1744,"context_line":""},{"line_number":1745,"context_line":"            dest_volume_driver \u003d dest_backend_config.safe_get(\u0027volume_driver\u0027)"},{"line_number":1746,"context_line":"            if (not dest_volume_driver or"},{"line_number":1747,"context_line":"                    not dest_volume_driver.startswith("},{"line_number":1748,"context_line":"                        \u0027cinder.volume.drivers.netapp.\u0027)):"}],"source_content_type":"text/x-python","patch_set":2,"id":"8eb434c9_b1eb4cf8","line":1745,"in_reply_to":"40e79a1c_5df06d8d","updated":"2026-06-17 06:23:35.000000000","message":"Good point, thanks for calling this out.\n\nThe previous implementation relied on local backend configuration to\ndetermine the volume driver, which may not be reliable in a multi-node\ndeployment where the destination backend is not present on the local host.\n\nThis has been updated to infer whether the destination backend is\nNetApp-based using the backend name, avoiding reliance on local config\nfor decision-making.\n\nBackend configuration is still used for client initialization, which\nmatches the current driver initialization model, but it is no longer\nused to make behavioral decisions.","commit_id":"a40aac7691899f2e3816bfbb0ab731842a76aa1e"},{"author":{"_account_id":36171,"name":"jayaanand borra","display_name":"jayaanand borra","email":"jayaanand.borra@netapp.com","username":"jayaanan","status":"netapp"},"change_message_id":"58dba07835dc99cd5ba7e9d00e1cf4679bd091b7","unresolved":true,"context_lines":[{"line_number":1742,"context_line":"            dest_backend_config \u003d config_utils.get_backend_configuration("},{"line_number":1743,"context_line":"                dest_backend_name)"},{"line_number":1744,"context_line":""},{"line_number":1745,"context_line":"            dest_volume_driver \u003d dest_backend_config.safe_get(\u0027volume_driver\u0027)"},{"line_number":1746,"context_line":"            if (not dest_volume_driver or"},{"line_number":1747,"context_line":"                    not dest_volume_driver.startswith("},{"line_number":1748,"context_line":"                        \u0027cinder.volume.drivers.netapp.\u0027)):"}],"source_content_type":"text/x-python","patch_set":2,"id":"a9473384_973a0cf8","line":1745,"in_reply_to":"40e79a1c_5df06d8d","updated":"2026-06-14 05:58:58.000000000","message":"I agree with Eric, Data_motion is NetApp specific data movements. Fallback needs higher level fallback not hear.Capabilities contain vendor name...mismatch can be used instead of host for fallback.","commit_id":"a40aac7691899f2e3816bfbb0ab731842a76aa1e"}]}