DaanHoogland commented on code in PR #13263:
URL: https://github.com/apache/cloudstack/pull/13263#discussion_r3316901966
##########
server/src/main/java/org/apache/cloudstack/vm/ImportVmTasksManagerImpl.java:
##########
@@ -99,14 +100,16 @@ private ImportVmTask.TaskState getStateFromFilter(String
tasksFilter) {
try {
return ImportVmTask.TaskState.getValue(tasksFilter);
} catch (IllegalArgumentException e) {
- throw new ServerApiException(ApiErrorCode.PARAM_ERROR,
String.format("Invalid value for task state: %s", tasksFilter));
+ throw new ServerApiException(ApiErrorCode.PARAM_ERROR,
+ String.format("Invalid value for task state: %s",
tasksFilter));
}
}
@Override
- public ImportVmTask createImportVMTaskRecord(DataCenter zone, Account
owner, long userId, String displayName, String vcenter, String datacenterName,
String sourceVMName, Host convertHost, Host importHost) {
+ public ImportVmTask createImportVMTaskRecord(DataCenter zone, Account
owner, long userId, String displayName,
+ String vcenter, String datacenterName, String sourceVMName, Host
convertHost, Host importHost) {
logger.debug("Creating import VM task entry for VM: {} for account {}
on zone {} " +
- "from the vCenter: {} / datacenter: {} / source VM:
{}",
+ "from the vCenter: {} / datacenter: {} / source VM: {}",
sourceVMName, owner.getAccountName(), zone.getName(),
displayName, vcenter, datacenterName);
Review Comment:
```suggestion
sourceVMName, owner.getAccountName(), zone.getName(),
vcenter, datacenterName, displayName);
```
--
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.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
