nvazquez commented on code in PR #9102:
URL: https://github.com/apache/cloudstack/pull/9102#discussion_r1907027393


##########
plugins/integrations/kubernetes-service/src/main/java/com/cloud/kubernetes/cluster/actionworkers/KubernetesClusterResourceModifierActionWorker.java:
##########
@@ -293,13 +259,13 @@ protected DeployDestination plan(final long nodesCount, 
final DataCenter zone, f
         throw new InsufficientServerCapacityException(msg, DataCenter.class, 
zone.getId());
     }
 
-    protected DeployDestination plan() throws 
InsufficientServerCapacityException {
+    protected DeployDestination plan(Long domainId, Long accountId, 
Hypervisor.HypervisorType hypervisorType) throws 
InsufficientServerCapacityException {
         ServiceOffering offering = 
serviceOfferingDao.findById(kubernetesCluster.getServiceOfferingId());
         DataCenter zone = 
dataCenterDao.findById(kubernetesCluster.getZoneId());
         if (logger.isDebugEnabled()) {
             logger.debug(String.format("Checking deployment destination for 
Kubernetes cluster : %s in zone : %s", kubernetesCluster.getName(), 
zone.getName()));
         }
-        return plan(kubernetesCluster.getTotalNodeCount(), zone, offering);
+        return plan(kubernetesCluster.getTotalNodeCount(), zone, offering, 
domainId, accountId, hypervisorType);

Review Comment:
   Nice catch, thanks



-- 
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: commits-unsubscr...@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to