diff --git a/plugins/hypervisors/kvm/src/main/java/com/cloud/hypervisor/kvm/resource/wrapper/LibvirtConvertInstanceCommandWrapper.java b/plugins/hypervisors/kvm/src/main/java/com/cloud/hypervisor/kvm/resource/wrapper/LibvirtConvertInstanceCommandWrapper.java index 504edb9d8882..9442907f3564 100644 --- a/plugins/hypervisors/kvm/src/main/java/com/cloud/hypervisor/kvm/resource/wrapper/LibvirtConvertInstanceCommandWrapper.java +++ b/plugins/hypervisors/kvm/src/main/java/com/cloud/hypervisor/kvm/resource/wrapper/LibvirtConvertInstanceCommandWrapper.java @@ -57,7 +57,6 @@ public Answer execute(ConvertInstanceCommand cmd, LibvirtComputingResource serve RemoteInstanceTO sourceInstance = cmd.getSourceInstance(); Hypervisor.HypervisorType sourceHypervisorType = sourceInstance.getHypervisorType(); String sourceInstanceName = sourceInstance.getInstanceName(); - String sourceInstancePath = sourceInstance.getInstancePath(); Hypervisor.HypervisorType destinationHypervisorType = cmd.getDestinationHypervisorType(); DataStoreTO conversionTemporaryLocation = cmd.getConversionTemporaryLocation(); long timeout = (long) cmd.getWait() * 1000; diff --git a/utils/src/main/java/com/cloud/utils/script/Script.java b/utils/src/main/java/com/cloud/utils/script/Script.java index 6cd622582be5..79c9511b0238 100644 --- a/utils/src/main/java/com/cloud/utils/script/Script.java +++ b/utils/src/main/java/com/cloud/utils/script/Script.java @@ -48,6 +48,7 @@ import com.cloud.utils.Pair; import com.cloud.utils.PropertiesUtil; +import com.cloud.utils.StringUtils; import com.cloud.utils.concurrency.NamedThreadFactory; import com.cloud.utils.script.OutputInterpreter.TimedOutLogger; @@ -155,25 +156,15 @@ protected String buildCommandLine(String[] command) { boolean obscureParam = false; for (int i = 0; i < command.length; i++) { String cmd = command[i]; - if (obscureParam) { - builder.append("******").append(" "); - obscureParam = false; - } else { - builder.append(command[i]).append(" "); - } - - if ("-y".equals(cmd) || "-z".equals(cmd)) { - obscureParam = true; - _passwordCommand = true; + if (StringUtils.isNotEmpty(cmd) && cmd.startsWith("vi://")) { + String[] tokens = cmd.split("@"); + if (tokens.length >= 2) { + builder.append("vi://").append("******@").append(tokens[1]).append(" "); + } else { + builder.append("vi://").append("******").append(" "); + } + continue; } - } - return builder.toString(); - } - - protected String buildCommandLine(List command) { - StringBuilder builder = new StringBuilder(); - boolean obscureParam = false; - for (String cmd : command) { if (obscureParam) { builder.append("******").append(" "); obscureParam = false;