kiranchavala commented on issue #9820: URL: https://github.com/apache/cloudstack/issues/9820#issuecomment-2603979669
@harikrishna-patnala As discussed we need to fix the following issue 1. Update the documentation https://docs.cloudstack.apache.org/en/4.20.0.0/adminguide/storage.html#manually-adding-local-storage-pool Mention that admin user should not remove the local.storage.uuid 2. Fix the issue , add a local storage with a incorrect mount point ``` 2025-01-21 05:12:47,452 DEBUG [kvm.storage.LibvirtStorageAdaptor] (agentRequest-Handler-3:[]) (logid:1ee3d271) Attempting to create storage pool 4341d700-5211-4a3a-b843-0d829c5dc5f5 2025-01-21 05:12:47,452 ERROR [kvm.storage.LibvirtStorageAdaptor] (agentRequest-Handler-3:[]) (logid:1ee3d271) /lcoal does not exists. Check local.storage.path in agent.properties. 2025-01-21 05:12:47,453 WARN [cloud.agent.Agent] (agentRequest-Handler-3:[]) (logid:1ee3d271) Caught: com.cloud.utils.exception.CloudRuntimeException: Failed to create storage pool: 4341d700-5211-4a3a-b843-0d829c5dc5f5 at com.cloud.hypervisor.kvm.storage.LibvirtStorageAdaptor.createStoragePool(LibvirtStorageAdaptor.java:788) at com.cloud.hypervisor.kvm.storage.KVMStoragePoolManager.createStoragePool(KVMStoragePoolManager.java:392) at com.cloud.hypervisor.kvm.storage.KVMStoragePoolManager.createStoragePool(KVMStoragePoolManager.java:386) at com.cloud.hypervisor.kvm.resource.wrapper.LibvirtModifyStoragePoolCommandWrapper.execute(LibvirtModifyStoragePoolCommandWrapper.java:42) at com.cloud.hypervisor.kvm.resource.wrapper.LibvirtModifyStoragePoolCommandWrapper.execute(LibvirtModifyStoragePoolCommandWrapper.java:35) at com.cloud.hypervisor.kvm.resource.wrapper.LibvirtRequestWrapper.execute(LibvirtRequestWrapper.java:78) at com.cloud.hypervisor.kvm.resource.LibvirtComputingResource.executeRequest(LibvirtComputingResource.java:1945) at com.cloud.agent.Agent.processRequest(Agent.java:686) at com.cloud.agent.Agent$AgentRequestHandler.doTask(Agent.java:1109) at com.cloud.utils.nio.Task.call(Task.java:83) at com.cloud.utils.nio.Task.call(Task.java:29) at java.base/java.util.concurrent.FutureTask.run(FutureTask.java:264) at java.base/java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1136) at java.base/java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:635) at java.base/java.lang.Thread.run(Thread.java:840) ``` 3. When there are multiple local storage uuid, and admin user deletes one local storage from the UI. Cloudstack should remove the entry in local.storage.uuid and local.storage.path. The default uuid should not be touched ``` local.storage.uuid=a43943c1-1759-4073-9db1-bc0ea19203aa,f5b1220b-4446-42dc-a872-cffd281f9f8c local.storage.path=/var/lib/libvirt/images,/var/lib/libvirt/images2 ``` -- 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]
