rafaelweingartner commented on a change in pull request #2315: A comprehensive
solution for #CLOUDSTACK-9025.
URL: https://github.com/apache/cloudstack/pull/2315#discussion_r150802121
##########
File path: api/src/com/cloud/hypervisor/HypervisorGuru.java
##########
@@ -45,7 +45,7 @@
VirtualMachineTO implement(VirtualMachineProfile vm);
/**
- * Give hypervisor guru opportunity to decide if certain command needs to
be delegated to other host, mainly to secondary storage VM host
+ * Give hypervisor guru opportunity to decide if certain command needs to
be delegated to other host, mainly from the secondary storage VM host to a real
hypervisor host
Review comment:
But still, this method gives the opportunity to change the default host that
is intended to execute a command right?
what about this message to help people understand the purpose of this method:
> Give hypervisor guru opportunity to decide if certain commands need to be
delegated to another host, for instance, we may have the opportunity to change
from a system VM (is considered a host) to a real host to execute commands.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
With regards,
Apache Git Services
