rajiv-jain-netapp commented on code in PR #13053:
URL: https://github.com/apache/cloudstack/pull/13053#discussion_r3193172010
##########
plugins/hypervisors/kvm/src/main/java/com/cloud/hypervisor/kvm/storage/IscsiAdmStorageAdaptor.java:
##########
@@ -158,8 +183,23 @@ public boolean connectPhysicalDisk(String volumeUuid,
KVMStoragePool pool, Map<S
return true;
}
Review Comment:
@winterhazel I introduced this nested logic because NetApp storage exposes
IQNs at the SVM level rather than at the volume level. As a result, all volumes
within the same SVM share the same IQN when creating physical disks. This check
ensures that if a connection to the SVM‑level IQN already exists, the failure
can be safely ignored; otherwise, an exception should be raised.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
