GabrielBrascher commented on a change in pull request #4575:
URL: https://github.com/apache/cloudstack/pull/4575#discussion_r627773900
##########
File path:
engine/orchestration/src/main/java/com/cloud/vm/VirtualMachineManagerImpl.java
##########
@@ -4219,14 +4219,18 @@ private void orchestrateMigrateForScale(final String
vmUuid, final long srcHostI
vm.getServiceOfferingId();
final long dstHostId = dest.getHost().getId();
final Host fromHost = _hostDao.findById(srcHostId);
+ Host srcHost = _hostDao.findById(srcHostId);
if (fromHost == null) {
- s_logger.info("Unable to find the host to migrate from: " +
srcHostId);
- throw new CloudRuntimeException("Unable to find the host to
migrate from: " + srcHostId);
+ String logMessageUnableToFindHost = String.format("Unable to find
host to migrate from %s.", srcHost);
+ s_logger.info(logMessageUnableToFindHost);
+ throw new CloudRuntimeException(logMessageUnableToFindHost);
}
+ Host dstHost = _hostDao.findById(dstHostId);
if (fromHost.getClusterId().longValue() != dest.getCluster().getId()) {
Review comment:
Done!
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
