[GitHub] [cloudstack] DaanHoogland commented on pull request #7887: xen/xcpng: revert java changes in PR #4672

["DaanHoogland (via GitHub)" <gi...@apache.org>]

DaanHoogland commented on PR #7887:
URL: https://github.com/apache/cloudstack/pull/7887#issuecomment-1695157440

   > @harikrishna-patnala @DaanHoogland I thought about it again, I think we should not set the vcpu max limit to the minimum cpu cores of host in the cluster. it might be possible that some hosts have enough capacity (cpu cores and cpu speed), but vms cannot be deployed.
   
   you are right @weizhouapache , we should actually set it to the maximum for any host in the cluster, if at all. I wonder if the setting is meant for operators to have some control over the kind of VMs that can be deployed...


-- 
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-unsubscribe@cloudstack.apache.org

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