[GitHub] [cloudstack] joseflauzino commented on pull request #6401: VM stats history visualization

[GitBox <gi...@apache.org>]

joseflauzino commented on PR #6401:
URL: https://github.com/apache/cloudstack/pull/6401#issuecomment-1205247188

   Thanks for the tests and review, @shwstppr .
   
   I'll test your suggestions soon.
   
   About the possible improvements you commented on:
   
   > default time period could be used the current time minus stats retention time. This is just my opinion. It will give better UX as user will see some stats when view is opened first.
   
   Depending on the data retention settings and the amount of VM instances in the cloud, using the current time minus the retention time can bring in a massive amount of data. So the delay in loading the VM stats could be very large when accessing this view. The idea of filtering by the last hour of available data when accessing this view is safer for this initial version. Maybe in the future, we can think of something more elaborate based on feedback from ACS users.
   
   > Something for future maybe, functionality could be extended to other system vms and VRs as well. Or maybe other resources too.
   
   I think so too! In the future it would be nice to have this for system VMs and other cloud resources.


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