andrijapanicsb commented on code in PR #363: URL: https://github.com/apache/cloudstack-documentation/pull/363#discussion_r1418944245
########## source/installguide/configuration.rst: ########## @@ -512,7 +512,7 @@ Basic Zone Configuration - **Name.** The name of the storage device. - **Protocol.** For XenServer, choose either NFS, iSCSI, or - PreSetup. For KVM, choose NFS, SharedMountPoint,CLVM, RBD or custom (for PowerFlex). For + PreSetup. For KVM, choose NFS, SharedMountPoint,CLVM, RBD, FiberChannel or custom (for PowerFlex). For Review Comment: The naming is not ideal, as fibrechannel can also (has been historically) configured outside of ACS, manuially, and just a Shared Mount Point would be feed to ACS. But leave it as it is. -- 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: dev-unsubscr...@cloudstack.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org