nvazquez commented on pull request #5892:
URL: https://github.com/apache/cloudstack/pull/5892#issuecomment-1020420081


   @rohityadavcloud the 'cores-per-socket' value could be bypassed or modified 
but it is restricted by the offering, since a VM won't start if the number of 
CPUs is not a multiple of the cores per socket number. I think perhaps the 
'platform' setting could lead to unintended situations as it includes more 
information, for example:
   ````
   
device-model:qemu-upstream-compat;vga:std;videoram:8;apic:true;viridian:false;timeoffset:0;pae:true;cores-per-socket:2;acpi:1;hpet:true;secureboot:false;nx:true
   ````
   
   I've checked both settings are visible and editable for users, do you think 
'platform' should be hidden for non-admins?


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