JoaoJandre commented on PR #8782: URL: https://github.com/apache/cloudstack/pull/8782#issuecomment-1997435998
> @JoaoJandre I suggest to check/fix the API calls from UI to list vms or vm metrics. > > It is not good to change the default behavior if it is not a bug, IMHO. @weizhouapache, @sureshanaparti and @rohityadavcloud, we could change only the UI behavior; however, why do we have this default behavior? We do not use these metrics that are returned on this API, when the metrics are used, users, operators and the UI use the `listVirtualMachineMetrics` API. That is, we already have an API that has this exact purpose. Not long ago we changed the default behavior of the `deleteTemplate` API on #7731; As with this situation, the one on #7731 wasn't a bug, it was an inconvenient behavior. A user/operator that only wants to list the VMs intuitively will use the `listVirtualMachines` API; however, depending on the environment's size, he will deal with slowness because of the default behavior. As with #7731, we could only a change the UI, but changing the default behavior makes more sense. -- 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
