Skip to content

Commit d57e3ac

Browse files
Apply suggestions from code review
Co-authored-by: dahn <daan.hoogland@gmail.com>
1 parent 28791e2 commit d57e3ac

File tree

2 files changed

+2
-6
lines changed

2 files changed

+2
-6
lines changed

plugins/hypervisors/kvm/src/main/java/com/cloud/hypervisor/kvm/resource/wrapper/LibvirtPrepareForMigrationCommandWrapper.java

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -84,9 +84,7 @@ public Answer execute(final PrepareForMigrationCommand command, final LibvirtCom
8484
if (interfaceDef != null && interfaceDef.getNetType() == GuestNetType.VHOSTUSER) {
8585
DpdkTO to = new DpdkTO(interfaceDef.getDpdkOvsPath(), interfaceDef.getDpdkSourcePort(), interfaceDef.getInterfaceMode());
8686
dpdkInterfaceMapping.put(nic.getMac(), to);
87-
if (logger.isDebugEnabled()) {
88-
logger.debug("Configured DPDK interface for VM {}", vm.getName());
89-
}
87+
logger.debug("Configured DPDK interface for VM {}", vm.getName());
9088
}
9189
}
9290

server/src/main/java/com/cloud/vm/UserVmManagerImpl.java

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -7189,9 +7189,7 @@ public VirtualMachine migrateVirtualMachine(Long vmId, Host destinationHost) thr
71897189
throw new CloudRuntimeException("Unable to find suitable destination to migrate VM " + vm.getInstanceName());
71907190
}
71917191

7192-
if (logger.isInfoEnabled()) {
7193-
logger.info("Starting migration of VM {} from host {} to host {} ", vm.getInstanceName(), srcHostId, dest.getHost().getId());
7194-
}
7192+
logger.info("Starting migration of VM {} from host {} to host {} ", vm.getInstanceName(), srcHostId, dest.getHost().getId());
71957193
collectVmDiskAndNetworkStatistics(vmId, State.Running);
71967194
_itMgr.migrate(vm.getUuid(), srcHostId, dest);
71977195
return findMigratedVm(vm.getId(), vm.getType());

0 commit comments

Comments
 (0)