From 6f76d8fc922c220b9d351ba9e974c66b30123fd3 Mon Sep 17 00:00:00 2001 From: Rohit Yadav Date: Wed, 25 Sep 2019 10:17:39 +0530 Subject: [PATCH] allocator: in case of null guest OS don't fail prioritisation completely In case of null guest OS found for a template, don't fail prioritisation completely (could still work based on HVM etc). Signed-off-by: Rohit Yadav --- .../agent/manager/allocator/impl/FirstFitAllocator.java | 8 +------- 1 file changed, 1 insertion(+), 7 deletions(-) diff --git a/server/src/main/java/com/cloud/agent/manager/allocator/impl/FirstFitAllocator.java b/server/src/main/java/com/cloud/agent/manager/allocator/impl/FirstFitAllocator.java index f4ad478861e1..186c654fda38 100644 --- a/server/src/main/java/com/cloud/agent/manager/allocator/impl/FirstFitAllocator.java +++ b/server/src/main/java/com/cloud/agent/manager/allocator/impl/FirstFitAllocator.java @@ -29,8 +29,6 @@ import org.apache.log4j.Logger; import org.springframework.stereotype.Component; -import com.google.common.base.Strings; - import com.cloud.agent.manager.allocator.HostAllocator; import com.cloud.capacity.CapacityManager; import com.cloud.capacity.CapacityVO; @@ -419,10 +417,6 @@ protected List prioritizeHosts(VMTemplateVO template, ServiceOff // Determine the guest OS category of the template String templateGuestOSCategory = getTemplateGuestOSCategory(template); - if (Strings.isNullOrEmpty(templateGuestOSCategory)) { - return hosts; - } - List prioritizedHosts = new ArrayList(); List noHvmHosts = new ArrayList(); @@ -453,7 +447,7 @@ protected List prioritizeHosts(VMTemplateVO template, ServiceOff String hostGuestOSCategory = getHostGuestOSCategory(host); if (hostGuestOSCategory == null) { continue; - } else if (templateGuestOSCategory.equals(hostGuestOSCategory)) { + } else if (templateGuestOSCategory != null && templateGuestOSCategory.equals(hostGuestOSCategory)) { highPriorityHosts.add(host); } else { lowPriorityHosts.add(host);