DaanHoogland commented on a change in pull request #5762:
URL: https://github.com/apache/cloudstack/pull/5762#discussion_r765863355



##########
File path: server/src/main/java/com/cloud/deploy/FirstFitPlanner.java
##########
@@ -367,8 +372,11 @@ protected void removeClustersCrossingThreshold(List<Long> 
clusterListForVmAlloca
                 // Remove clusters crossing disabled threshold
                 
clusterListForVmAllocation.removeAll(clustersCrossingThreshold);
 
-                s_logger.debug("Cannot allocate cluster list " + 
clustersCrossingThreshold.toString() + " for vm creation since their allocated 
percentage" +
-                        " crosses the disable capacity threshold defined at 
each cluster/ at global value for capacity Type : " + capacity + ", skipping 
these clusters");
+                String warnMessageForClusterReachedCapacityThreshold = 
String.format(
+                        "Cannot allocate cluster list %s for vm creation since 
their allocated percentage crosses the disable capacity threshold defined at 
each cluster at"
+                                + " global value [settings: %s, value: %s] for 
capacity Type : %s, skipping these clusters", 
clustersCrossingThreshold.toString(),

Review comment:
       ```suggestion
                           "Cannot allocate cluster list %s for vm creation 
since their allocated percentage crosses the disable capacity threshold defined 
at each cluster at"
                           + " global value [settings: %s, value: %s] for 
capacity Type : %s, skipping these clusters", 
clustersCrossingThreshold.toString(),
   ```




-- 
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]


Reply via email to