slavkap commented on issue #8352: URL: https://github.com/apache/cloudstack/issues/8352#issuecomment-1853715559
@weizhouapache, there is no issue with this code snippet. The problem is that it is invoked now for each storage plugin, and not all use a valid URL. Before this PR [this part of code](https://github.com/apache/cloudstack/pull/6699/files#diff-cf68e132ad7711cfa9250cda5819c09f51cba9933aa595cd82ad5a02b4edbe64R897) had been used only by the Default primary storage . According to this article [SolidFire CloudStack](https://www.shapeblue.com/solidfire-with-cloudstack/#Adding_SolidFire_1_1-based_Primary_Storage-3) the URL that should be provided is not valid also and I think the SolidFire's users will have the same problem -- 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]
