The recovery mode is using when Share Migration needs to convert rules to read-only. Thus, all rules are removed and re-added in read-only mode. For that, drivers have a "recovery mode" code path in update_access.
Although the recovery mode in the LVM driver is fairly inefficient (it removes each rule instead of just setting the list of rules according to the list of rules that the share should have), it attempts to perform the removal step without parsing IPv6 addresses, causing errors in this code path and preventing migration to run when IPv6 addresses are used. This can be seen in: https://github.com/openstack/manila/blob/81e708281f6bdeea2dba12eb08cf9c33265e7050/manila/share/drivers/helpers.py#L245
The correct behavior should be that the function "_get_parsed_address_or_cidr" is invoked to parse the address, as it is done for the other code paths. This function can be found in the same file: https://github.com/openstack/manila/blob/81e708281f6bdeea2dba12eb08cf9c33265e7050/manila/share/drivers/helpers.py#L308
Fix proposed to branch: master
Review: https:/