Skip to content

Commit 821992e

Browse files
Apply suggestions from code review
Co-authored-by: Fabricio Duarte <fabricio.duarte.jr@gmail.com>
1 parent 6a72d99 commit 821992e

2 files changed

Lines changed: 3 additions & 3 deletions

File tree

server/src/main/java/com/cloud/agent/manager/allocator/impl/FirstFitAllocator.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -234,12 +234,12 @@ protected List<Host> checkHostsCompatibilities(ServiceOffering offering, Exclude
234234
}
235235

236236
if (avoid.shouldAvoid(host)) {
237-
logger.debug("Host [{}] is in avoid set, skipping this and trying other available hosts", () -> host);
237+
logger.debug("Host [{}] is in avoid set, skipping this and trying other available hosts", host);
238238
continue;
239239
}
240240

241241
if (capacityManager.checkIfHostReachMaxGuestLimit(host)) {
242-
logger.debug("Adding host [{}] to the avoid set because this host already has the max number of running (user and/or system) VMs.", () -> host);
242+
logger.debug("Adding host [{}] to the avoid set because this host already has the max number of running (user and/or system) VMs.", host);
243243
avoid.addHost(host.getId());
244244
continue;
245245
}

server/src/main/java/com/cloud/server/ManagementServerImpl.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1652,7 +1652,7 @@ public Ternary<Pair<List<? extends Host>, Integer>, List<? extends Host>, Map<Ho
16521652
if (CollectionUtils.isEmpty(suitableHosts)) {
16531653
logger.warn("No suitable hosts found.");
16541654
} else {
1655-
logger.debug("Hosts having capacity and are suitable for migration: {} ", suitableHosts);
1655+
logger.debug("Hosts having capacity and are suitable for migration: {}", suitableHosts);
16561656
}
16571657

16581658
// Only list hosts of the same architecture as the source Host in a multi-arch zone

0 commit comments

Comments
 (0)