cloudstack-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Sateesh Chodapuneedi" <sateesh.chodapune...@citrix.com>
Subject Re: Review Request 12180: Fix NPE when adding invalid primary datastore
Date Sat, 29 Jun 2013 07:24:56 GMT

-----------------------------------------------------------
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
> 
>


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message