cloudstack-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From GitBox <...@apache.org>
Subject [GitHub] [cloudstack] cl-k-takahashi opened a new issue #3442: UserDispersingPlanner only counts Running vms on hosts.
Date Fri, 28 Jun 2019 10:54:13 GMT
cl-k-takahashi opened a new issue #3442: UserDispersingPlanner only counts Running vms on hosts.
URL: https://github.com/apache/cloudstack/issues/3442
 
 
   <!--
   Verify first that your issue/request is not already reported on GitHub.
   Also test if the latest release and master branch are affected too.
   Always add information AFTER of these HTML comments, but no need to delete the comments.
   -->
   
   ##### ISSUE TYPE
   <!-- Pick one below and delete the rest -->
    * Improvement Request
   
   ##### COMPONENT NAME
   <!--
   Categorize the issue, e.g. API, VR, VPN, UI, etc.
   -->
   ~~~
   UserDispersingPlanner.java
   ~~~
   
   ##### CLOUDSTACK VERSION
   <!--
   New line separated list of affected versions, commit ID for issues on master branch.
   -->
   ~~~
   Current version
   ~~~
   
   ##### CONFIGURATION
   <!--
   Information about the configuration if relevant, e.g. basic network, advanced networking,
etc.  N/A otherwise
   -->
   
   
   ##### OS / ENVIRONMENT
   <!--
   Information about the environment if relevant, N/A otherwise
   -->
   
   
   ##### SUMMARY
   <!-- Explain the problem/feature briefly -->
   When you try to deploy multiple vms in a short time, their state will go to 'Starting'
state.
   According to UserDispersingPlanner.java, it counts only Running vms on hosts to calculate
priority of destination hosts to deploy vm. As a result, these vms can concentrate to one
host if we try to deploy vms in a short time.
   To avoid this, I think we should count not only Running vms but also Starting vms.
   ```
           Long totalVmsOfAccount = vmInstanceDao.countRunningByAccount(accountId);
           if (s_logger.isDebugEnabled()) {
               s_logger.debug("Total VMs for account: " + totalVmsOfAccount);
           }
   ```

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

Mime
View raw message