GutoVeronezi commented on a change in pull request #4575:
URL: https://github.com/apache/cloudstack/pull/4575#discussion_r628259402
##########
File path:
engine/orchestration/src/main/java/com/cloud/vm/VirtualMachineManagerImpl.java
##########
@@ -4354,8 +4360,8 @@ private void orchestrateMigrateForScale(final String
vmUuid, final long srcHostI
s_logger.info("Migration was unsuccessful. Cleaning up: " +
vm);
_alertMgr.sendAlert(alertType, fromHost.getDataCenterId(),
fromHost.getPodId(),
- "Unable to migrate vm " + vm.getInstanceName() + "
from host " + fromHost.getName() + " in zone " + dest.getDataCenter().getName()
+ " and pod " +
- dest.getPod().getName(), "Migrate Command
failed. Please check logs.");
+ "Unable to migrate vm " + vm.getInstanceName() + "
from " + fromHost + " in zone " + dest.getDataCenter().getName() + " and pod " +
Review comment:
We can use `String.format` here too.
--
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]
