sureshanaparti commented on PR #8655: URL: https://github.com/apache/cloudstack/pull/8655#issuecomment-1945973470
> general question @sureshanaparti : the word Access used in this PR seems to me to mean Authorisation (or Authorization depending on your continent). Does this make sense? I think we should rename it to avoid confusion. Access is always needed and it seems that we are checking if we need access to a store instead of checking if we need authentication/authorisation to get access. ??? @DaanHoogland It's about making the volume/disk available in the host, and can be accessed at certain path. In managed storages eg. powerflex, disk is not available directly unless it is mapped to a particular host/client through their proprietary API/cmd). Will keep it the same for now, as there are other methods with similar names (we can do a name refactor later, in a separate PR), Is that Ok? -- 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]
