From c8c30e3617140da516684c4a6e5d0d7e3f011133 Mon Sep 17 00:00:00 2001
From: Pearl Dsilva <pearl1954@gmail.com>
Date: Wed, 13 May 2026 13:45:22 -0400
Subject: [PATCH] minor improvement

---
 .../main/java/com/cloud/storage/clvm/ClvmPoolManager.java    | 3 +--
 .../src/test/java/com/cloud/storage/ClvmPoolManagerTest.java | 5 +++--
 2 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/server/src/main/java/com/cloud/storage/clvm/ClvmPoolManager.java b/server/src/main/java/com/cloud/storage/clvm/ClvmPoolManager.java
index fba3deeecc2..768bae29b17 100644
--- a/server/src/main/java/com/cloud/storage/clvm/ClvmPoolManager.java
+++ b/server/src/main/java/com/cloud/storage/clvm/ClvmPoolManager.java
@@ -181,8 +181,7 @@ public class ClvmPoolManager implements Configurable {
         Long clusterId = pool.getClusterId();
         if (clusterId != null) {
             hosts = _hostDao.findByClusterId(clusterId, Host.Type.Routing);
-        }
-        if ((hosts == null || hosts.isEmpty()) && pool.getDataCenterId() > 0) {
+        } else if (pool.getDataCenterId() > 0) {
             hosts = _hostDao.findByDataCenterId(pool.getDataCenterId());
         }
         if (hosts == null || hosts.isEmpty()) {
diff --git a/server/src/test/java/com/cloud/storage/ClvmPoolManagerTest.java b/server/src/test/java/com/cloud/storage/ClvmPoolManagerTest.java
index 240d57315bd..e0fbdca8edd 100644
--- a/server/src/test/java/com/cloud/storage/ClvmPoolManagerTest.java
+++ b/server/src/test/java/com/cloud/storage/ClvmPoolManagerTest.java
@@ -46,6 +46,7 @@ import java.util.Collections;
 import static org.mockito.ArgumentMatchers.any;
 import static org.mockito.ArgumentMatchers.anyLong;
 import static org.mockito.ArgumentMatchers.eq;
+import static org.mockito.Mockito.lenient;
 import static org.mockito.Mockito.never;
 import static org.mockito.Mockito.times;
 import static org.mockito.Mockito.verify;
@@ -249,13 +250,13 @@ public class ClvmPoolManagerTest {
         when(pool.getPath()).thenReturn(VG_NAME);
         when(volsDetailsDao.findDetail(VOLUME_ID, ClvmPoolManager.CLVM_LOCK_HOST_ID)).thenReturn(null);
         when(hostDao.findByClusterId(10L, Host.Type.Routing)).thenReturn(Collections.emptyList());
-        when(hostDao.findByDataCenterId(1L)).thenReturn(Collections.emptyList());
+        lenient().when(hostDao.findByDataCenterId(1L)).thenReturn(Collections.emptyList());
 
         Long result = clvmPoolManager.queryCurrentLockHolder(VOLUME_ID, VOLUME_UUID, VOLUME_PATH, pool, false);
 
         Assert.assertNull(result);
         verify(hostDao, times(1)).findByClusterId(10L, Host.Type.Routing);
-        verify(hostDao, times(1)).findByDataCenterId(1L);
+        verify(hostDao, times(0)).findByDataCenterId(1L);
     }
 
     @Test