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

Reply via email to