harikrishna-patnala commented on issue #8789: URL: https://github.com/apache/cloudstack/issues/8789#issuecomment-2158103425
@rp- I see the following part has be already handled in the PR #8889 public boolean disconnectPhysicalDiskByPath(String localPath) { I'm trying to fix the case of running into the loop of all adapters by somehow passing the storage pool type to that caller cleanup() method but this is touching all over the code (which I fear it creates other regressions), instead I feel we can keep it the current way only since Fibrechannel adapter has already fixed. Whats your thoughts @rp- -- 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: commits-unsubscr...@cloudstack.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org