[GitHub] [cloudstack] GutoVeronezi commented on pull request #6803: ui,server,api: resource metrics improvements

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

GutoVeronezi commented on PR #6803:
URL: https://github.com/apache/cloudstack/pull/6803#issuecomment-1335575437

   > > This PR is addressing several changes in different contexts (VM stats, disk stats, system VM stats, and UI). I think it would be better both for managing, reviewing, and testing if they were separated in different PRs, also improving the description of the changes and how they would affect users.
   > 
   > Any concerns in how it affects users @GutoVeronezi ? cc @shwstppr
   
   @DaanHoogland, in both approaches users will have the feature; therefore, it does not affect them directly. As the changes are not dependent on each other, the suggestion to segregating it in several PRs focus on facilitating the reviewing, testing, and management of the features. In my opinion, it would be more interesting to have single scope PRs.


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