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

Reply via email to