----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/9917/ -----------------------------------------------------------
Review request for cloudstack, David Nalley and Chip Childers. Description ------- CLOUDSTACK-1604: deploy VM failed when global setting "vm.allocation.algorithm" is set to "userdispersing" Changes: - DeployPlannerSelector was newly introduced for BareMetal feature. It had the planner name hardcoded. - Change it to decide the planner by referring to the global config vm.allocation.algorithm value Builds ok. RAT check ok. Functionality tested ok. This addresses bug CLOUDSTACK-1604. Diffs ----- server/src/com/cloud/deploy/AbstractDeployPlannerSelector.java 62094eb server/src/com/cloud/deploy/HypervisorVmPlannerSelector.java 034a9aa Diff: https://reviews.apache.org/r/9917/diff/ Testing ------- Successfully deployed VM using the UserDispersingPlanner or UserConcentratedPodPlanner after changing the global setting. Thanks, Prachi Damle
