----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/9917/#review17901 -----------------------------------------------------------
David, I committed this to master. Commit hash: a7231015e8f27a4e43b9cc7571ceca5514285b94 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 Contained in branches: master Contained in no tag - Prachi Damle On March 14, 2013, 1:17 a.m., Prachi Damle wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/9917/ > ----------------------------------------------------------- > > (Updated March 14, 2013, 1:17 a.m.) > > > 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 > >
