rohityadavcloud commented on PR #8782:
URL: https://github.com/apache/cloudstack/pull/8782#issuecomment-1997341885

   > For both the listVirtualMachines and listVirtualMachineMetrics APIs, by 
default, ACS queries all VM details (the details parameter is default all) and 
consequently queries the cloud.vm_stats table for each VM,
   
   This is because the listVirtualMachinesMetrics API (and all Metrics API) are 
superset of the non-metrics APIs. All the Metrics APIs add something related to 
metrics in their API response in addition to what a non-metrics API would 
return. I think we shouldn't change the default behaviour of the non-metrics 
API - and if you really have a use-case for this see if there's any other way 
to do what you're trying to accomplish or worst-case add a global setting that 
allows the behaviour you want but keep the setting's default in such a way that 
it continue old behaviour for other users.


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