mlsorensen commented on a change in pull request #5789:
URL: https://github.com/apache/cloudstack/pull/5789#discussion_r774065861
##########
File path:
plugins/storage/volume/scaleio/src/main/java/org/apache/cloudstack/storage/datastore/driver/ScaleIOPrimaryDataStoreDriver.java
##########
@@ -629,7 +629,7 @@ public void copyAsync(DataObject srcData, DataObject
destData, Host destHost, As
private Answer copyTemplateToVolume(DataObject srcData, DataObject
destData, Host destHost) {
// Copy PowerFlex/ScaleIO template to volume
- LOGGER.debug("Initiating copy from PowerFlex template volume on host "
+ destHost != null ? destHost.getId() : "");
+ LOGGER.debug(String.format("Initiating copy from PowerFlex template
volume on host %s", destHost != null ? destHost.getId() : ""));
Review comment:
Implementing this. Going to use "not specified" @DaanHoogland since that
sounds less like an error. There is code later to resolve a destination host if
this is not specified.
--
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]
