----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/10842/ -----------------------------------------------------------
(Updated April 29, 2013, 10:01 p.m.) Review request for cloudstack, Chip Childers, Prachi Damle, and edison su. Changes ------- adding Prachi Description ------- This fixes a null pointer if selected pool on which to deploy is unable to be looked up via id. Used the same coding style found in the original code (large if/else block). If this is accepted, I'll fix master too. This patch doesn't apply cleanly, but the code is very similar. This addresses bug CLOUDSTACK-2281. Diffs ----- server/src/com/cloud/deploy/FirstFitPlanner.java 31fc43c Diff: https://reviews.apache.org/r/10842/diff/ Testing ------- Fixed broken system, tested against devcloud to ensure that it didn't break a stock/fresh install (advanced zone with 3 storage pools). Restarted VMs to ensure that the planner still functioned that way as well. Thanks, Marcus Sorensen