waterWang opened a new pull request, #13991:
URL: https://github.com/apache/cloudstack/pull/13991

   Fixes #13989
   
   ### Problem
   CloudStack builds its RBD connection strings with the legacy Ceph option 
`auth_supported=cephx` (or `auth_supported=none` when no auth user is 
configured). This option was removed in Ceph Tentacle 20.2.4. As a result, 
every RBD operation performed by the KVM agent through librados fails:
   
   ```
   agent.err: libvirt: Storage Driver error : internal error: failed to set 
RADOS option: auth_supported
   agent.log: Failed to create RBD storage pool: org.libvirt.LibvirtException
   ```
   
   ### Root cause
   `KVMPhysicalDisk.RBDStringBuilder` emits `auth_supported` in the RBD URI 
string passed to qemu/librados. Modern librados no longer accepts this option 
name.
   
   ### Fix
   Replace `auth_supported` with `auth_client_required` (values `cephx`/`none`) 
— the modern option used by the qemu rbd driver (`block/rbd.c` calls 
`rados_conf_set(cluster, "auth_client_required", ...)`) and accepted by current 
Ceph releases. The qemu `RbdAuthMode` enum only supports `cephx` and `none`, 
matching the existing branches.
   
   ### Changes
   - 
`plugins/hypervisors/kvm/src/main/java/com/cloud/hypervisor/kvm/storage/KVMPhysicalDisk.java`:
 emit `auth_client_required=cephx` / `auth_client_required=none`
   - 
`plugins/hypervisors/kvm/src/test/java/com/cloud/hypervisor/kvm/storage/KVMPhysicalDiskTest.java`:
 update 3 assertions to expect `auth_client_required=cephx`


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