sureshanaparti commented on PR #6429: URL: https://github.com/apache/cloudstack/pull/6429#issuecomment-1145601027
> Hi @utchoang, I think adding a `custom` protocol is needed for SolidFire and PowerFlex, and because the protocol parameter is set as mandatory @utchoang Correct, protocol is the required input at UI, for adding primary storage. This is imp to pick the correct storage provider, url and related parameter inputs. If `custom` protocol for PowerFlex is confusing, this can be updated to something meaningful, eg. PreSetup or PowerFlex (now or can be re-visited later ?), but ensure that all the provider details are properly picked and passed to the API call. Also, it's better to keep adding primary storage options in sync, at zone wizard and add primary storage dialog. Check if any documentation update needed for these changes. Thanks. -- 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]
