DaanHoogland commented on code in PR #8680:
URL: https://github.com/apache/cloudstack/pull/8680#discussion_r1495815416
##########
plugins/hypervisors/kvm/src/main/java/com/cloud/hypervisor/kvm/resource/wrapper/LibvirtMigrateCommandWrapper.java:
##########
@@ -308,7 +308,8 @@ Use VIR_DOMAIN_XML_SECURE (value = 1) prior to v1.0.0.
logger.error(String.format("Can't migrate domain [%s] due to:
[%s].", vmName, e.getMessage()), e);
result = e.getMessage();
if (result.startsWith("unable to connect to server") &&
result.endsWith("refused")) {
- result = String.format("Migration was refused connection to
destination: %s. Please check libvirt configuration compatibility and firewall
rules on the source and destination hosts.", destinationUri);
+ logger.debug(String.format("Migration failed as connection to
destination [%s] was refused. Please check libvirt configuration compatibility
and firewall rules on the source and destination hosts.", destinationUri));
Review Comment:
```suggestion
logger.debug("Migration failed as connection to destination
[{}] was refused. Please check libvirt configuration compatibility and firewall
rules on the source and destination hosts.", destinationUri);
```
--
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]
