(cloudstack) branch 4.19 updated: Ignore non-managed pools for storage pool access preparation (#9376)

[sureshanaparti]

This is an automated email from the ASF dual-hosted git repository.

sureshanaparti pushed a commit to branch 4.19
in repository https://gitbox.apache.org/repos/asf/cloudstack.git


The following commit(s) were added to refs/heads/4.19 by this push:
     new d9f77b841c5 Ignore non-managed pools for storage pool access 
preparation (#9376)
d9f77b841c5 is described below

commit d9f77b841c5c19d92bea82caf65e9adbf18fbe00
Author: Suresh Kumar Anaparti <sureshkumar.anapa...@gmail.com>
AuthorDate: Sun Jul 14 14:50:26 2024 +0530

    Ignore non-managed pools for storage pool access preparation (#9376)
---
 server/src/main/java/com/cloud/storage/StorageManagerImpl.java | 6 +-----
 1 file changed, 1 insertion(+), 5 deletions(-)

diff --git a/server/src/main/java/com/cloud/storage/StorageManagerImpl.java 
b/server/src/main/java/com/cloud/storage/StorageManagerImpl.java
index 74b19b61fd4..73b9ac8960d 100644
--- a/server/src/main/java/com/cloud/storage/StorageManagerImpl.java
+++ b/server/src/main/java/com/cloud/storage/StorageManagerImpl.java
@@ -2788,14 +2788,10 @@ public class StorageManagerImpl extends ManagerBase 
implements StorageManager, C
 
     @Override
     public boolean canHostPrepareStoragePoolAccess(Host host, StoragePool 
pool) {
-        if (host == null || pool == null) {
+        if (host == null || pool == null || !pool.isManaged()) {
             return false;
         }
 
-        if (!pool.isManaged()) {
-            return true;
-        }
-
         DataStoreProvider storeProvider = 
_dataStoreProviderMgr.getDataStoreProvider(pool.getStorageProviderName());
         DataStoreDriver storeDriver = storeProvider.getDataStoreDriver();
         return storeDriver instanceof PrimaryDataStoreDriver && 
((PrimaryDataStoreDriver)storeDriver).canHostPrepareStoragePoolAccess(host, 
pool);