----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12180/#review22562 -----------------------------------------------------------
engine/storage/volume/src/org/apache/cloudstack/storage/datastore/manager/PrimaryDataStoreProviderManagerImpl.java <https://reviews.apache.org/r/12180/#comment46279> LGTM. Can you also add datastore id in the log message. It's useful to know the datastore that is not found. - Sateesh Chodapuneedi On June 29, 2013, 12:48 a.m., Venkata Siva Vijayendra Bhamidipati wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/12180/ > ----------------------------------------------------------- > > (Updated June 29, 2013, 12:48 a.m.) > > > Review request for cloudstack, Chip Childers, Devdeep Singh, Kelven Yang, and > Sateesh Chodapuneedi. > > > Bugs: CLOUDSTACK-3110 > > > Repository: cloudstack-git > > > Description > ------- > > Fixing an NPE when an attempt is made to add a primary datastore by providing > an unreachable/nonexistent datastore/host. > > > Diffs > ----- > > > engine/storage/src/org/apache/cloudstack/storage/datastore/DataStoreManagerImpl.java > b92f92f > > engine/storage/volume/src/org/apache/cloudstack/storage/datastore/manager/PrimaryDataStoreProviderManagerImpl.java > 06b54e0 > > Diff: https://reviews.apache.org/r/12180/diff/ > > > Testing > ------- > > With fix in place, an attempt to add an invalid primary datastore will fail > with a GUI error dialog instead of failing silently with an NPE in the mgmt > server log. > > > File Attachments > ---------------- > > Refactored patch for master-6-17-stable branch > > https://reviews.apache.org/media/uploaded/files/2013/06/29/0001-CLOUDSTACK-3110-VMWARE-NPE-while-adding-primary-stor.patch > > > Thanks, > > Venkata Siva Vijayendra Bhamidipati > >