weizhouapache commented on code in PR #13991:
URL: https://github.com/apache/cloudstack/pull/13991#discussion_r3980496668


##########
plugins/hypervisors/kvm/src/main/java/com/cloud/hypervisor/kvm/storage/KVMPhysicalDisk.java:
##########
@@ -54,9 +54,9 @@ public static String RBDStringBuilder(KVMStoragePool 
storagePool, String image)
         rbdOpts += ":mon_host=" + composeOptionForMonHosts(monHost, monPort);
 
         if (authUserName == null) {
-            rbdOpts += ":auth_supported=none";
+            rbdOpts += ":auth_client_required=none";
         } else {
-            rbdOpts += ":auth_supported=cephx";
+            rbdOpts += ":auth_client_required=cephx";

Review Comment:
   > @waterWang Can you check and update the doc if required (that >= Ceph 
Tentacle 20.2.4 will be supported).
   
   thanks @sureshanaparti for review
   
   the configuration was introduced many years ago. 
   I could find in  Ceph 13 's doc : 
https://docs.ceph.com/en/mimic/rados/configuration/auth-config-ref/
   it should work with the Ceph releases after it.
   
   For earlier versions, I could not find the doc on https://docs.ceph.com/
   



-- 
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]

Reply via email to