----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/9917/#review17905 -----------------------------------------------------------
Ship it! Ship It! - Chip Childers 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 > >
